@adobe/helix-html-pipeline 1.8.2 → 1.8.4

package/CHANGELOG.md

@@ -1,3 +1,17 @@
+## [1.8.4](https://github.com/adobe/helix-html-pipeline/compare/v1.8.3...v1.8.4) (2022-10-22)
+
+
+### Bug Fixes
+
+* trim spaces after comma ([#167](https://github.com/adobe/helix-html-pipeline/issues/167)) ([e3e579a](https://github.com/adobe/helix-html-pipeline/commit/e3e579a690c2db7068bc5eabde4e0ce0e49084a3))
+
+## [1.8.3](https://github.com/adobe/helix-html-pipeline/compare/v1.8.2...v1.8.3) (2022-10-06)
+
+
+### Bug Fixes
+
+* **html:** do not skip custom headers for 3xx responses (json) ([396f3f4](https://github.com/adobe/helix-html-pipeline/commit/396f3f457c9acec852d51e427b1ca16a367b7bb3)), closes [#153](https://github.com/adobe/helix-html-pipeline/issues/153)
+
 ## [1.8.2](https://github.com/adobe/helix-html-pipeline/compare/v1.8.1...v1.8.2) (2022-10-05)
 
 

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@adobe/helix-html-pipeline",
-  "version": "1.8.2",
+  "version": "1.8.4",
   "description": "Helix HTML Pipeline",
   "main": "src/index.js",
   "types": "src/index.d.ts",
@@ -36,7 +36,7 @@
     "hast-util-select": "5.0.2",
     "hast-util-to-html": "8.0.3",
     "hast-util-to-string": "2.0.0",
-    "hastscript": "7.0.2",
+    "hastscript": "7.1.0",
     "mdast-util-gfm-footnote": "1.0.1",
     "mdast-util-gfm-strikethrough": "1.0.1",
     "mdast-util-gfm-table": "1.0.6",
@@ -69,14 +69,14 @@
     "@semantic-release/git": "10.0.1",
     "@semantic-release/npm": "9.0.1",
     "c8": "7.12.0",
-    "eslint": "8.24.0",
+    "eslint": "8.25.0",
     "eslint-import-resolver-exports": "1.0.0-beta.3",
     "eslint-plugin-header": "3.1.1",
     "eslint-plugin-import": "2.26.0",
-    "esmock": "2.0.4",
+    "esmock": "2.0.6",
     "husky": "8.0.1",
     "js-yaml": "4.1.0",
-    "jsdom": "20.0.0",
+    "jsdom": "20.0.1",
     "junit-report-builder": "3.0.1",
     "lint-staged": "13.0.3",
     "mocha": "10.0.0",

package/src/forms-pipe.js

@@ -130,7 +130,7 @@ export async function formsPipe(state, request) {
 
   const sourceLocation = resourceFetchResponse.headers.get('x-amz-meta-x-source-location');
   const referer = request.headers.get('referer') || 'unknown';
-  const sheetNames = sheets.split(',');
+  const sheetNames = sheets.split(',').map((s) => s.trim());
 
   if (!sourceLocation || !sheetNames.includes('incoming')) {
     return error(log, `Target workbook at ${resourcePath} is not setup to intake data.`, 403, response);

package/src/html-pipe.js

@@ -80,6 +80,9 @@ export async function htmlPipe(state, req) {
       // if content loading produced an error, we're done.
       log.error(`error running pipeline: ${res.status} ${res.error}`);
       res.headers.set('x-error', cleanupHeaderValue(res.error));
+      if (res.status < 500) {
+        await setCustomResponseHeaders(state, req, res);
+      }
       return res;
     }
 

package/src/json-pipe.js

@@ -89,6 +89,10 @@ export async function jsonPipe(state, req) {
 
   // if still not found, return status
   if (dataResponse.status !== 200) {
+    if (dataResponse.status < 500) {
+      await fetchMetadata(state, req, dataResponse);
+      await setCustomResponseHeaders(state, req, dataResponse);
+    }
     return dataResponse;
   }
   const data = dataResponse.body;