Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Delay query responses #39

Open
cyanfish opened this issue Jul 9, 2024 · 1 comment
Open

Delay query responses #39

cyanfish opened this issue Jul 9, 2024 · 1 comment
Labels
enhancement New feature or request

Comments

@cyanfish
Copy link

cyanfish commented Jul 9, 2024

Per https://datatracker.ietf.org/doc/html/rfc6762#section-6:

In any case where there may be multiple responses, such as queries
where the answer is a member of a shared resource record set, each
responder SHOULD delay its response by a random amount of time
selected with uniform random distribution in the range 20-120 ms.

I implemented this in my fork but perhaps it's worth making it configurable.

@jdomnitz jdomnitz added the enhancement New feature or request label Jul 9, 2024
@jdomnitz
Copy link
Owner

jdomnitz commented Jul 9, 2024

That's not spec correct either. A server is expected to be listening to broadcasts during that waiting period and respond accordingly. That waiting period only applies in some cases where as others like probe responses MUST be sent immediately.

This library needs a transmission scheduler with preemption to handle mandatory delays, response trimming, query aggregation and many other SHOULDs. I'll try to address these where it's unlikely to harm stability.

This library also needs to differentiate between shared and exclusive records. Then handling things like negative responses and deconfliction could be tackled.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

2 participants