Skip to content
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

fix compilation without deprecated OpenSSL APIs #77

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

neheb
Copy link

@neheb neheb commented Jul 11, 2021

Initialization is done internally with 1.1 and above.

Signed-off-by: Rosen Penev [email protected]

Initialization is done internally with 1.1 and above.

Signed-off-by: Rosen Penev <[email protected]>
@mjerris
Copy link
Member

mjerris commented Jul 12, 2021

are these calls never needed anymore or not needed only because we call them elsewhere in freeswitch? if the latter, this commit won’t do, it still needs to support use in other apps

@neheb
Copy link
Author

neheb commented Jul 12, 2021

no OpenSSL internally calls these. These APIs are no op in 1.1.

@xrobau
Copy link

xrobau commented Jul 28, 2021

Confirming @neheb 's post - here's the documentation that says to do exactly this.

https://wiki.openssl.org/index.php/Library_Initialization

@Neustradamus
Copy link

@freeswitch team: Have you planned the merging of this PR?

@Neustradamus
Copy link

@mjerris, @xrobau: Any news after more one year of this PR?

@xrobau
Copy link

xrobau commented Oct 13, 2022

Not that I can do much more than approve the changes, the Signalwire folk need to actually merge it 8)

I can't remember what I did to work around this with the freeswitch ubuntu building container, but you may want to look through that

https://github.com/xrobau/freeswitch-ubuntubuilder

@Neustradamus
Copy link

Dear @freeswitch team,

Any progress on this PR?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants