Minor change in oauth1/2client samples to avoid duplicate replies from HTTP redirect listener.

Author: Valtteri Heikkila

Release/samples/Oauth1Client/Oauth1Client.cpp

@@ -99,28 +99,24 @@ class oauth1_code_listener
         m_listener->support([this](http::http_request request) -> void
         {
             pplx::extensibility::scoped_critical_section_t lck(m_resplock);
-            if (request.request_uri().path() == U("/") && request.request_uri().query() != U(""))
+            m_config.token_from_redirected_uri(request.request_uri()).then([this,request](pplx::task<void> token_task) -> void
             {
-                m_config.token_from_redirected_uri(request.request_uri()).then([this,request](pplx::task<void> token_task) -> void
+                if (request.request_uri().path() == U("/") && request.request_uri().query() != U(""))
                 {
                     try
                     {
                         token_task.wait();
-                        request.reply(status_codes::OK, U("Ok."));
                         m_tce.set(true);
                     }
                     catch (oauth1_exception& e)
                     {
                         ucout << "Error: " << e.what() << std::endl;
-                        request.reply(status_codes::NotFound, U("Not found."));
                         m_tce.set(false);
                     }
-                });
-            }
-            else
-            {
-                request.reply(status_codes::NotFound, U("Not found."));
-            }
+                }
+            });
+            
+            request.reply(status_codes::OK, U("Ok."));
         });
         m_listener->open().wait();
     }

Release/samples/Oauth2Client/Oauth2Client.cpp

@@ -99,28 +99,24 @@ class oauth2_code_listener
         m_listener->support([this](http::http_request request) -> void
         {
             pplx::extensibility::scoped_critical_section_t lck(m_resplock);
-            if (request.request_uri().path() == U("/") && request.request_uri().query() != U(""))
+            m_config.token_from_redirected_uri(request.request_uri()).then([this,request](pplx::task<void>token_task) -> void
             {
-                m_config.token_from_redirected_uri(request.request_uri()).then([this,request](pplx::task<void> token_task) -> void
+                if (request.request_uri().path() == U("/") && request.request_uri().query() != U(""))
                 {
                     try
                     {
                         token_task.wait();
-                        request.reply(status_codes::OK, U("Ok."));
                         m_tce.set(true);
                     }
-                    catch (oauth2_exception& e)
+                    catch (oauth1_exception& e)
                     {
                         ucout << "Error: " << e.what() << std::endl;
-                        request.reply(status_codes::NotFound, U("Not found."));
                         m_tce.set(false);
                     }
-                });
-            }
-            else
-            {
-                request.reply(status_codes::NotFound, U("Not found."));
-            }
+                }
+            });
+
+            request.reply(status_codes::OK, U("Ok."));
         });
         m_listener->open().wait();
     }