Append extra params to URL

Author: reedmclean

TinCan/RemoteLRS.cs

@@ -81,6 +81,20 @@ public MyHTTPResponse(HttpWebResponse webResp)
             }
         }
 
+        private string AppendParamsToExistingQueryString(string currentQueryString, Dictionary<string, string> parameters)
+        {
+            foreach (KeyValuePair<String, String> entry in parameters)
+            {
+                if (currentQueryString != "")
+                {
+                    currentQueryString += "&";
+                }
+                currentQueryString += HttpUtility.UrlEncode(entry.Key) + "=" + HttpUtility.UrlEncode(entry.Value);
+            }
+
+            return currentQueryString;
+        }
+
         private MyHTTPResponse MakeSyncRequest(MyHTTPRequest req)
         {
             String url;
@@ -97,21 +111,13 @@ private MyHTTPResponse MakeSyncRequest(MyHTTPRequest req)
                 url += req.resource;
             }
 
-            if (req.queryParams != null)
+            String qs = "";
+            qs = AppendParamsToExistingQueryString(qs, req.queryParams);
+            qs = AppendParamsToExistingQueryString(qs, extended);
+
+            if (qs != "")
             {
-                String qs = "";
-                foreach (KeyValuePair<String, String> entry in req.queryParams)
-                {
-                    if (qs != "")
-                    {
-                        qs += "&";
-                    }
-                    qs += HttpUtility.UrlEncode(entry.Key) + "=" + HttpUtility.UrlEncode(entry.Value);
-                }
-                if (qs != "")
-                {
-                    url += "?" + qs;
-                }
+                url += "?" + qs;
             }
 
             // TODO: handle special properties we recognize, such as content type, modified since, etc.
@@ -123,13 +129,11 @@ private MyHTTPResponse MakeSyncRequest(MyHTTPRequest req)
             {
                 webReq.Headers.Add("Authorization", auth);
             }
-            if (req.headers != null)
+            foreach (KeyValuePair<String, String> entry in req.headers)
             {
-                foreach (KeyValuePair<String, String> entry in req.headers)
-                {
-                    webReq.Headers.Add(entry.Key, entry.Value);
-                }
+                webReq.Headers.Add(entry.Key, entry.Value);
             }
+            
 
             if (req.contentType != null)
             {

TinCanTests/RemoteLRSResourceTest.cs

@@ -339,5 +339,23 @@ public void TestDeleteAgentProfile()
             LRSResponse lrsRes = lrs.DeleteAgentProfile(doc);
             Assert.IsTrue(lrsRes.success);
         }
+
+        [Test]
+        public void TestExtendedParameters()
+        {
+            // RemoteLRS doesn't provide a helpful interface for testing
+            // that we successfully altered the request URL, but this test
+            // is helpful in manual testing and it at least ensures that
+            // specifying values in extended doesn't cause errors.
+            lrs.extended.Add("test", "param");
+            var doc = new StateDocument();
+            doc.activity = Support.activity;
+            doc.agent = Support.agent;
+            doc.id = "test";
+            doc.content = System.Text.Encoding.UTF8.GetBytes("Test value");
+
+            LRSResponse lrsRes = lrs.SaveState(doc);
+            Assert.IsTrue(lrsRes.success);
+        }
     }
 }