Merge pull request #59 from auth0/patch-headers

Add content-type header to the jwks request
auth0 · May 2, 2019 · e6ec173 · e6ec173
2 parents 9deba21 + 9fa6841
commit e6ec173
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 1 deletion.
diff --git a/src/main/java/com/auth0/jwk/UrlJwkProvider.java b/src/main/java/com/auth0/jwk/UrlJwkProvider.java
@@ -96,6 +96,7 @@ private Map<String, Object> getJwks() throws SigningKeyNotFoundException {
             if (readTimeout != null) {
                 c.setReadTimeout(readTimeout);
             }
+            c.setRequestProperty("Accept", "application/json");
             final InputStream inputStream = c.getInputStream();
             final JsonFactory factory = new JsonFactory();
             final JsonParser parser = factory.createParser(inputStream);

diff --git a/src/test/java/com/auth0/jwk/UrlJwkProviderTest.java b/src/test/java/com/auth0/jwk/UrlJwkProviderTest.java
@@ -198,7 +198,7 @@ protected URLConnection openConnection(URL url) throws IOException {
     }
 
     @Test
-    public void shouldConfigureURLConnectionTimeouts() throws Exception {
+    public void shouldConfigureURLConnection() throws Exception {
         URLConnection urlConnection = mock(URLConnection.class);
 
         // Although somewhat of a hack, this approach gets the job done - this method can 
@@ -213,12 +213,16 @@ public void shouldConfigureURLConnectionTimeouts() throws Exception {
         Jwk jwk = urlJwkProvider.get("NkJCQzIyQzRBMEU4NjhGNUU4MzU4RkY0M0ZDQzkwOUQ0Q0VGNUMwQg");
         assertNotNull(jwk);
 
+        //Request Timeout assertions
         ArgumentCaptor<Integer> connectTimeoutCaptor = ArgumentCaptor.forClass(Integer.class);
         verify(urlConnection).setConnectTimeout(connectTimeoutCaptor.capture());
         assertThat(connectTimeoutCaptor.getValue(), is(connectTimeout));
 
         ArgumentCaptor<Integer> readTimeoutCaptor = ArgumentCaptor.forClass(Integer.class);
         verify(urlConnection).setReadTimeout(readTimeoutCaptor.capture());
         assertThat(readTimeoutCaptor.getValue(), is(readTimeout));
+
+        //Request Headers assertions
+        verify(urlConnection).setRequestProperty("Accept", "application/json");
     }
 }