Skip to content

RFC correctness improvements#128

Merged
msimerson merged 3 commits into
masterfrom
rfc-fixes
May 26, 2026
Merged

RFC correctness improvements#128
msimerson merged 3 commits into
masterfrom
rfc-fixes

Conversation

@msimerson
Copy link
Copy Markdown
Collaborator

  • feat(packet): encode name compression pointers (RFC 1035 §4.1.4)
  • feat(server/udp): negotiated UDP payload size with TC=1 on oversize
  • feat(server/tcp): pipeline support (RFC 7766 §6.2.1.1)
  • feat(packet): EDNS extended RCODE supported
  • fix(packet): EDNS default UDP payload size raised to 4096
  • fix(packet): clamps TTLs to 2³¹−1
  • fix(packet): Label and name length validation (RFC 1035 §2.3.4)
  • fix(server/doh): accepts any (or absent) Accept header (RFC 8484 §4.1)
  • fix(server/doh): DoH POST requires Content-Type: application/dns-message
  • feat(server/doh): DoH responses include TTL-derived Cache-Control
  • fix(packet): Packet.Header.toBuffer writes Z=0 (RFC 1035 §4.1.1)

- feat(packet): encode name compression pointers (RFC 1035 §4.1.4)
- feat(server/udp): negotiated UDP payload size with TC=1 on oversize
- feat(server/tcp): pipeline support (RFC 7766 §6.2.1.1)
- feat(packet): EDNS extended RCODE supported
- fix(packet): EDNS default UDP payload size raised to 4096
- fix(packet): clamps TTLs to 2³¹−1
- fix(packet): Label and name length validation (RFC 1035 §2.3.4)
- fix(server/doh): accepts any (or absent) Accept header (RFC 8484 §4.1)
- fix(server/doh): DoH POST requires Content-Type: application/dns-message
- feat(server/doh): DoH responses include TTL-derived Cache-Control
- fix(packet): Packet.Header.toBuffer writes Z=0 (RFC 1035 §4.1.1)

This comment was marked as resolved.

@msimerson msimerson marked this pull request as ready for review May 26, 2026 16:11
@msimerson msimerson requested a review from Copilot May 26, 2026 16:11

This comment was marked as resolved.

@msimerson msimerson merged commit fd8c8f0 into master May 26, 2026
5 checks passed
@msimerson msimerson deleted the rfc-fixes branch May 26, 2026 17:04
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