Skip to content

Preserve DM url in og:image without FF - #54

Draft
RitwikSrivastava wants to merge 2 commits into
adobe:mainfrom
RitwikSrivastava:dmimage
Draft

Preserve DM url in og:image without FF#54
RitwikSrivastava wants to merge 2 commits into
adobe:mainfrom
RitwikSrivastava:dmimage

Conversation

@RitwikSrivastava

Copy link
Copy Markdown

Preserve Adobe Assets/Dynamic Media delivery URLs (*.adobeaemcloud.com/adobe/assets/..., *.scene7.com/is/image/...) as-is when used as page-metadata images (og:image, twitter:image, etc.), instead of rehosting them into the site's local media bucket by default.

Scoped narrowly to metadata images only, via a new isMetadataImage flag on nodes from addMetadata() — regular body images on the same kind of URL are unaffected and keep today's exact rehosting behavior, no config required.

Added isAdobeAssetDeliveryUrl() (domain-anchored, not path-shape-based, to avoid false positives) plus unit and integration tests covering both the exemption and the negative case (body image still uploads).

Please ensure your pull request adheres to the following guidelines:

  • make sure to link the related issues in this description
  • when merging / squashing, make sure the fixed issue references are visible in the commits, for easy compilation of release notes

Related Issues

Thanks for contributing!

@RitwikSrivastava
RitwikSrivastava marked this pull request as draft July 23, 2026 15:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant