Conversation
f76c74f to
9c6a757
Compare
|
I dunno
|
e72ce07 to
de4f5c0
Compare
|
Thanks for the quick feedback! I've slimmed this PR down to focus on the exposing of OOB information and to be able to reach the net.Conn. |
|
I think what is need is to just expose the Conn in some way. In https://codeberg.org/miekg/dns/src/branch/main/response.go#L24 |
de4f5c0 to
a45e3dc
Compare
|
alright, I've made the changes regarding Conn. This works for my use-case of reading TCP_INFO. But for the OOB destination data, this can't be fetched with the Conner interface, since this data is lost once the read happens. |
|
I rather copy this than https://codeberg.org/miekg/dns/src/branch/main/response.go#L30 Have a type Conner interface {
Conn() net.Conn
Session() *Session
}And copy the session code as well |
- Allow 3rd parties to control socket options - Provide a way to retrieve oob destination information
a45e3dc to
29f17b4
Compare
|
done and done! |
|
been busy, but I'll take a look. this is likely that last feature-PR that will be merged here. |
What
Allow 3rd parties to control socket optionsHow
Created a new listener param to set Control of net.ListenConfigExposed internal OOB state with a newPacketConnStateConnerinterfaceWhy
I need this change to allow my CoreDNS plugin to:
set the freebind socket optionIf this gets accepted, I'll follow up with a PR in the CoreDNS repo for the socket control options