Skip to content

Commit 6acf9ef

Browse files
authored
fix(client): only take single url on (args.resume && first) (#1578)
QUIC Interop Runner testcases `resumption` and `0-rtt` expect the client to download the first file, close the connection, and then download the remaining files on a second connection. https://github.com/quic-interop/quic-interop-runner/tree/master#test-cases Thus `neqo-client`, when `args.resume` is `true` must only take a single URL on the **first** loop iteration. On the second iteration it must take all remaining URLs. Regression introduced in #1569.
1 parent e447eef commit 6acf9ef

File tree

1 file changed

+4
-1
lines changed

1 file changed

+4
-1
lines changed

neqo-client/src/main.rs

Lines changed: 4 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1086,13 +1086,16 @@ fn main() -> Res<()> {
10861086

10871087
let hostname = format!("{host}");
10881088
let mut token: Option<ResumptionToken> = None;
1089+
let mut first = true;
10891090
while !urls.is_empty() {
1090-
let to_request = if args.resume || args.download_in_series {
1091+
let to_request = if (args.resume && first) || args.download_in_series {
10911092
urls.pop_front().into_iter().collect()
10921093
} else {
10931094
std::mem::take(&mut urls)
10941095
};
10951096

1097+
first = false;
1098+
10961099
token = if args.use_old_http {
10971100
old::old_client(
10981101
&args,

0 commit comments

Comments
 (0)