Skip to content

fix compilation without deprecated OpenSSL APIs#77

Open
neheb wants to merge 1 commit intofreeswitch:masterfrom
neheb:dep
Open

fix compilation without deprecated OpenSSL APIs#77
neheb wants to merge 1 commit intofreeswitch:masterfrom
neheb:dep

Conversation

@neheb
Copy link
Copy Markdown

@neheb neheb commented Jul 11, 2021

Initialization is done internally with 1.1 and above.

Signed-off-by: Rosen Penev rosenp@gmail.com

Initialization is done internally with 1.1 and above.

Signed-off-by: Rosen Penev <rosenp@gmail.com>
@mjerris
Copy link
Copy Markdown
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
Copy Markdown
Author

neheb commented Jul 12, 2021

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

@xrobau
Copy link
Copy Markdown

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
Copy Markdown

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

@Neustradamus
Copy link
Copy Markdown

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

@xrobau
Copy link
Copy Markdown

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
Copy Markdown

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