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

package/CHANGELOG.md

@@ -1,5 +1,20 @@
 # @backstage/plugin-catalog-backend
 
+## 1.4.0-next.1
+
+### Minor Changes
+
+- dd395335bc: Allow unknown typed location from being registered via the location service by configuration settings
+- 651c9d6800: The search index now does retain fields that have a very long value, but in the form of just a null. This makes it possible to at least filter for their existence.
+
+### Patch Changes
+
+- ce77e78c93: Fixes a bug to be able to utilize refresh keys after the entity is loaded from cache
+- 679f7c5e95: Include entity ref into error message when catalog policies fail
+- Updated dependencies
+  - @backstage/plugin-permission-node@0.6.5-next.1
+  - @backstage/backend-common@0.15.1-next.1
+
 ## 1.3.2-next.0
 
 ### Patch Changes

package/alpha/package.json

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

package/dist/index.alpha.d.ts

@@ -171,7 +171,8 @@ export declare class CatalogBuilder {
     private onProcessingError?;
     private processingInterval;
     private locationAnalyzer;
-    private permissionRules;
+    private readonly permissionRules;
+    private allowedLocationType;
     /**
      * Creates a catalog builder.
      */
@@ -293,6 +294,12 @@ export declare class CatalogBuilder {
      * @alpha
      */
     addPermissionRules(...permissionRules: Array<CatalogPermissionRule | Array<CatalogPermissionRule>>): void;
+    /**
+     * Sets up the allowed location types from being registered via the location service.
+     *
+     * @param allowedLocationTypes - the allowed location types
+     */
+    setAllowedLocationTypes(allowedLocationTypes: string[]): CatalogBuilder;
     /**
      * Wires up and returns all of the component parts of the catalog
      */

package/dist/index.beta.d.ts

@@ -171,7 +171,8 @@ export declare class CatalogBuilder {
     private onProcessingError?;
     private processingInterval;
     private locationAnalyzer;
-    private permissionRules;
+    private readonly permissionRules;
+    private allowedLocationType;
     /**
      * Creates a catalog builder.
      */
@@ -285,6 +286,12 @@ export declare class CatalogBuilder {
      */
     setEntityDataParser(parser: CatalogProcessorParser): CatalogBuilder;
     /* Excluded from this release type: addPermissionRules */
+    /**
+     * Sets up the allowed location types from being registered via the location service.
+     *
+     * @param allowedLocationTypes - the allowed location types
+     */
+    setAllowedLocationTypes(allowedLocationTypes: string[]): CatalogBuilder;
     /**
      * Wires up and returns all of the component parts of the catalog
      */

package/dist/index.cjs.js

@@ -754,6 +754,7 @@ class UrlReaderProcessor {
         for (const parseResult of cacheItem.value) {
           emit(parseResult);
         }
+        emit(pluginCatalogNode.processingResult.refresh(`${location.type}:${location.target}`));
       } else if (error.name === "NotFoundError") {
         if (!optional) {
           emit(pluginCatalogNode.processingResult.notFoundError(location, message));
@@ -2261,13 +2262,20 @@ function progressTracker() {
 }
 
 class DefaultLocationService {
-  constructor(store, orchestrator) {
+  constructor(store, orchestrator, options = {
+    allowedLocationTypes: ["url"]
+  }) {
     this.store = store;
     this.orchestrator = orchestrator;
+    this.options = options;
   }
   async createLocation(input, dryRun) {
-    if (input.type !== "url") {
-      throw new errors.InputError(`Registered locations must be of type 'url'`);
+    if (!this.options.allowedLocationTypes.includes(input.type)) {
+      throw new errors.InputError(
+        `Registered locations must be of an allowed type ${JSON.stringify(
+          this.options.allowedLocationTypes
+        )}`
+      );
     }
     if (dryRun) {
       return this.dryRunCreateLocation(input);
@@ -2823,10 +2831,17 @@ class DefaultCatalogProcessingOrchestrator {
     try {
       policyEnforcedEntity = await this.options.policy.enforce(entity);
     } catch (e) {
-      throw new errors.InputError("Policy check failed", e);
+      throw new errors.InputError(
+        `Policy check failed for ${catalogModel.stringifyEntityRef(entity)}`,
+        e
+      );
     }
     if (!policyEnforcedEntity) {
-      throw new Error("Policy unexpectedly returned no data");
+      throw new Error(
+        `Policy unexpectedly returned no data for ${catalogModel.stringifyEntityRef(
+          entity
+        )}`
+      );
     }
     return policyEnforcedEntity;
   }
@@ -2996,8 +3011,12 @@ function mapToRows(input, entityId) {
       result.push({ entity_id: entityId, key, value: null });
     } else {
       const value = String(rawValue).toLocaleLowerCase("en-US");
-      if (key.length <= MAX_KEY_LENGTH && value.length <= MAX_VALUE_LENGTH) {
-        result.push({ entity_id: entityId, key, value });
+      if (key.length <= MAX_KEY_LENGTH) {
+        result.push({
+          entity_id: entityId,
+          key,
+          value: value.length <= MAX_VALUE_LENGTH ? value : null
+        });
       }
     }
   }
@@ -3920,6 +3939,7 @@ class CatalogBuilder {
     this.processorsReplace = false;
     this.parser = void 0;
     this.permissionRules = Object.values(permissionRules);
+    this.allowedLocationType = ["url"];
   }
   static create(env) {
     return new CatalogBuilder(env);
@@ -3986,6 +4006,10 @@ class CatalogBuilder {
   addPermissionRules(...permissionRules) {
     this.permissionRules.push(...permissionRules.flat());
   }
+  setAllowedLocationTypes(allowedLocationTypes) {
+    this.allowedLocationType = allowedLocationTypes;
+    return this;
+  }
   async build() {
     var _a, _b;
     const { config, database, logger, permissions } = this.env;
@@ -4071,7 +4095,9 @@ class CatalogBuilder {
     );
     const locationAnalyzer = (_b = this.locationAnalyzer) != null ? _b : new RepoLocationAnalyzer(logger, integrations);
     const locationService = new AuthorizedLocationService(
-      new DefaultLocationService(locationStore, orchestrator),
+      new DefaultLocationService(locationStore, orchestrator, {
+        allowedLocationTypes: this.allowedLocationType
+      }),
       permissionEvaluator
     );
     const refreshService = new AuthorizedRefreshService(