Skip to content

Document UDP connection ID validation policy#2002

Merged
josecelano merged 5 commits into
torrust:developfrom
josecelano:1136-connection-id-validation-policy
Jul 20, 2026
Merged

Document UDP connection ID validation policy#2002
josecelano merged 5 commits into
torrust:developfrom
josecelano:1136-connection-id-validation-policy

Conversation

@josecelano

Copy link
Copy Markdown
Member

Summary

Scope

This is a specification-only PR. It does not implement the configuration or UDP behavior. Implementation starts after this specification is reviewed and merged.

Related to #1136
Related to #1978

Verification

  • linter all
  • pre-commit hook: cargo machete, cargo deny check bans, all linters, workspace doc tests
  • pre-push hook: nightly format/check/docs and full stable workspace tests
  • YAML frontmatter parsing
  • git diff --check
  • GPG signature verification

Copilot AI review requested due to automatic review settings July 20, 2026 12:40

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR updates the configuration-overhaul EPIC documentation to include and position issue #1136, and adds a new spec document defining the UDP connection ID validation policy (strict/disabled) intended for schema v3.

Changes:

  • Add a new issue specification doc for #1136 defining the per-listener UDP connection ID validation policy and verification plan.
  • Update EPIC #1978 spec to include #1136 (and reconcile ordering/links with existing #1987) and reflect 11 ordered subissues.

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 2 comments.

File Description
docs/issues/open/1978-configuration-overhaul-epic.md Updates EPIC metadata, links, and subissue ordering to include #1136 and align the dependency/plan sections accordingly.
docs/issues/open/1136-1978-configurable-udp-connection-id-validation-policy.md Adds the full specification for #1136, including security rationale, scope, acceptance criteria, and verification plan.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread docs/issues/open/1978-configuration-overhaul-epic.md Outdated
@josecelano josecelano self-assigned this Jul 20, 2026
@josecelano

Copy link
Copy Markdown
Member Author

ACK c7663a0

@josecelano
josecelano merged commit 5946eee into torrust:develop Jul 20, 2026
1 check passed
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