Use querystring for authorize url

nioc · nioc · commit 6157255ecdd5 · 2021-07-27T02:22:06.000+02:00
diff --git a/lib/index.js b/lib/index.js
@@ -88,7 +88,13 @@ class NetatmoClient {
       throw new Error('Redirect url must be provided')
     }
     this.state = statePrefix + Math.random() * 10000000000000000
-    return `${baseURL}/oauth2/authorize?client_id=${this.clientId}&redirect_uri=${this.redirectUrl}&scope=${this.scope}&state=${this.state}`
+    const query = querystring.stringify({
+      client_id: this.clientId,
+      redirect_uri: this.redirectUrl,
+      scope: this.scope,
+      state: this.state,
+    })
+    return `${baseURL}/oauth2/authorize?${query}`
   }
 
   /**

Original file line number	Diff line number	Diff line change
`@@ -88,7 +88,13 @@ class NetatmoClient {`
`88`	`88`	`throw new Error('Redirect url must be provided')`
`89`	`89`	`}`
`90`	`90`	`this.state = statePrefix + Math.random() * 10000000000000000`
`91`		- return `${baseURL}/oauth2/authorize?client_id=${this.clientId}&redirect_uri=${this.redirectUrl}&scope=${this.scope}&state=${this.state}`
	`91`	`+ const query = querystring.stringify({`
	`92`	`+ client_id: this.clientId,`
	`93`	`+ redirect_uri: this.redirectUrl,`
	`94`	`+ scope: this.scope,`
	`95`	`+ state: this.state,`
	`96`	`+ })`
	`97`	+ return `${baseURL}/oauth2/authorize?${query}`
`92`	`98`	`}`
`93`	`99`
`94`	`100`	`/**`