-
Notifications
You must be signed in to change notification settings - Fork 1.1k
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
move go-libp2p-tls here #1466
Merged
Merged
move go-libp2p-tls here #1466
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
close the underlying connection when the context is canceled
add support for ECDSA keys
avoid using interface{} when generating certificates
remove unneeded marshaling / unmarshaling when generating cert chain
TLS 1.3 will require Go >= 1.12, so we don't need any fixes that were specific to Go 1.10 any more.
TLS 1.3 support was recently made opt-in in Go 1.12, so we need to explicitly enable it.
use TLS 1.3
add an example (mainly for development)
improve peer verification
Co-Authored-By: marten-seemann <[email protected]>
remove setting of the TLS 1.3 GODEBUG flag
fix deprecated call to key.Bytes
fix: don't fail the handshake when the libp2p extension is critical
sync: update CI config files
add the peer ID to SecureInbound
According to RFC3280, the issuer field must not be empty.
migrate to standard Go tests, stop using Ginkgo
* use tls.Conn.HandshakeContext instead of tls.Conn.Handshake * make sure that crypto/tls picks up the handshake ctx cancelation in tests
feat: catch panics in TLS negotiation
Stebalien
approved these changes
Apr 27, 2022
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
As with the other cases, I'd consider renaming. But we can do that later.
We should retarget this PR libp2p/go-libp2p-tls#99 after this merges. Note to ourselves, we should comment on that PR after this merges. |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Part of #1187.