Skip to content

Narrow token-refresh and account-info client dependencies #8110

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Closed
isaacroldan wants to merge 1 commit into from
+5 −2
Closed

Narrow token-refresh and account-info client dependencies #8110

Changes from all commits
Commits
Show all changes
1 commit
Select commit
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
7 changes: 5 additions & 2 deletions packages/app/src/cli/utilities/developer-platform-client.ts
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -263,7 +263,10 @@ export interface MigrationDeveloperPlatformClient {
migrateToUiExtension: (input: MigrateToUiExtensionVariables) => Promise<MigrateToUiExtensionSchema>
}

const inProgressRefreshes = new WeakMap<DeveloperPlatformClient, Promise<string>>()
/** The subset of a developer platform client needed to refresh an expired token. */
type TokenRefreshableClient = Pick<DeveloperPlatformClient, 'session' | 'unsafeRefreshToken'>

const inProgressRefreshes = new WeakMap<TokenRefreshableClient, Promise<string>>()

/**
* Creates an unauthorized handler for a developer platform client that will refresh the token
Expand All @@ -275,7 +278,7 @@ const inProgressRefreshes = new WeakMap<DeveloperPlatformClient, Promise<string>
* @returns The unauthorized handler.
*/
export function createUnauthorizedHandler(
client: DeveloperPlatformClient,
client: TokenRefreshableClient,
tokenType: 'default' | 'businessPlatform' = 'default',
): UnauthorizedHandler {
return {
Expand Down
Loading