Skip to content

Commit f452722

Browse files
committed
style(rust): state the constraint rather than what the code used to do
The split is on the last `@` because a password may contain a slash. Why it is not on the authority is not the reader's problem.
1 parent 9a70f9b commit f452722

1 file changed

Lines changed: 1 addition & 1 deletion

File tree

  • packages/rust/armonik-transport/src

packages/rust/armonik-transport/src/proxy.rs

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -561,7 +561,7 @@ mod tests {
561561
"http://***@proxy.corp/path",
562562
),
563563
// A `/` inside the password is malformed, which is how such a value reaches this function
564-
// at all. Cutting the authority at the first `/` used to render the password in full.
564+
// at all, and is why the split is on the last `@` rather than on the authority.
565565
(
566566
"http://user:my/pass@proxy.corp:3128",
567567
"http://***@proxy.corp:3128",

0 commit comments

Comments
 (0)