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

cyrptobox: work with OpenSSL-1.1 #2587

Closed
wants to merge 1 commit into
base: master
from

Conversation

Projects
None yet
2 participants
@rsmarples

rsmarples commented Oct 12, 2018

ERR_load_EC_strings and friends don't need to be called with
OpenSSL-1.1. Infact, on some platforms it causes a cyclic init
which stops rspamd from working entirely.

Luckily OpenSSL-1.1 should load these string automatically so we
can avoid calling them.

cyrptobox: work with OpenSSL-1.1
ERR_load_EC_strings and friends don't need to be called with
OpenSSL-1.1. Infact, on some platforms it causes a cyclic init
which stops rspamd from working entirely.

Luckily OpenSSL-1.1 should load these string automatically so we
can avoid calling them.
@rsmarples

This comment has been minimized.

rsmarples commented Oct 12, 2018

I don't understand the drone failure above. Exit code 3 is not clear as to why it's failing....

@vstakhov

This comment has been minimized.

Member

vstakhov commented Oct 12, 2018

Ignore drone.ci for now. I'm with you about the deprecation for these functions, however, I'm not completely sure about the proposed fix. I will check that later, thanks for report!

@vstakhov vstakhov closed this in 7444a19 Oct 12, 2018

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment