From 0734a055ff84317bf1128fdf9071b40a849dfe84 Mon Sep 17 00:00:00 2001
From: Richard Farr <rfarr@oanda.com>
Date: Fri, 2 Jan 2015 14:17:34 -0500
Subject: [PATCH] Allow client to quit gracefully when an end of file condition
 occurs mid tick

    - Also updated documentation to stress verification of SSL credentials
    - Changed client to require TLSv1 since SSL3.0 is insecure
    - Cleaned up extraneous whitespace
---
 streaming.cpp | 19 +++++++++++++------
 1 file changed, 13 insertions(+), 6 deletions(-)

diff --git a/streaming.cpp b/streaming.cpp
index e021fb3..d38d632 100644
--- a/streaming.cpp
+++ b/streaming.cpp
@@ -39,13 +39,18 @@ void handleStream(streambuf* stream_buffer)
 
     while (iit!=eos) {
         ostringstream oss;
-        while (*iit != '\n') {
+        while ( !(*iit == '\n' || iit == eos) ) {
             oss << *iit++;
         }
-        
-        //print the tick 
+
+        //connection closed before tick completed
+        if (iit == eos) {
+            break;
+        }
+
+        //print the tick
         cout << oss.str() << endl;
-        
+
         *iit++;
     }
 }
@@ -53,11 +58,13 @@ void handleStream(streambuf* stream_buffer)
 int main ()
 {
     try {
-        const Context::Ptr context = new Context(Context::CLIENT_USE, "", "", "", Context::VERIFY_NONE, 9, false, "ALL:!ADH:!LOW:!EXP:!MD5:@STRENGTH");
+        // example code - do not verify SSL credentials. Production clients MUST
+        // verify the SSL certificate chain to ensure rates are authentic
+        const Context::Ptr context = new Context(Context::TLSV1_CLIENT_USE, "", "", "", Context::VERIFY_NONE, 9, false, "ALL:!ADH:!LOW:!EXP:!MD5:@STRENGTH");
 
         // prepare session
         URI uri(domain + std::string("/v1/prices?accountId=") + account_id + std::string("&instruments=") + instruments);
-            
+
         HTTPSClientSession session(uri.getHost(), uri.getPort(), context);
         session.setKeepAlive(true);