Merge pull request #1 from KirtanSoni/remove-scope-param

Remove scope param
fostertheweb · Dec 22, 2023 · 1f81f15 · 1f81f15
2 parents 5d14751 + 6947cc4
commit 1f81f15
Show file tree

Hide file tree

Showing 3 changed files with 5 additions and 7 deletions.
diff --git a/README.md b/README.md
@@ -47,7 +47,7 @@ import { SpotifyApi } from '@spotify/web-api-ts-sdk';
 
 // Choose one of the following:
 const sdk = SpotifyApi.withUserAuthorization("client-id", "https://localhost:3000", ["scope1", "scope2"]);
-const sdk = SpotifyApi.withClientCredentials("client-id", "secret", ["scope1", "scope2"]);
+const sdk = SpotifyApi.withClientCredentials("client-id", "secret");
 ```
 
 Each of these factory methods will return a `SpotifyApi` instance, which you can use to make requests to the Spotify Web API.

diff --git a/src/SpotifyApi.ts b/src/SpotifyApi.ts
@@ -169,8 +169,8 @@ export class SpotifyApi {
         return new SpotifyApi(strategy, config);
     }
 
-    public static withClientCredentials(clientId: string, clientSecret: string, scopes: string[] = [], config?: SdkOptions): SpotifyApi {
-        const strategy = new ClientCredentialsStrategy(clientId, clientSecret, scopes);
+    public static withClientCredentials(clientId: string, clientSecret: string, config?: SdkOptions): SpotifyApi {//change
+        const strategy = new ClientCredentialsStrategy(clientId, clientSecret);
         return new SpotifyApi(strategy, config);
     }
 

diff --git a/src/auth/ClientCredentialsStrategy.ts b/src/auth/ClientCredentialsStrategy.ts
@@ -10,8 +10,7 @@ export default class ClientCredentialsStrategy implements IAuthStrategy {
 
     constructor(
         private clientId: string,
-        private clientSecret: string,
-        private scopes: string[] = []
+        private clientSecret: string
     ) {
     }
 
@@ -46,8 +45,7 @@ export default class ClientCredentialsStrategy implements IAuthStrategy {
 
     private async getTokenFromApi(): Promise<AccessToken> {
         const options = {
-            grant_type: 'client_credentials',
-            scope: this.scopes.join(' ')
+            grant_type: 'client_credentials'
         } as any;
 
         const bodyAsString = Object.keys(options).map(key => key + '=' + options[key]).join('&');