Skip to content

support dgram unix socket#20

Open
flisky wants to merge 2 commits into
cloudflare:masterfrom
flisky:master
Open

support dgram unix socket#20
flisky wants to merge 2 commits into
cloudflare:masterfrom
flisky:master

Conversation

@flisky

@flisky flisky commented Nov 3, 2016

Copy link
Copy Markdown

such as imuxsock from rsyslog

such as imuxsock from rsyslog
@dndx

dndx commented Nov 3, 2016

Copy link
Copy Markdown

Thanks for your contribution! Can you please provide test cases for this addition inside t/sanity.t similar to TEST 2?

@flisky

flisky commented Nov 4, 2016

Copy link
Copy Markdown
Author

I'll try it in next few days.

@flisky

flisky commented Nov 9, 2016

Copy link
Copy Markdown
Author

Testcase added.

A known issue -
the directive udp_reply is required, but we won't call sock:receive(), so the following warning occurs:
WARNING: udp server failed to send reply: Connection refused

Is it acceptable for us? @agentzh, @dndx, Any thoughts on this?

@geemang2000

Copy link
Copy Markdown

👍

@calio

calio commented Dec 20, 2016

Copy link
Copy Markdown
Contributor

@dndx can you take a look?

@flisky

flisky commented Jun 2, 2017

Copy link
Copy Markdown
Author

I think maybe @p0pr0ck5 intersted? Thx,

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants