Skip to content

fix: rewrite aem.page/aem.live image srcs in editor to prevent 401 errors #930

Open
andreituicu wants to merge 8 commits into
mainfrom
fximg
Open

fix: rewrite aem.page/aem.live image srcs in editor to prevent 401 errors #930
andreituicu wants to merge 8 commits into
mainfrom
fximg

Conversation

@andreituicu
Copy link
Copy Markdown
Contributor

@andreituicu andreituicu commented May 13, 2026

Description

Images stored with *.aem.page or *.aem.live URLs now display correctly in the editor without triggering 401 errors.

When DA documents contain images served from AEM preview/publish hosts, the editor lacks the auth those hosts require. This PR rewrites those image URLs to their *.preview.da.live equivalent at render time via ProseMirror nodeViews — display-only, never written back to the document or synced via collab.

How Has This Been Tested?

Manually with: https://fximg--da-live--adobe.aem.live/edit#/andreituicu/da-test/mediabus

Screenshots (if appropriate):

The image with the cat is behind helix auth: https://main--da-test--andreituicu.aem.page/media_1222da127066e69ab4693b1aa98ad4cd6fdfe1fe4.jpg?width=2000&format=webply&optimize=medium

mediabus-images

Types of changes

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to change)

Checklist:

  • I have signed the Adobe Open Source CLA.
  • My code follows the code style of this project.
  • My change requires a change to the documentation.
  • I have updated the documentation accordingly.
  • I have read the CONTRIBUTING document.
  • I have added tests to cover my changes.
  • All new and existing tests passed.

@aem-code-sync
Copy link
Copy Markdown

aem-code-sync Bot commented May 13, 2026

Hello, I'm the AEM Code Sync Bot and I will run some actions to deploy your branch.
In case there are problems, just click the checkbox below to rerun the respective action.

  • Re-sync branch
Commits

@andreituicu andreituicu changed the title Fximg fix: rewrite aem.page/aem.live image srcs in editor to prevent 401 errors May 13, 2026
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