Closed jarhodes314 closed 1 week ago
:white_check_mark: Passed | :x: Failed | :next_track_button: Skipped | Total | Pass % | :stopwatch: Duration |
---|---|---|---|---|---|
453 | 0 | 3 | 453 | 100 | 1h7m1.955003s |
Attention: Patch coverage is 70.58824%
with 15 lines
in your changes missing coverage. Please review.
Project coverage is 78.2%. Comparing base (
23c2714
) to head (bdff513
). Report is 4 commits behind head on main.
Proposed changes
~This should fix a customer issue we have observed seemingly due to duplicate host headers from the c8y-auth-proxy. Currently it just removes the headers, although I am currently trying to reproduce the issue and write a test to verify it is now actually fixed.~
This fixes a bug in
c8y_auth_proxy
where the original headers were sent unmodified, meaning the Host header that was sent with the request originated from the request to the proxy, not the request the proxy makes to Cumulocity. This meant the request had a host of127.0.0.1:8001
rather than the host that we were proxying to (e.g.xyz.cumulocity.com
).Types of changes
Paste Link to the issue
Checklist
cargo fmt
as mentioned in CODING_GUIDELINEScargo clippy
as mentioned in CODING_GUIDELINESFurther comments