zhaofengli / attic

Multi-tenant Nix Binary Cache
https://docs.attic.rs
Other
964 stars 73 forks source link

fixup: stream error logging #154

Closed cole-h closed 2 months ago

cole-h commented 2 months ago

The call to into_inner() discards the wrapper type constructed by map_err(). So instead, map_err() the actual stream, and call Body::from_stream on the wrapped stream.


This was originally fixed in https://github.com/zhaofengli/attic/pull/137, but the update to axum 0.7 in https://github.com/zhaofengli/attic/pull/138 migrated that functionality slightly incorrectly.

zhaofengli commented 2 months ago

Oops, that was a mistake. Thanks for the fix!