Skip to content

[5.10] make it possible to specify permitted network port access at runtime #7178

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

Merged
merged 1 commit into from
Dec 11, 2023

Conversation

MaxDesiatov
Copy link
Contributor

Cherry-pick of #7151

motivation: The 'network' plugin permission in SwiftPM allows a plugin to request external network access to all or to a set of ports, but the command line flag does not allow to specify ports forcing users to allow all

changes:

  • support passing ports to local and all --allow-network-connections flag
  • write parsing logic to extract ports from local:port,port,port and all:port,port,port
  • update remedy description to accurately specify the usage
  • update tests

rdar://116241125

…7151)

motivation: The 'network' plugin permission in SwiftPM allows a plugin
to request external network access to all or to a set of ports, but the
command line flag does not allow to specify ports forcing users to allow
all

changes:
* support passing ports to `local` and `all` --allow-network-connections flag
* write parsing logic to extract ports from `local:port,port,port` and
`all:port,port,port`
* update remedy description to accurately specify the usage
* update tests

rdar://116241125
@MaxDesiatov MaxDesiatov added enhancement swift 5.10 This PR targets the 5.10 branch labels Dec 11, 2023
@MaxDesiatov
Copy link
Contributor Author

cc @euanh

@MaxDesiatov
Copy link
Contributor Author

@swift-ci test

@MaxDesiatov MaxDesiatov enabled auto-merge (squash) December 11, 2023 17:19
@MaxDesiatov MaxDesiatov merged commit b31f19e into release/5.10 Dec 11, 2023
@MaxDesiatov MaxDesiatov deleted the maxd/network-access-5.10 branch December 11, 2023 18:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement swift 5.10 This PR targets the 5.10 branch
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants