Unix sockets are UDP-esque, not TCP.

[robholland]

No description provided.

[agentzh]

@robholland There are two different types of unix domain sockets, stream type and datagram type. I think the existing stream typed unix domain sockets are important to retain. You're welcome, however, to submit a patch to add support for an additional option to control which type of unix domain socket to use.

[robholland]

Ok, I'll change the code to be enabled by a datagram option, rather than presuming that to be the case based on the presence of path.

[calio]

1. The reason we have a separate send_buffer is that the current lua thread will be switched out when we call sock:send, and another lua thread may run and write something to the log buffer. If we use the same buffer for storing purpose and sending purpose, then if send operation succeeds and we clear the log buffer, some newly come log messages may also be cleared. I made send_buffer a lua table in https://github.com/cloudflare/lua-resty-logger-socket/tree/send_buffer_table, you can do something further on top of this.
2. sock:setpeername also accept host and port as arguments, so I think you can add udp support for both regular socket and unix domain socket.

[robholland]

Ok, I'll take a look at that branch, thanks. I'll also test plain UDP.

[robholland]

This code currently reads ok to me, but testing in production it's behaving strangely in datagram mode, both connected to a unix domain socket and plain UDP. I'll do more testing tomorrow.

[flisky]

The cosocket has already support udp unix socket. I think PR #20 fixed this.