fixed 2 unit tests

uri_builder_tests:to_string_invalid_uri
uri_builder_tests:append_query_locale

Author: hanzhumsft

Release/include/cpprest/base_uri.h

@@ -227,7 +227,7 @@ namespace web {
         /// Creates a URI from the given URI components.
         /// </summary>
         /// <param name="components">A URI components object to create the URI instance.</param>
-        _ASYNCRTIMP uri(const details::uri_components &components) : m_components(components) { m_uri = m_components.join(); }
+        _ASYNCRTIMP uri(const details::uri_components &components);
 
         /// <summary>
         /// Creates a URI from the given encoded string. This will throw an exception if the string

Release/include/cpprest/http_headers.h

@@ -159,7 +159,7 @@ class http_headers
     {
         if (has(name))
         {
-            m_headers[name] =  m_headers[name].append(_XPLATSTR(", ")).append(utility::conversions::print_string(value));
+            m_headers[name].append(_XPLATSTR(", ")).append(utility::conversions::print_string(value));
         }
         else
         {

Release/include/cpprest/uri_builder.h

@@ -238,7 +238,7 @@ namespace web
         uri_builder &append_query(const utility::string_t &name, const T &value, bool do_encoding = true)
         {
             auto encodedName = name;
-            auto encodedValue = ::utility::conversions::print_string(value);
+            auto encodedValue = ::utility::conversions::print_string(value, std::locale::classic());
 
             if (do_encoding)
             {

Release/src/uri/uri.cpp

@@ -73,7 +73,7 @@ utility::string_t uri_components::join()
 
         if (m_port > 0)
         {
-            ret.append({ _XPLATSTR(':') }).append(utility::conversions::print_string(m_port));
+            ret.append({ _XPLATSTR(':') }).append(utility::conversions::print_string(m_port, std::locale::classic()));
         }
     }
 
@@ -104,6 +104,15 @@ utility::string_t uri_components::join()
 
 using namespace details;
 
+uri::uri(const details::uri_components &components) : m_components(components)
+{
+    m_uri = m_components.join();
+    if (!details::uri_parser::validate(m_uri))
+    {
+        throw uri_exception("provided uri is invalid: " + utility::conversions::to_utf8string(m_uri));
+    }
+}
+
 uri::uri(const utility::string_t &uri_string)
 {
     if (!details::uri_parser::parse(uri_string, m_components))