fix build break and unit tests break caused by payload coming from tfs

Author: hohong

Release/include/http_client.h

@@ -269,7 +269,7 @@ class http_client
     /// <param name="handler">A function object representing the pipeline stage.</param>
     void add_handler(std::function<pplx::task<http_response>(http_request, std::shared_ptr<http::http_pipeline_stage>)> handler)
     {
-        m_pipeline->append(std::make_shared<::web::http::details::function_pipeline_wrapper>(handler));
+        m_pipeline->append(std::make_shared< ::web::http::details::function_pipeline_wrapper>(handler));
     }
 
 	/// <summary>
@@ -399,7 +399,7 @@ class http_client
 
     void build_pipeline(const uri &base_uri, const http_client_config& client_config);
 
-    std::shared_ptr<::web::http::http_pipeline> m_pipeline;
+    std::shared_ptr< ::web::http::http_pipeline> m_pipeline;
 
 	uri _base_uri;
 };

Release/src/http/common/http_helpers.cpp

@@ -71,7 +71,7 @@ namespace details
        || _wcsicmp(ct, content_type_application_xjavascript.c_str()) == 0
        )			
 #else
-        if(   boost::iequals(content_type.substr(0,4), content_type_application_json)) 
+        if(   boost::iequals(content_type.substr(), content_type_application_json) 
            || boost::iequals(content_type, content_type_text_json)
            || boost::iequals(content_type, content_type_text_xjson) 
            || boost::iequals(content_type, content_type_text_javascript)

Release/tests/Functional/http/utilities/testlistener/include/http_listener.h

@@ -201,7 +201,7 @@ class http_listener : public http_listener_interface
     /// <param name="handler">A function object representing the pipeline stage.</param>
     void add_handler(std::function<pplx::task<http_response>(http_request, std::shared_ptr<http::http_pipeline_stage>)> handler)
     {
-        m_pipeline->append(std::make_shared<::web::http::details::function_pipeline_wrapper>(handler));
+        m_pipeline->append(std::make_shared< ::web::http::details::function_pipeline_wrapper>(handler));
     }
 
     /// <summary>
@@ -292,7 +292,7 @@ class http_listener : public http_listener_interface
             this->m_supported_methods = std::move(other.m_supported_methods);
             this->m_pipeline = std::move(other.m_pipeline);
 
-            std::shared_ptr<::web::http::http_pipeline_stage> lastStage = this->m_pipeline->last_stage();
+            std::shared_ptr< ::web::http::http_pipeline_stage> lastStage = this->m_pipeline->last_stage();
             auto listenerStage = static_cast<_listener_stage *>(lastStage.get());
             listenerStage->reset_listener(this);
 
@@ -340,4 +340,4 @@ class http_listener : public http_listener_interface
 } // namespace http
 } // namespace web
 
-#endif  /* _CASA_HTTP_LISTENER_H */
+#endif  /* _CASA_HTTP_LISTENER_H */

Release/tests/Functional/http/utilities/testlistener/src/logging/log.cpp

@@ -38,7 +38,7 @@ using namespace utility;
 using namespace logging;
 using namespace pplx;
 
-std::shared_ptr<::logging::_Log> s_default;
+std::shared_ptr< ::logging::_Log> s_default;
 
 void log::post(log_level level, const utility::string_t &message)
 {
@@ -64,21 +64,21 @@ void log::set_default(_Inout_opt_ ::logging::_Log *log)
 {
     if ( log == nullptr )
         log = new logging::_details::SilentLog;
-    s_default = std::shared_ptr<::logging::_Log>(log); 
+    s_default = std::shared_ptr< ::logging::_Log>(log); 
 }
 
-void log::set_default(std::shared_ptr<::logging::_Log> log)
+void log::set_default(std::shared_ptr< ::logging::_Log> log)
 {
     if ( !log )
         log = std::make_shared<logging::_details::SilentLog>();
     s_default = log; 
 }
 
-std::shared_ptr<::logging::_Log> log::get_default()
+std::shared_ptr< ::logging::_Log> log::get_default()
 { 
     if ( !s_default )
     {
-        s_default = std::shared_ptr<::logging::_Log>(new logging::_details::ConsoleLog(true, true));
+        s_default = std::shared_ptr< ::logging::_Log>(new logging::_details::ConsoleLog(true, true));
     }
     return s_default; 
 }