Closed potaga closed 5 months ago
Attention: Patch coverage is 57.50000%
with 17 lines
in your changes are missing coverage. Please review.
Project coverage is 78.42%. Comparing base (
52f2075
) to head (430a9a0
).
Files | Patch % | Lines |
---|---|---|
udp_mux.go | 62.50% | 5 Missing and 4 partials :warning: |
udp_muxed_conn.go | 50.00% | 6 Missing and 2 partials :warning: |
:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.
Description
UDPMux is using a map to lookup addresses of each packets. Unfortunately the key is based on a string and each time we want to check the map, a conversion of the UDP address to string is made (.String()) which is expensive. This CR replace the string key by a binary key called ipPort. This structure contains a netip.Addr field and ipPort could be used as a map key