@adobe/helix-html-pipeline 1.8.4 → 1.8.6

package/CHANGELOG.md

@@ -1,3 +1,17 @@
+## [1.8.6](https://github.com/adobe/helix-html-pipeline/compare/v1.8.5...v1.8.6) (2022-11-01)
+
+
+### Bug Fixes
+
+* handle errors when loading config ([#181](https://github.com/adobe/helix-html-pipeline/issues/181)) ([a6e61a9](https://github.com/adobe/helix-html-pipeline/commit/a6e61a98cffe41bb22795c3a262c95cdf869f06f)), closes [#179](https://github.com/adobe/helix-html-pipeline/issues/179)
+
+## [1.8.5](https://github.com/adobe/helix-html-pipeline/compare/v1.8.4...v1.8.5) (2022-10-31)
+
+
+### Bug Fixes
+
+* **deps:** update dependency github-slugger to v2 (1.x) ([#177](https://github.com/adobe/helix-html-pipeline/issues/177)) ([fc42ffb](https://github.com/adobe/helix-html-pipeline/commit/fc42ffbec5daea2a9547bd58cf5c7b6ba489d153))
+
 ## [1.8.4](https://github.com/adobe/helix-html-pipeline/compare/v1.8.3...v1.8.4) (2022-10-22)
 
 

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@adobe/helix-html-pipeline",
-  "version": "1.8.4",
+  "version": "1.8.6",
   "description": "Helix HTML Pipeline",
   "main": "src/index.js",
   "types": "src/index.d.ts",
@@ -31,7 +31,7 @@
   "dependencies": {
     "@adobe/helix-markdown-support": "5.0.10",
     "@adobe/helix-shared-utils": "2.1.0",
-    "github-slugger": "1.4.0",
+    "github-slugger": "2.0.0",
     "hast-util-raw": "7.2.2",
     "hast-util-select": "5.0.2",
     "hast-util-to-html": "8.0.3",
@@ -69,17 +69,17 @@
     "@semantic-release/git": "10.0.1",
     "@semantic-release/npm": "9.0.1",
     "c8": "7.12.0",
-    "eslint": "8.25.0",
+    "eslint": "8.26.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.6",
+    "esmock": "2.0.7",
     "husky": "8.0.1",
     "js-yaml": "4.1.0",
     "jsdom": "20.0.1",
     "junit-report-builder": "3.0.1",
     "lint-staged": "13.0.3",
-    "mocha": "10.0.0",
+    "mocha": "10.1.0",
     "mocha-multi-reporters": "1.5.1",
     "remark-gfm": "3.0.1",
     "semantic-release": "19.0.5"

package/src/json-pipe.js

@@ -9,6 +9,7 @@
  * OF ANY KIND, either express or implied. See the License for the specific language
  * governing permissions and limitations under the License.
  */
+import { cleanupHeaderValue } from '@adobe/helix-shared-utils';
 import fetchMetadata from './steps/fetch-metadata.js';
 import setCustomResponseHeaders from './steps/set-custom-response-headers.js';
 import { PipelineResponse } from './PipelineResponse.js';
@@ -16,6 +17,7 @@ import jsonFilter from './utils/json-filter.js';
 import { extractLastModified, updateLastModified } from './utils/last-modified.js';
 import fetchConfig from './steps/fetch-config.js';
 import { getPathInfo } from './utils/path.js';
+import { PipelineStatusError } from './PipelineStatusError.js';
 
 /**
  * Checks the fstab for folder mapping entries and then re-adjusts the path infos if needed.
@@ -73,47 +75,57 @@ export async function jsonPipe(state, req) {
     });
   }
 
-  // fetch config and apply the folder mapping
-  await fetchConfig(state, req);
-  await folderMapping(state);
+  try {
+    // fetch config and apply the folder mapping
+    await fetchConfig(state, req);
+    await folderMapping(state);
 
-  // fetch data from content bus
-  const { path } = state.info;
-  state.timer?.update('json-fetch');
-  let dataResponse = await s3Loader.getObject('helix-content-bus', `${contentBusId}/${partition}${path}`);
+    // fetch data from content bus
+    const { path } = state.info;
+    state.timer?.update('json-fetch');
+    let dataResponse = await s3Loader.getObject('helix-content-bus', `${contentBusId}/${partition}${path}`);
 
-  // if not found, fall back to code bus
-  if (dataResponse.status === 404) {
-    dataResponse = await s3Loader.getObject('helix-code-bus', `${owner}/${repo}/${ref}${path}`);
-  }
+    // if not found, fall back to code bus
+    if (dataResponse.status === 404) {
+      dataResponse = await s3Loader.getObject('helix-code-bus', `${owner}/${repo}/${ref}${path}`);
+    }
 
-  // if still not found, return status
-  if (dataResponse.status !== 200) {
-    if (dataResponse.status < 500) {
-      await fetchMetadata(state, req, dataResponse);
-      await setCustomResponseHeaders(state, req, dataResponse);
+    // 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;
     }
-    return dataResponse;
-  }
-  const data = dataResponse.body;
+    const data = dataResponse.body;
 
-  // filter data
-  const response = jsonFilter(state, data, {
-    limit: limit ? Number.parseInt(limit, 10) : undefined,
-    offset: offset ? Number.parseInt(offset, 10) : undefined,
-    sheet,
-    raw: limit === undefined && offset === undefined && sheet === undefined,
-  });
+    // filter data
+    const response = jsonFilter(state, data, {
+      limit: limit ? Number.parseInt(limit, 10) : undefined,
+      offset: offset ? Number.parseInt(offset, 10) : undefined,
+      sheet,
+      raw: limit === undefined && offset === undefined && sheet === undefined,
+    });
 
-  // set last-modified
-  updateLastModified(state, response, extractLastModified(dataResponse.headers));
+    // set last-modified
+    updateLastModified(state, response, extractLastModified(dataResponse.headers));
 
-  // set surrogate key
-  response.headers.set('x-surrogate-key', `${contentBusId}${path}`.replace(/\//g, '_'));
+    // set surrogate key
+    response.headers.set('x-surrogate-key', `${contentBusId}${path}`.replace(/\//g, '_'));
 
-  // Load metadata from metadata.json
-  await fetchMetadata(state, req, response);
-  await setCustomResponseHeaders(state, req, response);
+    // Load metadata from metadata.json
+    await fetchMetadata(state, req, response);
+    await setCustomResponseHeaders(state, req, response);
 
-  return response;
+    return response;
+  } catch (e) {
+    const res = new PipelineResponse('', {
+      status: e instanceof PipelineStatusError ? e.code : 500,
+    });
+    const level = res.status >= 500 ? 'error' : 'info';
+    log[level](`pipeline status: ${res.status} ${e.message}`, e);
+    res.headers.set('x-error', cleanupHeaderValue(e.message));
+    return res;
+  }
 }

package/src/utils/id-slugger.js

@@ -9,7 +9,7 @@
  * OF ANY KIND, either express or implied. See the License for the specific language
  * governing permissions and limitations under the License.
  */
-import GithubSlugger from 'github-slugger';
+import { slug } from 'github-slugger';
 
 export class IDSlugger {
   constructor() {
@@ -22,7 +22,7 @@ export class IDSlugger {
    * @return {string}       A unique slug string
    */
   slug(value) {
-    let id = GithubSlugger.slug(value)
+    let id = slug(value)
       // remove leading numbers
       .replace(/^\d+-+/, '');
 

package/test/StaticS3Loader.js

@@ -34,6 +34,11 @@ export class StaticS3Loader {
       body: '',
       headers: new Map(),
     };
+    if (response instanceof Error) {
+      // eslint-disable-next-line no-console
+      console.log(`StaticS3Loader: failing ${bucketId}/${key} -> ${response.message}`);
+      throw response;
+    }
     // eslint-disable-next-line no-console
     console.log(`StaticS3Loader: loading ${bucketId}/${key} -> ${response.status}`);
     return response;