Skip to content

Commit 4ec35c8

Browse files
Only include installed extensions (microsoft#229889)
Fixes microsoft#229886
1 parent 589cd8a commit 4ec35c8

File tree

1 file changed

+6
-1
lines changed

1 file changed

+6
-1
lines changed

src/vs/workbench/contrib/authentication/browser/actions/manageTrustedExtensionsForAccountAction.ts

Lines changed: 6 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -107,7 +107,12 @@ class ManageTrustedExtensionsForAccountActionImpl {
107107
}
108108

109109
private async _getItems(providerId: string, accountLabel: string) {
110-
const allowedExtensions = this._authenticationAccessService.readAllowedExtensions(providerId, accountLabel);
110+
let allowedExtensions = this._authenticationAccessService.readAllowedExtensions(providerId, accountLabel);
111+
// only include extensions that are installed
112+
const resolvedExtensions = await Promise.all(allowedExtensions.map(ext => this._extensionService.getExtension(ext.id)));
113+
allowedExtensions = resolvedExtensions
114+
.map((ext, i) => ext ? allowedExtensions[i] : undefined)
115+
.filter(ext => !!ext);
111116
const trustedExtensionAuthAccess = this._productService.trustedExtensionAuthAccess;
112117
const trustedExtensionIds =
113118
// Case 1: trustedExtensionAuthAccess is an array

0 commit comments

Comments
 (0)