fix: remove deprecated implicit grant strategy

Author: sspenst

README.md

@@ -68,7 +68,6 @@ console.table(items.artists.items.map((item) => ({
 
 - Authorization Code Flow with PKCE
 - Client Credentials Flow
-- Implicit Grant Flow
 - Mixed Server and Client Side Authentication
 
 We do auto-token refresh when expired and a refresh token is available.

example/script.ts

@@ -1,13 +1,13 @@
 import { SpotifyApi } from "../src/index";
 import AuthorizationCodeWithPKCEStrategy from "../src/auth/AuthorizationCodeWithPKCEStrategy";
 
-const implicitGrantStrategy = new AuthorizationCodeWithPKCEStrategy(
+const authStrategy = new AuthorizationCodeWithPKCEStrategy(
     import.meta.env.VITE_SPOTIFY_CLIENT_ID,
     import.meta.env.VITE_REDIRECT_TARGET,
     ['user-read-private', 'user-read-email', 'playlist-modify-public', 'playlist-modify-private, user-read-playback-state, user-modify-playback-state']
 );
 
-const spotify = new SpotifyApi(implicitGrantStrategy);
+const spotify = new SpotifyApi(authStrategy);
 const profile = await spotify.currentUser.profile();
 console.log(profile);
 

src/SpotifyApi.test.ts

@@ -6,7 +6,6 @@ import { FetchApiMock } from "./test/FetchApiMock";
 import { validAlbumResult } from "./test/data/validAlbumResult";
 import AuthorizationCodeWithPKCEStrategy from "./auth/AuthorizationCodeWithPKCEStrategy";
 import ClientCredentialsStrategy from "./auth/ClientCredentialsStrategy";
-import ImplicitGrantStrategy from "./auth/ImplicitGrantStrategy";
 import ProvidedAccessTokenStrategy from "./auth/ProvidedAccessTokenStrategy";
 import { AccessToken, SdkOptions } from "./types";
 import InMemoryCachingStrategy from "./caching/InMemoryCachingStrategy";
@@ -80,11 +79,6 @@ describe("SpotifyAPI Instance", () => {
             expect(sut["authenticationStrategy"].constructor.name).toBe(ClientCredentialsStrategy.name);
         });
 
-        it("can create an instance with the implicit grant strategy configured", async () => {
-            const sut = SpotifyApi.withImplicitGrant("client-id", "secret", ["scope1", "scope2"]);
-            expect(sut["authenticationStrategy"].constructor.name).toBe(ImplicitGrantStrategy.name);
-        });
-
         it("can create an instance with the provided access token strategy configured", async () => {
             const sut = SpotifyApi.withAccessToken("client-id", {} as AccessToken);
             expect(sut["authenticationStrategy"].constructor.name).toBe(ProvidedAccessTokenStrategy.name);

src/SpotifyApi.ts

@@ -15,7 +15,6 @@ import IAuthStrategy, { isEmptyAccessToken } from "./auth/IAuthStrategy.js";
 import UsersEndpoints from "./endpoints/UsersEndpoints.js";
 import CurrentUserEndpoints from "./endpoints/CurrentUserEndpoints.js";
 import ClientCredentialsStrategy from "./auth/ClientCredentialsStrategy.js";
-import ImplicitGrantStrategy from "./auth/ImplicitGrantStrategy.js";
 import AuthorizationCodeWithPKCEStrategy from "./auth/AuthorizationCodeWithPKCEStrategy.js";
 import DefaultResponseDeserializer from "./serialization/DefaultResponseDeserializer.js";
 import DefaultResponseValidator from "./responsevalidation/DefaultResponseValidator.js";
@@ -174,11 +173,6 @@ export class SpotifyApi {
         return new SpotifyApi(strategy, config);
     }
 
-    public static withImplicitGrant(clientId: string, redirectUri: string, scopes: string[] = [], config?: SdkOptions): SpotifyApi {
-        const strategy = new ImplicitGrantStrategy(clientId, redirectUri, scopes);
-        return new SpotifyApi(strategy, config);
-    }
-
     /**
      * Use this when you're running in a Node environment, and accepting the access token from a client-side `performUserAuthorization` call.
      * You can also use this method if you already have an access token and don't want to use the built-in authentication strategies.