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

Privacy and PSK identifiers #1333

Merged
merged 1 commit into from
Feb 17, 2024
Merged

Privacy and PSK identifiers #1333

merged 1 commit into from
Feb 17, 2024

Conversation

emanjon
Copy link
Contributor

@emanjon emanjon commented Dec 18, 2023

Based on Christian Huitema discussion of potential solutions.
https://mailarchive.ietf.org/arch/msg/tls/QuKsIu1gZFDfLn1x-ZnOE_LQxyc/

Encrypting the PSK identity using mechanisms external to TLS is missing in the current text. I added as little as possible. I did not go into details of the external encryption (could be asymmetric, symmetric group key, or pairwise symmetric with trial decryption).

Copy link
Contributor

@ekr ekr left a comment

Choose a reason for hiding this comment

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

This LGTM. Will give it a few days to see if anyone objects.

@emanjon
Copy link
Contributor Author

emanjon commented Dec 28, 2023

Christian Huitema pointed out in a private discussion that alternatively the PSK can be encrypted instead of the PSK ID. This is already used in real-world systems, e.g., IETF ACE. I think it would be good to mention that as well. This is very similar to the TLS internal session tickets but external to TLS.

@martinthomson
Copy link
Contributor

Does this need more words to describe what sort of encryption would be acceptable?

Something that included a nonce would be OK, just as something rerandomizable would. But something that is effectively static would not achieve the stated goal.

@seanturner
Copy link
Contributor

@emanjon thoughts?

@ekr
Copy link
Contributor

ekr commented Feb 17, 2024

I am just going to merge this minimal change. We're giving people a hint, not telling them how to design it.

@ekr ekr merged commit 4fbb9ea into tlswg:main Feb 17, 2024
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.

4 participants