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

ci: set minimal permissions on workflow ngtcp2-quictls.yml #11055

Closed
wants to merge 2 commits into from

Conversation

diogoteles08
Copy link
Contributor

Hi! This PR brings a small security adjustment to limit the permissions of the workflow ngtcp2-quictls.yml, so that it follows the principle of least privilege. Exact same idea of the merged PR #9928

@github-actions github-actions bot added CI Continuous Integration HTTP/3 h3 or quic related labels Apr 28, 2023
Signed-off-by: Diogo Teles Sant'Anna <diogoteles@google.com>
Copy link

@KarmaHaze KarmaHaze left a comment

Choose a reason for hiding this comment

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

@dfandrich dfandrich closed this in 31303c3 Apr 28, 2023
@dfandrich
Copy link
Contributor

Thanks!

bch pushed a commit to bch/curl that referenced this pull request Jul 19, 2023
Signed-off-by: Diogo Teles Sant'Anna <diogoteles@google.com>

Closes curl#11055
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
CI Continuous Integration HTTP/3 h3 or quic related
Development

Successfully merging this pull request may close these issues.

3 participants