Closed jarhodes314 closed 1 week ago
Attention: Patch coverage is 93.33333%
with 3 lines
in your changes missing coverage. Please review.
Project coverage is 78.3%. Comparing base (
e54f81a
) to head (d1359b0
). Report is 1 commits behind head on main.
:white_check_mark: Passed | :x: Failed | :next_track_button: Skipped | Total | Pass % | :stopwatch: Duration |
---|---|---|---|---|---|
453 | 0 | 3 | 453 | 100 | 58m36.590424999s |
Proposed changes
Improve the host header tests introduced in #2946 by actually parsing the headers instead of simply checking for the presence of a string in the incoming request. Additionally simplify the logic for obtaining the websocket host.
Types of changes
Paste Link to the issue
Checklist
cargo fmt
as mentioned in CODING_GUIDELINEScargo clippy
as mentioned in CODING_GUIDELINESFurther comments