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

Cherry pick PR #3979: Add disable_h2 and matching h5vcc.settings #3980

Merged
merged 1 commit into from
Aug 13, 2024

Conversation

cobalt-github-releaser-bot
Copy link
Collaborator

Refer to the original PR: #3979

This adds a command-line switch disable_h2 that disables the HTTP/2 protocol (spdy), and a matching h5vcc.settings parameter 'HTTP2' (with backing by PersistentSettings) for run-time disabling of HTTP/2 and spdy. Similar to 'QUIC' and 'HTTP3', the setting takes effect immediately for new connections only.

This also ensures that when the command-line parameter disable_quic or the new disable_h2 is used, that that can not be overuled later with 5vcc.settings or from the corresponding PersistentSetting.

b/205134049

@cobalt-github-releaser-bot
Copy link
Collaborator Author

Important

There were merge conflicts while cherry picking! Check out cherry-pick-24.lts.1+-3979 and fix the conflicts before proceeding. Check the log at https://github.com/youtube/cobalt/actions/runs/10377019661 for details.

@jellefoks
Copy link
Member

Important

There were merge conflicts while cherry picking! Check out cherry-pick-24.lts.1+-3979 and fix the conflicts before proceeding. Check the log at https://github.com/youtube/cobalt/actions/runs/10377019661 for details.

Pushed an update for this branch.

@jellefoks jellefoks closed this Aug 13, 2024
@jellefoks jellefoks reopened this Aug 13, 2024
@jellefoks jellefoks marked this pull request as ready for review August 13, 2024 21:10
Copy link
Member

@kaidokert kaidokert left a comment

Choose a reason for hiding this comment

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

looks okay - even though it would have been sufficient to just backport the commandline setting only, i think. Persistent setting may add extra risk, but lgtm anyway

Copy link
Member

@jellefoks jellefoks left a comment

Choose a reason for hiding this comment

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

Note: I can't merge this, no permission.

@kaidokert kaidokert merged commit f514f84 into 24.lts.1+ Aug 13, 2024
545 of 550 checks passed
@kaidokert kaidokert deleted the cherry-pick-24.lts.1+-3979 branch August 13, 2024 23:08
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.

3 participants