@adobe/helix-html-pipeline 3.11.9 → 3.11.11

package/CHANGELOG.md

@@ -1,3 +1,18 @@
+## [3.11.11](https://github.com/adobe/helix-html-pipeline/compare/v3.11.10...v3.11.11) (2023-06-03)
+
+
+### Bug Fixes
+
+* ignore invalid urls in link rewriter ([#329](https://github.com/adobe/helix-html-pipeline/issues/329)) ([03a8e35](https://github.com/adobe/helix-html-pipeline/commit/03a8e354af1f88d0d7dc1b9ad4537c7fbf8382f3)), closes [#328](https://github.com/adobe/helix-html-pipeline/issues/328)
+
+## [3.11.10](https://github.com/adobe/helix-html-pipeline/compare/v3.11.9...v3.11.10) (2023-06-01)
+
+
+### Bug Fixes
+
+* coverage ([#326](https://github.com/adobe/helix-html-pipeline/issues/326)) ([31a34b9](https://github.com/adobe/helix-html-pipeline/commit/31a34b96a5ed01c002e9951507712904f8ff5615))
+* output image title attribute ([3893776](https://github.com/adobe/helix-html-pipeline/commit/3893776a5b6ce786e69e9475dc148955a634d719)), closes [#308](https://github.com/adobe/helix-html-pipeline/issues/308)
+
 ## [3.11.9](https://github.com/adobe/helix-html-pipeline/compare/v3.11.8...v3.11.9) (2023-05-25)
 
 

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@adobe/helix-html-pipeline",
-  "version": "3.11.9",
+  "version": "3.11.11",
   "description": "Helix HTML Pipeline",
   "main": "src/index.js",
   "types": "src/index.d.ts",
@@ -57,12 +57,12 @@
     "mdast-util-gfm-task-list-item": "1.0.2",
     "mdast-util-to-hast": "12.3.0",
     "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.6",
+    "micromark-extension-gfm-footnote": "1.1.2",
+    "micromark-extension-gfm-strikethrough": "1.0.7",
+    "micromark-extension-gfm-table": "1.0.7",
     "micromark-extension-gfm-tagfilter": "1.0.2",
-    "micromark-extension-gfm-task-list-item": "1.0.4",
-    "micromark-util-combine-extensions": "1.0.0",
+    "micromark-extension-gfm-task-list-item": "1.0.5",
+    "micromark-util-combine-extensions": "1.1.0",
     "mime": "3.0.0",
     "rehype-format": "4.0.1",
     "rehype-parse": "8.0.4",
@@ -82,21 +82,21 @@
     "@semantic-release/changelog": "6.0.3",
     "@semantic-release/git": "10.0.1",
     "@semantic-release/npm": "10.0.3",
-    "c8": "7.13.0",
-    "eslint": "8.41.0",
+    "c8": "7.14.0",
+    "eslint": "8.42.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.3",
+    "esmock": "2.3.1",
     "husky": "8.0.3",
     "js-yaml": "4.1.0",
-    "jsdom": "22.0.0",
+    "jsdom": "22.1.0",
     "junit-report-builder": "3.0.1",
     "lint-staged": "13.2.2",
     "mocha": "10.2.0",
     "mocha-multi-reporters": "1.5.1",
     "remark-gfm": "3.0.1",
-    "semantic-release": "21.0.2"
+    "semantic-release": "21.0.3"
   },
   "lint-staged": {
     "*.js": "eslint",

package/src/steps/create-pictures.js

@@ -18,7 +18,7 @@ const BREAK_POINTS = [
   { width: '750' },
 ];
 
-export function createOptimizedPicture(src, alt = '') {
+export function createOptimizedPicture(src, alt = '', title = undefined) {
   const url = new URL(src, 'https://localhost/');
   const { pathname, hash = '' } = url;
   const props = new URLSearchParams(hash.substring(1));
@@ -55,6 +55,7 @@ export function createOptimizedPicture(src, alt = '') {
     return h('img', {
       loading: 'lazy',
       alt,
+      'data-title': title === alt ? undefined : title,
       type: v.type,
       src: srcset,
       width,
@@ -78,8 +79,8 @@ export default async function createPictures({ content }) {
   const { hast } = content;
 
   visitParents(hast, isMediaImage, (img, parents) => {
-    const { src, alt } = img.properties;
-    const picture = createOptimizedPicture(src, alt);
+    const { src, alt, title } = img.properties;
+    const picture = createOptimizedPicture(src, alt, title);
 
     // check if parent has style and unwrap if needed
     const parent = parents[parents.length - 1];

package/src/steps/utils.js

@@ -185,25 +185,29 @@ export function rewriteUrl(state, url) {
   if (!url || !url.startsWith('https://')) {
     return url;
   }
-  const { pathname, search, hash } = new URL(url);
-
-  if (AZURE_BLOB_REGEXP.test(url)) {
-    const filename = pathname.split('/').pop();
-    const [name, props] = hash.split('?');
-    const extension = name.split('.').pop() || 'jpg';
-    const newHash = props ? `#${props}` : '';
-    return `./media_${filename}.${extension}${newHash}`;
-  }
+  try {
+    const { pathname, search, hash } = new URL(url);
+
+    if (AZURE_BLOB_REGEXP.test(url)) {
+      const filename = pathname.split('/').pop();
+      const [name, props] = hash.split('?');
+      const extension = name.split('.').pop() || 'jpg';
+      const newHash = props ? `#${props}` : '';
+      return `./media_${filename}.${extension}${newHash}`;
+    }
 
-  if (MEDIA_BLOB_REGEXP.test(url)) {
-    return `.${pathname}${hash}`;
-  }
+    if (MEDIA_BLOB_REGEXP.test(url)) {
+      return `.${pathname}${hash}`;
+    }
 
-  if (HELIX_URL_REGEXP.test(url)) {
-    if (hash && pathname === state.info?.path) {
-      return hash;
+    if (HELIX_URL_REGEXP.test(url)) {
+      if (hash && pathname === state.info?.path) {
+        return hash;
+      }
+      return `${pathname}${search}${hash}`;
     }
-    return `${pathname}${search}${hash}`;
+  } catch {
+    // ignore
   }
 
   return url;