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 support for name compression to net/server. #351

Open
ximon18 opened this issue Jul 10, 2024 · 1 comment
Open

Add support for name compression to net/server. #351

ximon18 opened this issue Jul 10, 2024 · 1 comment
Labels
enhancement New feature or request

Comments

@ximon18
Copy link
Member

ximon18 commented Jul 10, 2024

Currently the unstable server code under src/net/server uses StreamTarget<Target> everywhere.

To support DNS name compression it would need to support wrapping that in StaticCompressor, or for AXFR that should probably be TreeCompressor.

(on a semi-related note: StreamTarget is perhaps misnamed as it supports both datagram and stream targets)

@ximon18 ximon18 added the enhancement New feature or request label Jul 10, 2024
@Philip-NLnetLabs
Copy link
Member

We need to avoid hard coding MessageBuilder and leave it to a higher level message object to use a particular type of compression

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