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

nss: try to connect even if libnssckbi.so fails to load #3016

Closed
wants to merge 1 commit into from

Conversation

kdudka
Copy link
Contributor

@kdudka kdudka commented Sep 19, 2018

One can still use CA certificates stored in NSS database.

Reported-by: Maxime Legros
Bug: https://curl.haxx.se/mail/lib-2018-09/0077.html

One can still use CA certificates stored in NSS database.

Reported-by: Maxime Legros
Bug: https://curl.haxx.se/mail/lib-2018-09/0077.html
Copy link
Member

@bagder bagder left a comment

Choose a reason for hiding this comment

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

Makes sense!

@kdudka kdudka closed this in eb0b3ac Sep 19, 2018
@kdudka kdudka deleted the nss-trust branch September 19, 2018 15:00
@jay
Copy link
Member

jay commented Sep 19, 2018

Does this fix the entire bug, if the module names are hardcoded shouldn't it be ifdef'd so that WIN32 uses .dll instead of .so

@kdudka
Copy link
Contributor Author

kdudka commented Sep 20, 2018

Yes, that is an orthogonal issue. Note that according to https://curl.haxx.se/mail/lib-2018-09/0077.html it is not just the .dll vs. .so suffix but also the lib prefix. Whether or not all Windows distributions of NSS use the same names for shared libraries is, however, not clear from that bug report and I have no way to experiment with that.

jay added a commit to jay/curl that referenced this pull request Oct 3, 2018
- Use .DLL extension instead of .so to load modules on Windows.

Bug: https://curl.haxx.se/mail/lib-2018-09/0077.html
Reported-by: Maxime Legros

Ref: curl#3016

Closes #xxxx
jay added a commit that referenced this pull request Oct 3, 2018
- Use .DLL extension instead of .so to load modules on Windows.

Bug: https://curl.haxx.se/mail/lib-2018-09/0077.html
Reported-by: Maxime Legros

Ref: #3016

Closes #3086
@lock lock bot locked as resolved and limited conversation to collaborators Dec 19, 2018
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.

3 participants