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

Remove unknown_psk_identity? #230

Closed
ekr opened this issue Aug 27, 2015 · 1 comment
Closed

Remove unknown_psk_identity? #230

ekr opened this issue Aug 27, 2015 · 1 comment

Comments

@ekr
Copy link
Contributor

ekr commented Aug 27, 2015

This doesn't really make sense with the current PSK
negotiation scheme where the client provides multiple PSKs in
flight 1.

@davegarrett
Copy link
Contributor

There's no reason a client couldn't send a giant pile of identities and still have them all be useless for the server in question. Likewise, the server could still have a bug and pick an unsupported PSK, in which case having a specific error would help debugging. There's also the new use-case of PSK resumption, so if a client picks a PSK for resumption that the server doesn't have around to use anymore, it could easily hit this error.

It's also a pre-existing error alert, so I don't see a point in removing it now. Might as well just leave in as it's already standardized and is technically optional anyway. (defined in RFC 4279)

@ekr ekr closed this as completed Sep 22, 2015
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

No branches or pull requests

2 participants