Skip to content

Add Local Network Access to iframe allow#4182

Open
gohai wants to merge 1 commit into
processing:developfrom
gohai:allow-local-network-access
Open

Add Local Network Access to iframe allow#4182
gohai wants to merge 1 commit into
processing:developfrom
gohai:allow-local-network-access

Conversation

@gohai

@gohai gohai commented Jun 16, 2026

Copy link
Copy Markdown

Issue:

See https://learn.microsoft.com/en-us/deployedge/ms-edge-local-network-access for details. This still yields to the browser on whether or not to allow the actual request, but it adds the possibility to interface with a local host via CORS requests from within the iframe.

I have verified that this pull request:

  • has no linting errors (npm run lint)
  • has no test errors (npm run test)
  • has no typecheck errors (npm run typecheck)
  • is from a uniquely-named feature branch and is up to date with the develop branch.
  • is descriptively named and links to an issue number, i.e. Fixes #123
  • meets the standards outlined in the accessibility guidelines

See https://learn.microsoft.com/en-us/deployedge/ms-edge-local-network-access for details. This still yields to the browser on whether or not to allow the actual request, but it adds the possibility to interface with a local host via CORS requests from within the iframe.
@welcome

welcome Bot commented Jun 16, 2026

Copy link
Copy Markdown

🎉 Thanks for opening this pull request! Please check out our contributing guidelines if you haven't already.

@gohai gohai changed the title Add Local Network Access restrictions to iframe allow Add Local Network Access to iframe allow Jun 16, 2026
@gohai

gohai commented Jun 17, 2026

Copy link
Copy Markdown
Author

To add some motivation for the change: I maintain a p5 library to interface with ComfyUI, which is typically running on local, GPU-equipped machines that practitioners remotely access from their laptops. I have been using the p5 web editor in my workshops, but recent Chrome changes surrounding Local Network Access restrictions broke this workflow when using the web editor.

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