Skip to content

fix(network): harden TLS and DTLS security configuration - #2162

Merged
mbuckton merged 19 commits into
developmentfrom
MAPS-0-tls-dtls-security
Aug 7, 2026
Merged

fix(network): harden TLS and DTLS security configuration#2162
mbuckton merged 19 commits into
developmentfrom
MAPS-0-tls-dtls-security

Conversation

@mbuckton

@mbuckton mbuckton commented Aug 7, 2026

Copy link
Copy Markdown
Contributor

Summary

  • load DTLS endpoints from security.dtls and TLS endpoints from security.tls
  • preserve configured DTLS client-certificate requirements
  • expose peer certificate identities for required and requested TLS/DTLS authentication
  • verify outbound TLS hostnames by default, with hostnameVerificationEnabled as an explicit opt-out
  • apply nested TLS/DTLS settings during dynamic updates and update stored endpoint state
  • serialize context, CRL, hostname, and key/trust-store settings consistently
  • add focused configuration, DTLS, update-path, and outbound-engine regression tests

Companion change

SSL engine and CRL trust-manager fixes: Maps-Messaging/authentication_library#464

Verification

Focused JUnit tests were added for profile selection, nested updates, stored-state updates, DTLS client-auth preservation, and hostname verification. The available GitHub Maven workflow is running against the branch; no full local build is available in this managed environment.

SECURITY: TLS and DTLS transport hardening

mbuckton added 16 commits August 7, 2026 15:19
SECURITY: enable hostname verification by default
SECURITY: preserve SSL settings across load and update
SECURITY: persist dynamic TLS policy changes
SECURITY: use the DTLS security profile
SECURITY: do not disable client certificates per session
SECURITY: retain certificate identity after handshake
SECURITY: retain requested certificate identity
SECURITY: enable configurable endpoint identification
SECURITY: make dynamic TLS policy updates effective
SECURITY: document CRL interval units
@coderabbitai

coderabbitai Bot commented Aug 7, 2026

Copy link
Copy Markdown

Important

Review skipped

Auto reviews are disabled on base/target branches other than the default branch.

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro Plus

Run ID: 3df92972-7f72-4208-9f55-6aac7aee5bb1

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

  • 🔍 Trigger review

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@mbuckton
mbuckton marked this pull request as ready for review August 7, 2026 06:02
@mbuckton
mbuckton merged commit 9964714 into development Aug 7, 2026
2 of 3 checks passed
@mbuckton
mbuckton deleted the MAPS-0-tls-dtls-security branch August 7, 2026 06:02
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