Skip to content

fix: keep permissions on adapted json responses - #660

Merged
benpeter merged 2 commits into
mainfrom
playwfix
Aug 13, 2026
Merged

fix: keep permissions on adapted json responses#660
benpeter merged 2 commits into
mainfrom
playwfix

Conversation

@benpeter

Copy link
Copy Markdown
Contributor

adaptJsonResponse rebuilds the response with new Response(body, resp), which copies status, statusText and headers only, so the permissions daFetch set on the original were dropped. on hlx6 that left source.save returning a response without them. da-live's initProse then threw on permissions.some(...), so creating a document did not open the editor.

the same helper runs in config.get, so this also restores write access to the site-config sheet on hlx6.

testing: da-live's Playwright Helix suite, pinned to this branch, is 109 passed and 0 failed. it had been 27 failed since 2026-08-07. hand-tested on da.live against a local nx2, where creating a document on a hlx6 site opens the editor with contenteditable="true".

@aem-code-sync

aem-code-sync Bot commented Aug 12, 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

@benpeter
benpeter merged commit 4a3e266 into main Aug 13, 2026
3 checks passed
@benpeter
benpeter deleted the playwfix branch August 13, 2026 15:45
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