Proxy protocol header

[megsdevs]

VAL-1250

[codecov]

Codecov Report

Merging #3 (e7dbdec) into master (13ae39f) will increase coverage by 8.59%.
The diff coverage is 83.58%.

@@            Coverage Diff             @@
##           master       #3      +/-   ##
==========================================
+ Coverage   68.13%   76.73%   +8.59%     
==========================================
  Files          11       13       +2     
  Lines        1133     1732     +599     
==========================================
+ Hits          772     1329     +557     
- Misses        361      403      +42     

Files	Coverage Δ
src/tls_openssl/mod.rs	82.96% <100.00%> (+6.59%)	⬆️
src/tls_rustls/mod.rs	77.03% <100.00%> (+6.64%)	⬆️
src/proxy_protocol/future.rs	87.75% <87.75%> (ø)
src/server.rs	77.31% <80.64%> (+5.39%)	⬆️
src/proxy_protocol/mod.rs	83.23% <83.23%> (ø)

... and 2 files with indirect coverage changes

[0xAlcibiades]

Just the one change and will check back on the v1 support.

[0xAlcibiades]

programatik29/axum-server#89 - original PR/feedback for ref.

[0xAlcibiades]

Yes, I was more looking for a comment here on the why of why they are exported in code. Should have made that more clear.

…

On Oct 12, 2023, at 23:05, Dave Belvedere ***@***.***> wrote: @dbelv commented on this pull request. --------------------------------------------------------------- In [src/tls_rustls/mod.rs](#3 (comment)): > @@ -303,7 +303,7 @@ async fn config_from_pem_file( } #[cfg(test)] -mod tests { +pub(crate) mod tests { It allows other test modules to import the public functions — Reply to this email directly, [view it on GitHub](#3 (comment)), or [unsubscribe](https://github.com/notifications/unsubscribe-auth/AVOT3C3XSBQ3LOTTW5ZOUQLX7CVYNANCNFSM6AAAAAA55Y5FDA). You are receiving this because your review was requested.Message ID: ***@***.***>