-
Notifications
You must be signed in to change notification settings - Fork 37.7k
Open
Labels
authenticationIssues with the Authentication platformIssues with the Authentication platformdebtCode quality issuesCode quality issues
Milestone
Description
Followup from #260061
We can now remove the timeout here
vscode/src/vs/workbench/services/accounts/common/defaultAccount.ts
Lines 266 to 279 in d226599
| private async getSessions(authProviderId: string): Promise<readonly AuthenticationSession[]> { | |
| for (let attempt = 1; attempt <= 3; attempt++) { | |
| try { | |
| return await this.authenticationService.getSessions(authProviderId, undefined, undefined, true); | |
| } catch (error) { | |
| this.logService.warn(`[DefaultAccount] Attempt ${attempt} to get sessions failed:`, getErrorMessage(error)); | |
| if (attempt === 3) { | |
| throw error; | |
| } | |
| await timeout(500); | |
| } | |
| } | |
| throw new Error('Unable to get sessions after multiple attempts'); | |
| } |
Copilot
Metadata
Metadata
Assignees
Labels
authenticationIssues with the Authentication platformIssues with the Authentication platformdebtCode quality issuesCode quality issues