Skip to content

fix(api): strip org/site prefix from destination before building hlx6 copy URL - #644

Merged
tripodsan merged 1 commit into
mainfrom
fix-copy
Aug 6, 2026
Merged

fix(api): strip org/site prefix from destination before building hlx6 copy URL#644
tripodsan merged 1 commit into
mainfrom
fix-copy

Conversation

@tripodsan

Copy link
Copy Markdown
Contributor

Summary

  • source.copy on hlx6 sites was building the source-bus URL directly from the caller-supplied destination, which includes the /org/site/ prefix required by the DA source API but not valid for the hlx6 source bus path.
  • Strip the /org/site/ prefix from destination before passing it to getDaApiPath when copying on hlx6 sites, so the resulting URL targets the correct path.

Test plan

  • Updated test/nx2/utils/api.test.js source.copy tests to pass a destination with the /org/site/ prefix (matching real caller behavior) and assert against the stripped value.
  • npm test passes (1149 tests, 0 failed).
  • npx eslint passes on changed files.

… copy URL

destination is passed with a /org/site/ prefix for the DA source API,
but the hlx6 source bus path must not include it, causing source.copy
to hit the wrong path on hlx6 sites.
@aem-code-sync

aem-code-sync Bot commented Aug 6, 2026

Copy link
Copy Markdown

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

@tripodsan
tripodsan requested a review from rofe August 6, 2026 12:15
@tripodsan
tripodsan merged commit 02b1d09 into main Aug 6, 2026
3 checks passed
@tripodsan
tripodsan deleted the fix-copy branch August 6, 2026 12:22
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.

2 participants