@backstage/plugin-catalog-backend 1.3.1-next.0 → 1.3.1-next.1

package/CHANGELOG.md

@@ -1,5 +1,18 @@
 # @backstage/plugin-catalog-backend
 
+## 1.3.1-next.1
+
+### Patch Changes
+
+- e3d3018531: Fix issue for conditional decisions based on properties stored as arrays, like tags.
+
+  Before this change, having a permission policy returning conditional decisions based on metadata like tags, such like `createCatalogConditionalDecision(permission, catalogConditions.hasMetadata('tags', 'java'),)`, was producing wrong results. The issue occurred when authorizing entities already loaded from the database, for example when authorizing `catalogEntityDeletePermission`.
+
+- Updated dependencies
+  - @backstage/plugin-catalog-common@1.0.5-next.0
+  - @backstage/backend-common@0.15.0-next.1
+  - @backstage/integration@1.3.0-next.1
+
 ## 1.3.1-next.0
 
 ### Patch Changes

package/alpha/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@backstage/plugin-catalog-backend",
-  "version": "1.3.1-next.0",
+  "version": "1.3.1-next.1",
   "main": "../dist/index.cjs.js",
   "types": "../dist/index.alpha.d.ts"
 }

package/dist/index.cjs.js

@@ -886,6 +886,12 @@ const createPropertyRule = (propertyType) => createCatalogPermissionRule({
   resourceType: pluginCatalogCommon.RESOURCE_TYPE_CATALOG_ENTITY,
   apply: (resource, key, value) => {
     const foundValue = lodash.get(resource[propertyType], key);
+    if (Array.isArray(foundValue)) {
+      if (value !== void 0) {
+        return foundValue.includes(value);
+      }
+      return foundValue.length > 0;
+    }
     if (value !== void 0) {
       return value === foundValue;
     }
@@ -4026,6 +4032,7 @@ class CatalogBuilder {
           (resourceRef) => entitiesByRef[catalogModel.stringifyEntityRef(catalogModel.parseEntityRef(resourceRef))]
         );
       },
+      permissions: pluginCatalogCommon.catalogPermissions,
       rules: this.permissionRules
     });
     const stitcher = new Stitcher(dbClient, logger);