Skip to content

Commit ba83460

Browse files
authored
Catch error when unable to get current user and show an error message. (#6784)
1 parent f4ab8eb commit ba83460

File tree

1 file changed

+5
-0
lines changed

1 file changed

+5
-0
lines changed

src/github/credentials.ts

Lines changed: 5 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -402,8 +402,13 @@ export class CredentialStore extends Disposable {
402402

403403
private setCurrentUser(github: GitHub): void {
404404
const getUser: ReturnType<typeof github.octokit.api.users.getAuthenticated> = new Promise(resolve => {
405+
Logger.debug('Getting current user', CredentialStore.ID);
405406
github.octokit.call(github.octokit.api.users.getAuthenticated, {}).then(result => {
407+
Logger.debug(`Got current user ${result.data.login}`, CredentialStore.ID);
406408
resolve(result);
409+
}).catch(e => {
410+
vscode.window.showErrorMessage(vscode.l10n.t('Unable to get the currently logged in user, GitHub Pull Requests will not work correctly'));
411+
Logger.error(`Failed to get current user: ${e}, ${e.message}`, CredentialStore.ID);
407412
});
408413
});
409414
github.currentUser = new Promise(resolve => {

0 commit comments

Comments
 (0)