@adobe/helix-html-pipeline 3.11.5 → 3.11.7

package/CHANGELOG.md

@@ -1,3 +1,17 @@
+## [3.11.7](https://github.com/adobe/helix-html-pipeline/compare/v3.11.6...v3.11.7) (2023-05-22)
+
+
+### Bug Fixes
+
+* folder mapping should only be applied if direct addressed resource is missing ([#307](https://github.com/adobe/helix-html-pipeline/issues/307)) ([cf9cfc0](https://github.com/adobe/helix-html-pipeline/commit/cf9cfc0f00aed7a8fd7a502d0b2c77e9aa6e710d))
+
+## [3.11.6](https://github.com/adobe/helix-html-pipeline/compare/v3.11.5...v3.11.6) (2023-05-13)
+
+
+### Bug Fixes
+
+* **deps:** update dependency @adobe/helix-markdown-support to v6.1.2 ([75bbcdb](https://github.com/adobe/helix-html-pipeline/commit/75bbcdb3db030df16b0b6c0d8695fdc1f5513e3d))
+
 ## [3.11.5](https://github.com/adobe/helix-html-pipeline/compare/v3.11.4...v3.11.5) (2023-05-04)
 
 

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@adobe/helix-html-pipeline",
-  "version": "3.11.5",
+  "version": "3.11.7",
   "description": "Helix HTML Pipeline",
   "main": "src/index.js",
   "types": "src/index.d.ts",
@@ -39,7 +39,7 @@
     "node": ">=16.x"
   },
   "dependencies": {
-    "@adobe/helix-markdown-support": "6.1.1",
+    "@adobe/helix-markdown-support": "6.1.2",
     "@adobe/helix-shared-utils": "3.0.0",
     "@adobe/mdast-util-gridtables": "1.0.6",
     "@adobe/remark-gridtables": "1.0.2",
@@ -50,7 +50,7 @@
     "hast-util-to-html": "8.0.4",
     "hast-util-to-string": "2.0.0",
     "hastscript": "7.2.0",
-    "jose": "4.14.3",
+    "jose": "4.14.4",
     "mdast-util-gfm-footnote": "1.0.2",
     "mdast-util-gfm-strikethrough": "1.0.3",
     "mdast-util-gfm-table": "1.0.7",
@@ -59,15 +59,15 @@
     "mdast-util-to-string": "3.2.0",
     "micromark-extension-gfm-footnote": "1.1.0",
     "micromark-extension-gfm-strikethrough": "1.0.5",
-    "micromark-extension-gfm-table": "1.0.5",
+    "micromark-extension-gfm-table": "1.0.6",
     "micromark-extension-gfm-tagfilter": "1.0.2",
     "micromark-extension-gfm-task-list-item": "1.0.4",
     "micromark-util-combine-extensions": "1.0.0",
     "mime": "3.0.0",
     "rehype-format": "4.0.1",
     "rehype-parse": "8.0.4",
-    "remark-parse": "10.0.1",
-    "strip-markdown": "5.0.0",
+    "remark-parse": "10.0.2",
+    "strip-markdown": "5.0.1",
     "unified": "10.1.2",
     "unist-util-map": "3.1.3",
     "unist-util-remove": "3.1.1",
@@ -83,14 +83,14 @@
     "@semantic-release/git": "10.0.1",
     "@semantic-release/npm": "10.0.3",
     "c8": "7.13.0",
-    "eslint": "8.39.0",
+    "eslint": "8.41.0",
     "eslint-import-resolver-exports": "1.0.0-beta.5",
     "eslint-plugin-header": "3.1.1",
     "eslint-plugin-import": "2.27.5",
-    "esmock": "2.2.1",
+    "esmock": "2.2.3",
     "husky": "8.0.3",
     "js-yaml": "4.1.0",
-    "jsdom": "21.1.1",
+    "jsdom": "22.0.0",
     "junit-report-builder": "3.0.1",
     "lint-staged": "13.2.2",
     "mocha": "10.2.0",

package/src/html-pipe.js

@@ -82,13 +82,21 @@ export async function htmlPipe(state, req) {
     }
 
     // ...and apply the folder mapping
-    await folderMapping(state);
+    state.timer?.update('content-fetch');
+    let contentPromise = await fetchContent(state, req, res);
+    // ...but only if the current resource doesn't exist
+    if (res.status === 404) {
+      await folderMapping(state);
+      if (state.info.unmappedPath) {
+        contentPromise = fetchContent(state, req, res);
+      }
+    }
 
     // load metadata and content in parallel
-    state.timer?.update('content-fetch');
+    state.timer?.update('metadata-fetch');
     await Promise.all([
       fetchConfigAll(state, req, res),
-      fetchContent(state, req, res),
+      contentPromise,
       fetchMappedMetadata(state),
     ]);
 

package/src/json-pipe.js

@@ -54,6 +54,8 @@ async function fetchJsonContent(state, req, res) {
     ret = await s3Loader.getObject('helix-code-bus', `${owner}/${repo}/${ref}${path}`);
   }
   if (ret.status === 200) {
+    res.status = 200;
+    delete res.error;
     state.content.data = ret.body;
 
     // store extra source location if present
@@ -117,7 +119,6 @@ export async function jsonPipe(state, req) {
         },
       });
     }
-    await folderMapping(state);
 
     /** @type PipelineResponse */
     const res = new PipelineResponse('', {
@@ -126,10 +127,20 @@ export async function jsonPipe(state, req) {
       },
     });
 
+    // apply the folder mapping if the current resource doesn't exist
     state.timer?.update('json-fetch');
+    let contentPromise = await fetchJsonContent(state, req, res);
+    if (res.status === 404) {
+      await folderMapping(state);
+      if (state.info.unmappedPath) {
+        contentPromise = fetchJsonContent(state, req, res);
+      }
+    }
+
+    state.timer?.update('json-metadata-fetch');
     await Promise.all([
       fetchConfigAll(state, req, res),
-      fetchJsonContent(state, req, res),
+      contentPromise,
     ]);
 
     await authenticate(state, req, res);

package/src/steps/fetch-content.js

@@ -48,6 +48,8 @@ export default async function fetchContent(state, req, res) {
   }
 
   if (ret.status === 200) {
+    res.status = 200;
+    delete res.error;
     state.content.data = ret.body;
 
     // store extra source location if present