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

Add RR batching functionality. #383

Merged
merged 368 commits into from
Sep 25, 2024
Merged

Add RR batching functionality. #383

merged 368 commits into from
Sep 25, 2024

Conversation

ximon18
Copy link
Member

@ximon18 ximon18 commented Sep 4, 2024

Taken from the xfr branch in order to split PR #335 into several smaller PRs.

This PR implements a "batcher" of resource records which can fill up an in-progress response until it is full then invoke a callback to send it before starting to fill the next response. This is useful for large multi-response answers to queries such as AXFR and IXFR.

Will be followed by an XFR specific batcher implementation that is used by the XFR middleware in the xfr branch.

Philip-NLnetLabs and others added 30 commits September 3, 2024 23:55
…G signing uses mock time during Stelline tests.
…rving doesn't do this yet and this still needs fixing, and (b) it violates the expectations of the TSIG Stelline test that verifies at a byte level the TSIG response signature.
Co-authored-by: Ximon Eighteen <3304436+ximon18@users.noreply.github.com>
@ximon18 ximon18 marked this pull request as ready for review September 23, 2024 14:20
@ximon18 ximon18 changed the base branch from add-rfc1996-notify-support to main September 24, 2024 18:37
Add RFC 1996 "slave" side minimalistic NOTIFY request support, delegating the actual work to a caller supplied helper that implements a new Notifiable trait.
@ximon18 ximon18 merged commit e47b226 into main Sep 25, 2024
26 checks passed
@ximon18 ximon18 deleted the rr-batching branch September 25, 2024 13:08
ximon18 added a commit that referenced this pull request Oct 3, 2024
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.

2 participants