Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add private network access #2466

Merged
merged 5 commits into from
Dec 19, 2024
Merged

Add private network access #2466

merged 5 commits into from
Dec 19, 2024

Conversation

autonome
Copy link
Collaborator

@autonome autonome commented Dec 18, 2024

Seems clearly referred to as a standalone feature in developer conversations, but also there's not much there there in BCD yet around it.

NOTE: see mdn/browser-compat-data#25467

@github-actions github-actions bot added the feature definition Creating or defining new features or groups of features. label Dec 18, 2024
Copy link
Collaborator

@jamesnw jamesnw left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Some minor wording suggestions, but LGTM!

features/private-network-access.yml Outdated Show resolved Hide resolved
@@ -0,0 +1,7 @@
name: Private Network Access
description: The Private Network Access feature is a set of modifications to Fetch and HTML which are intended to mitigate the risks associated with unintentional exposure of devices and servers on a client’s internal network to the web at large.
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
description: The Private Network Access feature is a set of modifications to Fetch and HTML which are intended to mitigate the risks associated with unintentional exposure of devices and servers on a client’s internal network to the web at large.
description: Private Network Access modifies Fetch and HTML to mitigate the risks associated with unintentional exposure of devices and servers on a client’s internal network to the web at large.

Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The description talks about modifying other features. It's not clear how this is a feature to developers. I had a skim through the spec, so I don't really understand this thing, but I suspect we want to say something more in this direction:

Suggested change
description: The Private Network Access feature is a set of modifications to Fetch and HTML which are intended to mitigate the risks associated with unintentional exposure of devices and servers on a client’s internal network to the web at large.
description: The `Access-Control-Request-Private-Network: true` and `Access-Control-Allow-Private-Network: true` HTTP request and response headers establish permission for websites to make requests to internal networks, such as to routers.

Curiously, BCD doesn't appear to represent the request header. If we went this route, we probably ought to file a bug on BCD reporting it as missing.

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It's not clear how this is a feature to developers.

Yeah, it's complicated. Think about it this way:

  • developers could make requests to the local network since forever
  • now that's being removed
  • PNA is the new way to safely do it

The "What's new" section here lays out the "feature" part:

https://developer.chrome.com/blog/private-network-access-preflight

Your edit makes this much more clearly a feature than the "mitigation" wording, thanks!

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Committed this change, which maybe is enough until BCD is sorted.

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Or we could leave this open and blocked until then.

Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

As long as the Request.targetAddressSpace property is representative of browser support for this (which it appears to be, at least for now) we don't have to block on the missing keys. Let's go with this.

@ddbeck ddbeck merged commit c5d942d into web-platform-dx:main Dec 19, 2024
3 checks passed
@autonome autonome deleted the pna branch December 19, 2024 13:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature definition Creating or defining new features or groups of features.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants