@forge/manifest
12.1.0-next.612.1.0-next.7
out/validators/permissions-validator.js~
out/validators/permissions-validator.jsModified+7−17
Index: package/out/validators/permissions-validator.js
===================================================================
--- package/out/validators/permissions-validator.js
+++ package/out/validators/permissions-validator.js
@@ -210,9 +210,9 @@
perms: manifest.typedContent.permissions.external?.scripts?.map((key) => typeof key === 'object' ? key.address : key)
}
];
mapping.forEach((item) => this.validateExternalPermissionURLs(errors, item.element, item.perms, manifest));
- const { fetch: originalFetch, ...restOfExternalPermissions } = manifest.typedContent.permissions.external || {};
+ const { fetch: originalFetch, configurable: _configurable, ...restOfExternalPermissions } = manifest.typedContent.permissions.external || {};
Object.entries(originalFetch || {}).forEach(([key, values]) => {
if (values.some((value) => typeof value === 'string')) {
errors.push({
message: text_1.errors.permissions.deprecatedEgressPermission(`fetch.${key}`),
@@ -239,26 +239,16 @@
}
});
const remoteMap = manifest.typedContent.remotes?.reduce((prev, item) => {
if ((0, features_1.configurableRemotesEnabled)()) {
- if (typeof item.configurable !== 'undefined') {
- if (typeof item.configurable.default !== 'undefined') {
- const baseUrl = item.configurable.default;
- return prev.set(item.key, baseUrl);
- }
- else {
- return prev.set(item.key, null);
- }
+ if (item.baseUrl) {
+ const baseUrl = typeof item.baseUrl === 'string' ? item.baseUrl : item.baseUrl.default;
+ return prev.set(item.key, baseUrl);
}
- else {
- if (typeof item.baseUrl !== 'undefined') {
- const baseUrl = typeof item.baseUrl === 'string' ? item.baseUrl : item.baseUrl.default;
- return prev.set(item.key, baseUrl);
- }
- else {
- return prev;
- }
+ if (!item.baseUrl && item.configurable) {
+ return prev.set(item.key, null);
}
+ return prev;
}
else {
if (item.baseUrl !== undefined) {
const baseUrl = typeof item.baseUrl === 'string' ? item.baseUrl : item.baseUrl.default;