fluent-logger-python icon indicating copy to clipboard operation
fluent-logger-python copied to clipboard

This patch supports sending logs over UDP

Open panaetov opened this issue 8 years ago • 6 comments

To send logs over udp you need to specify host parameter as udp://ip[:port]. If you want to use TCP you may specify no prefixes, e.g, localhost or define host as tcp://ip[:port]. Also I have made some changes in tests, namely, I have replaced thread-based realization of fluent's mock with thin wrapper of socket object. It makes possible to use old tests to test both tpc and udp implementations. Also it makes tests more straight and clear, because you can check messages recieved by mock in real-time (now you cannot check messages before connection to mock is closed).

panaetov avatar Mar 14 '17 16:03 panaetov

Coverage Status

Coverage increased (+0.8%) to 90.61% when pulling 49a6182c58d2b3cb94fb0bb83a05669a0c95c5e6 on panaetov:feature/send_logs_with_udp into 5c4dff132d0f955282cf36f3df5701ab27a43ded on fluent:master.

coveralls avatar Mar 14 '17 16:03 coveralls

Looks like this PR addresses #75.

protoss-player avatar Mar 14 '17 16:03 protoss-player

Coverage Status

Coverage increased (+0.3%) to 90.045% when pulling 8c0b4b8fe51a14918e952d2672f4a91610bc37bf on panaetov:feature/send_logs_with_udp into 5c4dff132d0f955282cf36f3df5701ab27a43ded on fluent:master.

coveralls avatar Mar 15 '17 11:03 coveralls

Coverage Status

Coverage increased (+0.3%) to 90.045% when pulling 8c0b4b8fe51a14918e952d2672f4a91610bc37bf on panaetov:feature/send_logs_with_udp into 5c4dff132d0f955282cf36f3df5701ab27a43ded on fluent:master.

coveralls avatar Mar 15 '17 11:03 coveralls

Coverage Status

Coverage increased (+0.3%) to 90.045% when pulling 8c0b4b8fe51a14918e952d2672f4a91610bc37bf on panaetov:feature/send_logs_with_udp into 5c4dff132d0f955282cf36f3df5701ab27a43ded on fluent:master.

coveralls avatar Mar 15 '17 11:03 coveralls

JSON support should be separated to other PR.

repeatedly avatar Apr 04 '17 15:04 repeatedly