search

nanomsg / nng

nanomsg-next-generation -- light-weight brokerless messaging
https://nng.nanomsg.org
MIT License
3.78k stars 485 forks source link

Log messages when peer sends too large message. #1812

Closed gdamore closed 5 months ago

gdamore commented 5 months ago

Also while here, remove unused sockaddr members from some structs. This should save a bit of memory for servers with a lot of conns.

fixes #

Note that the above format should be used in your git commit comments. You agree that by submitting a PR, you have read and agreed to our contributing guidelines.
codecov[bot] commented 5 months ago

Codecov Report

Attention: Patch coverage is 86.66667% with 2 lines in your changes are missing coverage. Please review.

Project coverage is 79.16%. Comparing base (d230d9b) to head (b7097e1).

Files Patch % Lines
src/sp/transport/tls/tls.c 0.00% 2 Missing :warning:
Additional details and impacted files ```diff @@ Coverage Diff @@ ## master #1812 +/- ## ========================================== + Coverage 79.05% 79.16% +0.10% ========================================== Files 95 95 Lines 21329 21335 +6 ========================================== + Hits 16861 16889 +28 + Misses 4468 4446 -22 ```

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.