Skip to content

Fragment TLS 1.3 NewSessionTicket to max fragment length - #11578

Open
embhorn wants to merge 1 commit into
wolfSSL:masterfrom
embhorn:gh11079
Open

embhorn wants to merge 1 commit into
wolfSSL:masterfrom
embhorn:gh11079

Conversation

@embhorn

@embhorn embhorn commented Sep 29, 2026

Copy link
Copy Markdown
Member

Description

Split the ticket into several records when it is larger than the negotiated max fragment length.

Fixes #11079

Testing

Added a regression test.

Checklist

  • added tests
  • updated/added doxygen
  • updated appropriate READMEs
  • Updated manual and documentation

@embhorn embhorn self-assigned this Sep 29, 2026
Copilot AI balanced review requested due to automatic review settings September 29, 2026 12:59

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Copilot review overview

🟡 Changes recommended

The regression test can pass without exercising fragmentation when the generated ticket fits in one record.

Review effort: Balanced
Findings: 1 Low severity

Open (1)
What changed in this PR

Adds TLS 1.3 NewSessionTicket fragmentation to respect negotiated record limits.

Changes:

  • Fragments oversized tickets across adjacent TLS records.
  • Updates ticket-size rationale for resumed ClientHello messages.
  • Adds a fragmentation and resumption test.
File Description
src/​tls13.c Implements ticket fragmentation.
src/​internal.c Updates the peer-certificate exclusion rationale.
tests/​api/​test_tls13.c Adds the regression test.
tests/​api/​test_tls13.h Registers the new test.

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

Comment thread tests/api/test_tls13.c
@github-actions

github-actions Bot commented Sep 29, 2026 •

Copy link
Copy Markdown

MemBrowse Memory Report

gcc-arm-cortex-m4-openssl-compat

  • FLASH: .text +256 B (+0.0%, 789,276 B / 1,048,576 B, total: 75% used)

gcc-arm-cortex-m4-pq

  • FLASH: .text +256 B (+0.1%, 307,184 B / 1,048,576 B, total: 29% used)

gcc-arm-cortex-m4-rsa-only

  • FLASH: .text +192 B (+0.1%, 337,328 B / 1,048,576 B, total: 32% used)

gcc-arm-cortex-m4-tls13

  • FLASH: .text +192 B (+0.1%, 246,110 B / 262,144 B, total: 94% used)

gcc-arm-cortex-m7-pq

  • FLASH: .text +192 B (+0.1%, 308,080 B / 1,048,576 B, total: 29% used)

gcc-arm-cortex-m7-tls13

@embhorn embhorn assigned wolfSSL-Bot and unassigned embhorn Sep 29, 2026
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.

TLS 1.3 NewSessionTicket is not fragmented under reduced max fragment length

3 participants