@adobe/helix-html-pipeline 5.7.0 → 5.7.2

package/CHANGELOG.md

@@ -1,3 +1,17 @@
+## [5.7.2](https://github.com/adobe/helix-html-pipeline/compare/v5.7.1...v5.7.2) (2024-02-08)
+
+
+### Bug Fixes
+
+* handle sitemap auth in hlx.page ([#533](https://github.com/adobe/helix-html-pipeline/issues/533)) ([bc95dae](https://github.com/adobe/helix-html-pipeline/commit/bc95daecbdc0a806b61cd00360e68100beb28ece))
+
+## [5.7.1](https://github.com/adobe/helix-html-pipeline/compare/v5.7.0...v5.7.1) (2024-02-08)
+
+
+### Bug Fixes
+
+* put <loc> on a single line ([#532](https://github.com/adobe/helix-html-pipeline/issues/532)) ([c8626eb](https://github.com/adobe/helix-html-pipeline/commit/c8626eb6a45819cdb8f6348752fb2883d9a2a7d9))
+
 # [5.7.0](https://github.com/adobe/helix-html-pipeline/compare/v5.6.3...v5.7.0) (2024-02-05)
 
 

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@adobe/helix-html-pipeline",
-  "version": "5.7.0",
+  "version": "5.7.2",
   "description": "Helix HTML Pipeline",
   "main": "src/index.js",
   "types": "src/index.d.ts",

package/src/sitemap-pipe.js

@@ -11,7 +11,7 @@
  */
 import escape from 'lodash.escape';
 import { cleanupHeaderValue } from '@adobe/helix-shared-utils';
-import { authenticate, requireProject } from './steps/authenticate.js';
+import { requireProject } from './steps/authenticate.js';
 import fetchConfig from './steps/fetch-config.js';
 import fetchContent from './steps/fetch-content.js';
 import fetchConfigAll from './steps/fetch-config-all.js';
@@ -21,9 +21,9 @@ import setCustomResponseHeaders from './steps/set-custom-response-headers.js';
 import { PipelineStatusError } from './PipelineStatusError.js';
 import { PipelineResponse } from './PipelineResponse.js';
 
-async function generateSitemap(state, partition) {
+async function generateSitemap(state) {
   const {
-    owner, repo, ref, contentBusId, s3Loader, log,
+    owner, repo, ref, contentBusId, s3Loader, log, partition,
     previewHost, liveHost, config: { host: prodCDN } = {},
   } = state;
   const ret = await s3Loader.getObject('helix-content-bus', `${contentBusId}/live/sitemap.json`);
@@ -45,9 +45,7 @@ async function generateSitemap(state, partition) {
     ? (previewHost || `${ref}--${repo}--${owner}.hlx.page`)
     : (prodCDN || liveHost || `${ref}--${repo}--${owner}.hlx.live`);
   const loc = ({ path, lastModified }) => `  <url>
-    <loc>
-      https://${host}${escape(path)}
-    </loc>
+    <loc>https://${host}${escape(path)}</loc>
     <lastmod>${new Date(lastModified * 1000).toISOString().substring(0, 10)}</lastmod>
   </url>`;
   const xml = [
@@ -76,12 +74,9 @@ async function generateSitemap(state, partition) {
  * @returns {PipelineResponse}
  */
 export async function sitemapPipe(state, req) {
-  const { partition, log } = state;
+  const { log } = state;
   state.type = 'sitemap';
 
-  // force loading from preview
-  state.partition = 'preview';
-
   if (state.info?.path !== '/sitemap.xml') {
     // this should not happen as it would mean that the caller used the wrong route. so we respond
     // with a 500 to indicate that something is wrong.
@@ -100,8 +95,6 @@ export async function sitemapPipe(state, req) {
     },
   });
 
-  // check if .auth request
-
   try { // fetch config first, since we need to compute the content-bus-id from the fstab ...
     state.timer?.update('config-fetch');
     await fetchConfig(state, req, res);
@@ -120,11 +113,9 @@ export async function sitemapPipe(state, req) {
     ]);
 
     await requireProject(state, req, res);
-    if (res.error !== 401) {
-      await authenticate(state, req, res);
-    }
+    // authentication is handled in hlx.page vcl
     if (res.status === 404) {
-      const ret = await generateSitemap(state, partition);
+      const ret = await generateSitemap(state);
       if (ret.status === 200) {
         res.status = 200;
         delete res.error;