fix: add explicit Accept-Encoding header to http requests

fixes #648
panva · Dec 22, 2023 · abcb564 · abcb564
1 parent 4b42b48
commit abcb564
Show file tree

Hide file tree

Showing 4 changed files with 11 additions and 5 deletions.
diff --git a/docs/README.md b/docs/README.md
@@ -569,7 +569,10 @@ requests.
 
 ```js
 const DEFAULT_HTTP_OPTIONS = {
-  headers: { 'User-Agent': `${pkg.name}/${pkg.version} (${pkg.homepage})` },
+  headers: {
+    'User-Agent': `${pkg.name}/${pkg.version} (${pkg.homepage})`,
+    'Accept-Encoding': 'identity',
+  },
   timeout: 3500,
 };
 ```

diff --git a/lib/helpers/request.js b/lib/helpers/request.js
@@ -39,7 +39,10 @@ const setDefaults = (props, options) => {
 };
 
 setDefaults([], {
-  headers: { 'User-Agent': `${pkg.name}/${pkg.version} (${pkg.homepage})` },
+  headers: {
+    'User-Agent': `${pkg.name}/${pkg.version} (${pkg.homepage})`,
+    'Accept-Encoding': 'identity',
+  },
   timeout: 3500,
 });
 

diff --git a/lib/issuer.js b/lib/issuer.js
@@ -138,7 +138,6 @@ class Issuer {
   }
 
   static async discover(uri) {
-
     const wellKnownUri = resolveWellKnownUri(uri);
 
     const response = await request.call(this, {

diff --git a/test/client/client_instance.test.js b/test/client/client_instance.test.js
@@ -210,9 +210,10 @@ describe('Client', () => {
           state: 'state',
           scope: 'openid profile email',
         }),
-      ).to.eql('https://op.example.com/auth?client_id=identifier&scope=openid%20profile%20email&response_type=code&state=state');
+      ).to.eql(
+        'https://op.example.com/auth?client_id=identifier&scope=openid%20profile%20email&response_type=code&state=state',
+      );
     });
-
   });
 
   describe('#endSessionUrl', function () {