Changeset 15968 in josm for trunk/test/functional/org/openstreetmap/josm

Timestamp:

2020-03-01T01:02:22+01:00 (5 years ago)

Author:

simon04

Message:

fix #18812 - HttpClient: specify Accept=*/* to prevent Java from adding Accept=text/html, image/gif, image/jpeg, *; q=.2, */*; q=.2

File:

• trunk/test/functional/org/openstreetmap/josm/tools/HttpClientTest.java (modified) (2 diffs)

trunk/test/functional/org/openstreetmap/josm/tools/HttpClientTest.java

@@ -85 +85 @@
         assertThat(client.getURL(), is(new URL("https://httpbin.org/")));
         assertThat(client.getRequestMethod(), is("GET"));
-        assertThat(client.getRequestHeader("Accept"), nullValue());
+        assertThat(client.getRequestHeader("Accept"), is("*/*"));
         client.setAccept("text/html");
         assertThat(client.getRequestHeader("Accept"), is("text/html"));
@@ -119 +119 @@
             assertThat(root.getJsonObject("headers").get("Cache-Control"), nullValue());
             assertThat(root.getJsonObject("headers").get("Pragma"), nullValue());
+        }
+    }
+
+    /**
+     * Test JOSM User-Agent
+     * @throws IOException if an I/O error occurs
+     */
+    @Test
+    public void testHeaders() throws IOException {
+        try (InputStream in = HttpClient.create(new URL("https://httpbin.org/headers")).connect(progress).getContent();
+             JsonReader json = JsonProvider.provider().createReader(in)) {
+            final JsonObject headers = json.readObject().getJsonObject("headers");
+            assertThat(headers.getString("Accept"), is("*/*"));
+            assertThat(headers.getString("Accept-Encoding"), is("gzip, deflate"));
+            assertThat(headers.getString("User-Agent"), is(Version.getInstance().getFullAgentString()));
         }
     }