search

cloudflare / lua-resty-logger-socket

Raw-socket-based Logger Library for Nginx (based on ngx_lua)
486 stars 130 forks source link

Datagram Unix socket doesn't work #36

Open lvu opened 6 years ago

lvu commented 6 years ago

_do_connect() fails if socket_type is "udp" and path is specified instead of host, port

In this case, a UDP socket is created, and then its :connect() method is called, but there is no such method, so attempt to call method 'connect' (a nil value) error is thrown.

A simple fix is here: https://github.com/cloudflare/lua-resty-logger-socket/pull/35

flisky commented 6 years ago

see #20，almost 1 years old.

lvu commented 6 years ago

You mean nobody cares anymore? :(

idanga commented 4 years ago

No reason to use udp over UDS. https://stackoverflow.com/questions/13953912/difference-between-unix-domain-stream-and-datagram-sockets

JimmyBaize commented 3 years ago

No reason to use udp over UDS. https://stackoverflow.com/questions/13953912/difference-between-unix-domain-stream-and-datagram-sockets

But imuxsock from rsyslog only support udp UDS.