We read every piece of feedback, and take your input very seriously.
To see all available qualifiers, see our documentation.
There was an error while loading. Please reload this page.
1 parent 3435d5c commit 9459bf6Copy full SHA for 9459bf6
packages/snaps-controllers/src/snaps/SnapController.ts
@@ -4305,6 +4305,10 @@ export class SnapController extends BaseController<
4305
4306
// If the permission doesn't have dependencies, or if at least one of
4307
// its dependencies is desired, include it in the desired permissions.
4308
+ // NOTE: This effectively means that any permissions granted in the manifest
4309
+ // that are considered dynamic, will not be automatically revoked
4310
+ // when the permission is removed from the manifest.
4311
+ // TODO: Deal with this technical debt.
4312
if (!hasDependencies || hasDependency) {
4313
accumulator[permissionName] = oldPermissions[permissionName];
4314
}
0 commit comments