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

Fixes #495. Implement name compression in DNS response packet generation. #503

Closed
wants to merge 5 commits into from

Conversation

amialkow
Copy link
Contributor

Implement name compression during response packet generation. Compression is required by certain clients like UDP to fit response in packet size limit. While generating packet small cache stores recently used names (currently 4 entries) and uses relative references to previous instances of the same name. Each reused instance is just two bytes of relative reference (0xC000 + offset). Cache is currently performing lookup for query name, responses and CNAMEs.

…sion is required by certain clients like UDP to fit response in packet size limit. While generating packet small cache stores recently used names (currently 4 entries) and uses relative references to previous instances of the same name. Each reused instance is just two bytes of relative reference (0xC000 + offset). Cache is currently performing lookup for query name, responses and CNAMEs.
@amialkow amialkow changed the title Fixes 95. Implement name compression in DNS response packet generation. Fixes #495. Implement name compression in DNS response packet generation. May 15, 2021
@amialkow
Copy link
Contributor Author

Tree compilation nee to be fixed first to pull this feature, already started in #504. Closing for now. Will submit separately as standalone change (#495 and #505).

@amialkow amialkow closed this May 15, 2021
@amialkow amialkow deleted the develop branch May 15, 2021 20:50
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.

1 participant