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

Session ticket tls12 back up #5063

Closed
wants to merge 32 commits into from

Conversation

boquan-fang
Copy link
Contributor

@boquan-fang boquan-fang commented Jan 27, 2025

Testing:

Merge to main and run the CI. This draft PR is for testing purpose only.

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

Boquan Fang added 29 commits January 3, 2025 23:25
* checked the chosen psk keying material expiration.
* add reference check for conn->config
* makes the s2n_generate_ticket_lifetime function more readable
* fix the chosen_psk logic
* assert unsigned integer subtractions
* fix testing: ensure that keying material expiration is one week after
  current time
* use s2n_stuffer_reserve_uint32 to skip the session ticket lifetime
  calculation.
* add s2n_stuffer_reserve_uint32 to s2n_stuff.h
* specific errno for lifetime is zero
* renaming variable names in generate lifetime function
* use simple calculation to get key intro time
* add tests for zero lifetime case in tls 1.2 and 1.3
* fix the PSK lifetime is shortest
    * makes it to be half a week
* query real time two times for key intro time and current time without
  actually creating a key
* check for server keying material and PSK only for TLS1.3
* the client should not accept and should delete the nst if nst's
  lifetime has expired
@github-actions github-actions bot added the s2n-core team label Jan 27, 2025
@boquan-fang boquan-fang force-pushed the session-ticket-tls12-back-up branch 5 times, most recently from b5e1a99 to 75cb3f4 Compare January 27, 2025 19:52
@boquan-fang boquan-fang force-pushed the session-ticket-tls12-back-up branch from 75cb3f4 to e972c89 Compare January 27, 2025 20:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant