MWPW-134103 Fix Blog Regex

[Brandon32]

• Updating the html exclude regular expression to include /blog with no trailing slash
• Previously appendHtmlPostfix was only called durring loadArea, but now that html postfix has been updated appendHtmlToLink is called more frequently with decorateLinks. [MWPW-131409] Update .html postfix milo#927
• The Experience Cloud blog, https://business.adobe.com/blog, does not have a trailing slash and since gnav is loading fragments, fragments/gnav/resources.plain.html, the appendHtmlToLink function is now ran against the blog link.
  • Note, stage works correctly because the domains do not match.

Resolves: MWPW-134103

Test URLs:
N/A

[aem-code-sync]

Hello, I'm Franklin Bot and I will run some test suites that validate the page speed.
In case there are problems, just click the checkbox below to rerun the respective action.

• Re-run PSI Checks

[codecov-commenter]

Codecov Report

Merging #132 (8a556f5) into main (8bdf271) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main     #132   +/-   ##
=======================================
  Coverage   96.36%   96.36%           
=======================================
  Files           6        6           
  Lines         468      468           
=======================================
  Hits          451      451           
  Misses         17       17           

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

[overmyheadandbody]

Is there no test URL that can be provided here?

[overmyheadandbody]

Is it safe to assume that blog will not have pages for the africa, mena_en or mena_ar regions? This regex would not pass for those.

[JasonHowellSlavin]

Is it safe to assume that blog will not have pages for the africa, mena_en or mena_ar regions? This regex would not pass for those.

the dx blog only supports regions au, uk, de, fr, ko, jp, so it should be okay in this instance!

[Brandon32]

Is there no test URL that can be provided here?

We only add .html extension in Stage and Prod so a branch is not going to show the this issue. There's the early return with DOT_HTML_PATH on line 280 so it will not reach the regex test on non-html pages.

• https://business.adobe.com/resources/main.html
• https://business.stage.adobe.com/resources/main.html

Also you can see if you click on "Resources" in the gnav and click "Experience Cloud Blog" that the issue only appears in production, this is because the origins also have to match before it reaches the regex test, line 291.

• https://bmarshal-blog-exclude--bacom--adobecom.hlx.page/resources/main

You can take a look at the branch but the only way to test is in production or with an online regex tester.