Skip to content

Conversation

@tiran
Copy link
Contributor

@tiran tiran commented Sep 26, 2018

See: #4197
Fixes: #3526
Signed-off-by: Christian Heimes [email protected]

See: pyca#4197
Signed-off-by: Christian Heimes <[email protected]>
(OPENSSL_VERSION_NUMBER >= 0x10100000 && !CRYPTOGRAPHY_IS_LIBRESSL)
#define CRYPTOGRAPHY_OPENSSL_110F_OR_GREATER \
(OPENSSL_VERSION_NUMBER >= 0x1010006f && !CRYPTOGRAPHY_IS_LIBRESSL)
#define CRYPTOGRAPHY_OPENSSL_111_OR_GREATER \
Copy link
Member

Choose a reason for hiding this comment

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

This isn't actually used, please remove it.

static const long Cryptography_HAS_CIPHER_DETAILS = 1;
#endif
#if CRYPTOGRAPHY_OPENSSL_LESS_THAN_111
Copy link
Member

Choose a reason for hiding this comment

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

This attribute needs to be available only when it's really present in OpenSSL. See #4476 for all the places that need editing.

@reaperhulk
Copy link
Member

Replaced by #4555

@reaperhulk reaperhulk closed this Nov 3, 2018
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Aug 4, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Development

Successfully merging this pull request may close these issues.

Add binding for OP_NO_TLSv1_3

3 participants