You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
@kealist was trying to read https://www.mtgjson.com/files/AllCards.json and it did not support any of the cipher suites we use. Their TLS 1.2 options all use Curve25519, so I went ahead and added support for picking curves since that wasn't too difficult.
But besides all using "x25519", there's another difference...that all of them use ECDSA instead of RSA. This is a bit less obvious how to implement.
More generally--the TLS code needs a revisiting now that there is more of a sound set of cryptography primitives to build on.
Here are the ciphers supported by TLS 1.2 currently on mtgjson:
@kealist was trying to read
https://www.mtgjson.com/files/AllCards.json
and it did not support any of the cipher suites we use. Their TLS 1.2 options all use Curve25519, so I went ahead and added support for picking curves since that wasn't too difficult.But besides all using "x25519", there's another difference...that all of them use ECDSA instead of RSA. This is a bit less obvious how to implement.
More generally--the TLS code needs a revisiting now that there is more of a sound set of cryptography primitives to build on.
Here are the ciphers supported by TLS 1.2 currently on mtgjson:
The text was updated successfully, but these errors were encountered: