-
Notifications
You must be signed in to change notification settings - Fork 5
Open
Labels
developmentStandard developmentStandard development
Description
Specification
Recently when we were debugging an issue in Polykey we ran into the following error.
INFO:polykey.PolykeyAgent.NodeConnectionManager.NodeConnectionForward [13.239.117.143:1314].QUICClient:ErrorQUICConnectionPeerTLS: QUIC Connection local TLS error - Peer closed with transport code 301
For the sake of clarity, if we create TLS errors like this we should include the error name along with the code so we don't have to dig though the code to know what the error is. We already have a defined enum called CryptoError that provides the forward and reverse mapping for these codes. We just need to add this to the error description so we can see Peer closed with transport code 301 (CertificateExpired)
The same treatment can be applied to the ConnectionErrorCode enum usage as well.
Additional context
Related: MatrixAI/Polykey#787
Tasks
- when creating the message for
ErrorQUICConnectionPeerTLSandErrorQUICConnectionLocalTLSerrors we want to use theCryptoErrorto include the name of the error code. - The same needs to be applied to
ConnectionErrorCodecodes.
Metadata
Metadata
Assignees
Labels
developmentStandard developmentStandard development