@adobe/helix-html-pipeline 5.5.1 → 5.5.2

package/CHANGELOG.md

@@ -1,3 +1,10 @@
+## [5.5.2](https://github.com/adobe/helix-html-pipeline/compare/v5.5.1...v5.5.2) (2024-01-25)
+
+
+### Bug Fixes
+
+* include surrogate keys for 404 sitemap.xml ([#513](https://github.com/adobe/helix-html-pipeline/issues/513)) ([d81798f](https://github.com/adobe/helix-html-pipeline/commit/d81798f096f6fded11f3280c0c53e4806d2bf5ee))
+
 ## [5.5.1](https://github.com/adobe/helix-html-pipeline/compare/v5.5.0...v5.5.1) (2024-01-23)
 
 

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@adobe/helix-html-pipeline",
-  "version": "5.5.1",
+  "version": "5.5.2",
   "description": "Helix HTML Pipeline",
   "main": "src/index.js",
   "types": "src/index.d.ts",
@@ -43,7 +43,7 @@
   },
   "dependencies": {
     "@adobe/helix-markdown-support": "7.1.0",
-    "@adobe/helix-shared-utils": "3.0.0",
+    "@adobe/helix-shared-utils": "3.0.1",
     "@adobe/mdast-util-gridtables": "3.0.2",
     "@adobe/remark-gridtables": "2.0.2",
     "cookie": "0.6.0",

package/src/sitemap-pipe.js

@@ -51,7 +51,7 @@ export async function sitemapPipe(state, req) {
   /** @type PipelineResponse */
   const res = new PipelineResponse('', {
     headers: {
-      'content-type': 'text/html; charset=utf-8',
+      'content-type': 'text/plain; charset=utf-8',
     },
   });
 
@@ -81,13 +81,7 @@ export async function sitemapPipe(state, req) {
 
     if (res.error) {
       // if content loading produced an error, we're done.
-      const level = res.status >= 500 ? 'error' : 'info';
-      log[level](`pipeline status: ${res.status} ${res.error}`);
-      res.headers.set('x-error', cleanupHeaderValue(res.error));
-      if (res.status < 500) {
-        await setCustomResponseHeaders(state, req, res);
-      }
-      return res;
+      throw new PipelineStatusError(res.status, res.error);
     }
 
     state.timer?.update('serialize');
@@ -96,16 +90,15 @@ export async function sitemapPipe(state, req) {
     await setXSurrogateKeyHeader(state, req, res);
   } catch (e) {
     res.error = e.message;
-    if (e instanceof PipelineStatusError) {
-      res.status = e.code;
-    } else {
-      res.status = 500;
-    }
+    res.status = e.code || 500;
 
     const level = res.status >= 500 ? 'error' : 'info';
-    log[level](`pipeline status: ${res.status} ${res.error}`, e);
+    log[level](`pipeline status: ${res.status} ${res.error}`);
     res.headers.set('x-error', cleanupHeaderValue(res.error));
+    if (res.status < 500) {
+      await setCustomResponseHeaders(state, req, res);
+      await setXSurrogateKeyHeader(state, req, res);
+    }
   }
-
   return res;
 }