Preserve non-standard/unparseable query strings as-is in snippet output

This is particularly notable for cases like the &&& repro here, but is
also clearly visible in cases like ?a=b&c, which would often become
something more like '?a=b&c='.

This only applies when the queryString in the HAR is empty, which will
be rare, but HTTP Toolkit (and others) can use this to leave that blank
and send only the raw URL to preserve formatting at the cost of the
losing the nice structure query param setting code in the output.

Author: pimterry

src/index.js

@@ -219,20 +219,29 @@ HTTPSnippet.prototype.prepare = function (request) {
   // merge all possible queryString values
   request.queryObj = Object.assign(request.queryObj, request.uriObj.query)
 
-  // reset uriObj values for a clean url
-  request.uriObj.query = null
-  request.uriObj.search = null
-  request.uriObj.path = request.uriObj.pathname
-
-  // keep the base url clean of queryString
-  request.url = url.format(request.uriObj)
-
-  // update the uri object
-  request.uriObj.query = request.queryObj
-  request.uriObj.search = qs.stringify(request.queryObj)
-
-  if (request.uriObj.search) {
-    request.uriObj.path = request.uriObj.pathname + '?' + request.uriObj.search
+  // In some cases (unparseable query - preference to use raw in exporter) the queryString might
+  // be empty while the URL search is not. In that case, we prefer the URL search.
+  const hasQueryObj = Object.keys(request.queryObj).length > 0
+  if (hasQueryObj || !request.uriObj.search) {
+    // reset uriObj values for a clean url
+    request.uriObj.query = null
+    request.uriObj.search = null
+    request.uriObj.path = request.uriObj.pathname
+
+    // keep the base url clean of queryString
+    request.url = url.format(request.uriObj)
+
+    // update the uri object
+    request.uriObj.query = request.queryObj
+    request.uriObj.search = qs.stringify(request.queryObj)
+
+    if (request.uriObj.search) {
+      request.uriObj.path = request.uriObj.pathname + '?' + request.uriObj.search
+    }
+  } else {
+    // We have no request.queryObj (so snippets won't send query params in a structured way)
+    // We keep the queryString in request.url (so it's sent raw everywhere)
+    // request.fullUrl is recreated below (maybe mild fixed?) but preserves raw search etc
   }
 
   // construct a full url

test/fixtures/output/c/libcurl/unparseable-query.c

@@ -0,0 +1,6 @@
+CURL *hnd = curl_easy_init();
+
+curl_easy_setopt(hnd, CURLOPT_CUSTOMREQUEST, "GET");
+curl_easy_setopt(hnd, CURLOPT_URL, "http://mockbin.com/har?&&&");
+
+CURLcode ret = curl_easy_perform(hnd);

test/fixtures/output/clojure/clj_http/unparseable-query.clj

@@ -0,0 +1,3 @@
+(require '[clj-http.client :as client])
+
+(client/get "http://mockbin.com/har?&&&")

test/fixtures/output/csharp/httpclient/unparseable-query.cs

@@ -0,0 +1,12 @@
+var client = new HttpClient();
+var request = new HttpRequestMessage
+{
+    Method = HttpMethod.Get,
+    RequestUri = new Uri("http://mockbin.com/har?&&&"),
+};
+using (var response = await client.SendAsync(request))
+{
+    response.EnsureSuccessStatusCode();
+    var body = await response.Content.ReadAsStringAsync();
+    Console.WriteLine(body);
+}

test/fixtures/output/csharp/restsharp/unparseable-query.cs

@@ -0,0 +1,3 @@
+var client = new RestClient("http://mockbin.com/har?&&&");
+var request = new RestRequest(Method.GET);
+IRestResponse response = client.Execute(request);

test/fixtures/output/go/native/unparseable-query.go

@@ -0,0 +1,23 @@
+package main
+
+import (
+	"fmt"
+	"net/http"
+	"io"
+)
+
+func main() {
+
+	url := "http://mockbin.com/har?&&&"
+
+	req, _ := http.NewRequest("GET", url, nil)
+
+	res, _ := http.DefaultClient.Do(req)
+
+	defer res.Body.Close()
+	body, _ := io.ReadAll(res.Body)
+
+	fmt.Println(res)
+	fmt.Println(string(body))
+
+}

test/fixtures/output/http/1.1/unparseable-query

@@ -0,0 +1,2 @@
+GET /har?&&& HTTP/1.1
+Host: mockbin.com

test/fixtures/output/java/asynchttp/unparseable-query.java

@@ -0,0 +1,8 @@
+AsyncHttpClient client = new DefaultAsyncHttpClient();
+client.prepare("GET", "http://mockbin.com/har?&&&")
+  .execute()
+  .toCompletableFuture()
+  .thenAccept(System.out::println)
+  .join();
+
+client.close();

test/fixtures/output/java/nethttp/unparseable-query.java

@@ -0,0 +1,6 @@
+HttpRequest request = HttpRequest.newBuilder()
+    .uri(URI.create("http://mockbin.com/har?&&&"))
+    .method("GET", HttpRequest.BodyPublishers.noBody())
+    .build();
+HttpResponse<String> response = HttpClient.newHttpClient().send(request, HttpResponse.BodyHandlers.ofString());
+System.out.println(response.body());

test/fixtures/output/java/okhttp/unparseable-query.java

@@ -0,0 +1,8 @@
+OkHttpClient client = new OkHttpClient();
+
+Request request = new Request.Builder()
+  .url("http://mockbin.com/har?&&&")
+  .get()
+  .build();
+
+Response response = client.newCall(request).execute();