Add CURLOPT_POSTFIELDSIZE to address SAM issue 1830

NREL · Aug 16, 2024 · 4d787dd · 4d787dd
1 parent edc2573
commit 4d787dd
Showing 1 changed file with 9 additions and 2 deletions.
diff --git a/src/easycurl.cpp b/src/easycurl.cpp
@@ -155,8 +155,15 @@ class wxEasyCurl::DLThread : public wxThread {
             wxString encoded = uri.BuildURI();
             curl_easy_setopt(curl, CURLOPT_URL, (const char *) encoded.c_str());
 
-            if (!m_sc->m_postData.IsEmpty())
-                curl_easy_setopt(curl, CURLOPT_POSTFIELDS, (const char *) m_sc->m_postData.c_str());
+            if (!m_sc->m_postData.IsEmpty()) {
+// SAM issue 1830
+                auto len = m_sc->m_postData.length();
+                curl_easy_setopt(curl, CURLOPT_POSTFIELDSIZE, len);
+                curl_easy_setopt(curl, CURLOPT_POSTFIELDS, (const char*)m_sc->m_postData.c_str());
+                //                wxURI uriPostdata(m_sc->m_postData);
+//                wxString encodedPostData = uriPostdata.BuildURI();
+//                curl_easy_setopt(curl, CURLOPT_POSTFIELDS, (const char*)encodedPostData.c_str());
+            }
 
             curl_easy_setopt(curl, CURLOPT_SSL_VERIFYPEER, 0L);
             curl_easy_setopt(curl, CURLOPT_SSL_VERIFYHOST, 0L);