@adobe/helix-config 3.13.0 → 3.13.1

package/CHANGELOG.md

@@ -1,3 +1,10 @@
+## [3.13.1](https://github.com/adobe/helix-config/compare/v3.13.0...v3.13.1) (2024-08-14)
+
+
+### Bug Fixes
+
+* respect only original-site ([#162](https://github.com/adobe/helix-config/issues/162)) ([155c485](https://github.com/adobe/helix-config/commit/155c4853ed9d3f16b455a181d1340e9a01690033))
+
 # [3.13.0](https://github.com/adobe/helix-config/compare/v3.12.0...v3.13.0) (2024-08-13)
 
 

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@adobe/helix-config",
-  "version": "3.13.0",
+  "version": "3.13.1",
   "description": "Helix Config",
   "main": "src/index.js",
   "types": "src/index.d.ts",

package/src/config-view.js

@@ -189,14 +189,18 @@ async function loadMetadata(ctx, config, partition) {
  * loads the original site information from the `.hlx.json` file
  * @param ctx the context
  * @param contentBusId the content bus id
+ * @param legacy if true, the original-repository is returned, otherwise the original-site.
  * @returns {Promise<string>} the original site.
  */
-async function fetchOriginalSite(ctx, contentBusId) {
+async function fetchOriginalSite(ctx, contentBusId, legacy) {
   const key = `${contentBusId}/.hlx.json`;
   const res = await ctx.loader.getObject(HELIX_CONTENT_BUS, key);
   if (res.body) {
     const json = res.json();
-    return json['original-site'] || json['original-repository'] || '';
+    if (legacy) {
+      return json['original-repository'];
+    }
+    return json['original-site'];
   }
   ctx.log.error(`failed to load ${key}: ${res.status}`);
   return '';
@@ -405,21 +409,19 @@ export async function getConfigResponse(ctx, opts) {
     });
   }
   // validate original-site
-  if (!config.legacy) {
-    const originalSite = await fetchOriginalSite(ctx, config.content.contentBusId);
-    if (originalSite && originalSite !== `${org}/${site}`) {
-      ctx.log.error(`original site ${originalSite} does not match requested ${org}/${site}.`);
-      if (scope === SCOPE_ADMIN) {
-        return new PipelineResponse('', {
-          status: 403,
-          headers: {
-            'x-error': 'original site mismatch',
-            ...surrogateHeaders,
-          },
-        });
-      }
-      // todo: also send 403 for all scopes....but first observe only
+  const originalSite = await fetchOriginalSite(ctx, config.content.contentBusId, config.legacy);
+  if (originalSite && originalSite !== `${org}/${site}`) {
+    ctx.log.error(`original site ${originalSite} does not match requested ${org}/${site}.`);
+    if (scope === SCOPE_ADMIN) {
+      return new PipelineResponse('', {
+        status: 403,
+        headers: {
+          'x-error': 'original site mismatch',
+          ...surrogateHeaders,
+        },
+      });
     }
+    // todo: also send 403 for all scopes....but first observe only
   }
 
   if (config.extends && scope !== SCOPE_RAW) {