Skip to content

exclude H3 from connection type arguments list - #73

Open
Devel08 wants to merge 1 commit into
cloudflare:mainfrom
Devel08:exclude-h3
Open

Devel08 wants to merge 1 commit into
cloudflare:mainfrom
Devel08:exclude-h3

Conversation

@Devel08

@Devel08 Devel08 commented Sep 27, 2026

Copy link
Copy Markdown
Contributor

H3 isn't an implemented connection type, thus indeed panics, this PR removes it from the arguments list, so that the user cannot cause a panic, possible values will be h1-clear-text, h1 and h2.

$ cargo run -q saturate -t
error: a value is required for '--conn-type <CONN_TYPE>' but none was supplied
  [possible values: h1-clear-text, h1, h2]

For more information, try '--help

ran cargo fmt and cargo clippy. Closes #51

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.

mach saturate -t h3 panics: ConnectionType::H3 => todo!()

1 participant