fix: remove all undefined/null values

Fixes #40

Author: orgads

dist/index.cjs

@@ -6,8 +6,10 @@ function oauth(axios, { url, ...credentials }) {
       ...credentials,
       ...moreCredentials
     };
-    if ("scope" in body && !body.scope) {
-      delete body.scope;
+    for (const key of Object.keys(body)) {
+      if (!body[key]) {
+        delete body[key];
+      }
     }
     return axios({
       url,

dist/index.mjs

@@ -4,8 +4,10 @@ function oauth(axios, { url, ...credentials }) {
       ...credentials,
       ...moreCredentials
     };
-    if ("scope" in body && !body.scope) {
-      delete body.scope;
+    for (const key of Object.keys(body)) {
+      if (!body[key]) {
+        delete body[key];
+      }
     }
     return axios({
       url,

src/oauth.js

@@ -5,9 +5,9 @@ export function oauth (axios, { url, ...credentials }) {
       ...moreCredentials
     }
 
-    // remove blank scope
-    if ('scope' in body && !body.scope) {
-      delete body.scope
+    // remove all blank values
+    for (const key of Object.keys(body)) {
+      if (!body[key]) { delete body[key] }
     }
 
     return axios({

src/oauth.test.js

@@ -70,6 +70,26 @@ describe('oauth()', function () {
       })
     })
 
+    test('should omit any null or undefined value', async function () {
+      const params = {
+        url: 'https://oauth.com/2.0/token',
+        grant_type: 'client_credentials',
+        client_id: undefined,
+        client_secret: undefined,
+        scope: null
+      }
+
+      const actualConfig = {}
+      const auth = oauth(fakeAxios(actualConfig), params)
+      await auth()
+
+      assert.deepStrictEqual(actualConfig, {
+        url: 'https://oauth.com/2.0/token',
+        method: 'post',
+        data: 'grant_type=client_credentials'
+      })
+    })
+
     test('should resolve to the OAuth token response', async function () {
       const expectedData = { access_token: 'FAKE_TOKEN', expires_in: 5 }
       const auth = oauth(fakeAxios({}, expectedData), {})