AMP-31064: Fix link generation.

Author: ginchauspe

wp-react-lib/src/util/index.js

@@ -1,23 +1,42 @@
 const useHash = process.env.VITE_REACT_APP_USE_HASH_LINKS;
+const WP_ROOT = process.env.VITE_REACT_APP_WP || '/wp';
 
+// Escapes special regex metacharacters in a literal string.
+const escapeRegex = (s) => s.replace(/[.*+?^${}()|[\]\\]/g, '\\$&');
+
+// Builds a regex that matches the scheme + configured WP hostname(s) + optional WP
+// subdirectory path (e.g. /wp), so the entire origin prefix can be replaced with the
+// locale slug in one step.
+// Returns null when no valid hosts are configured — callers must treat null as "skip".
+const buildUrlRegex = () => {
+    const hosts = process.env.VITE_REACT_APP_WP_HOSTS
+        ?.split(",").map(h => h.trim()).filter(Boolean) || [];
+    if (!hosts.length) return null;
+    const hostsPattern = hosts.map(escapeRegex).join('|');
+    const wpRootPattern = escapeRegex(WP_ROOT);
+    return new RegExp(`^(http|https)://(${hostsPattern})(${wpRootPattern})?`, 'ig');
+};
 
 export const replaceLink = (url, locale) => {
     //console.log("--------- replaceLink--------------")
     //console.log(process.env.REACT_APP_WP_HOSTS)
-    const replacementTarget = process.env.VITE_REACT_APP_WP_HOSTS.split(",")
-    let all = new RegExp("^(http|https)://(" + replacementTarget.join('|') + ")", "ig");
-    if (useHash && url) {
-        return url.replaceAll(all, "#" + locale)
-    } else if (url) {
-        return url.replaceAll(all, "/" + locale)
+    if (!url) return url;
+    const all = buildUrlRegex();
+    // Guard: if WP_HOSTS is not configured the regex would be wrong — return unchanged.
+    if (!all) return url;
+    if (useHash) {
+        return url.replaceAll(all, '#' + locale);
     }
+    return url.replaceAll(all, '/' + locale);
 }
 
 export const replaceHTMLinks = (html, locale) => {
     //console.log("--------- replaceHTMLinks--------------")
     // console.log(process.env.REACT_APP_WP_HOSTS)
-    const replacementTarget = process.env.VITE_REACT_APP_WP_HOSTS?.split(",") || []
-    let all = new RegExp("^(http|https)://(" + replacementTarget.join('|') + ")", "ig");
+    if (!html) return html;
+    const all = buildUrlRegex();
+    // Guard: if WP_HOSTS is not configured the regex would be wrong — return unchanged.
+    if (!all) return html;
 
     let link;
     let regex = /href\s*=\s*(['"])(https?:\/\/.+?)\1/ig;
@@ -58,4 +77,4 @@ export const removePatternBrackets = (html) => {
     }
 };
 
-export default {replaceHTMLinks, replaceLink}
+export default {replaceHTMLinks, replaceLink}