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

apps: call ERR_print_errors when OSSL_PROVIDER_load fails #14818

Closed

Conversation

gotthardp
Copy link
Contributor

The ERR_print_errors often displays the reason why the provider couldn't be loaded. Hence it is quite important for debugging.

The ERR_print_errors often displays the reason why the provider
couldn't be loaded. Hence it is quite important for debugging.
@paulidale paulidale added approval: review pending This pull request needs review by a committer branch: master Merge to master branch labels Apr 12, 2021
@mspncp mspncp added approval: done This pull request has the required number of approvals and removed approval: review pending This pull request needs review by a committer labels Apr 12, 2021
@openssl-machine
Copy link
Collaborator

24 hours has passed since 'approval: done' was set, but this PR has failing CI tests. Once the tests pass it will get moved to 'approval: ready to merge' automatically, alternatively please review and set the label manually.

openssl-machine pushed a commit that referenced this pull request Apr 13, 2021
The ERR_print_errors often displays the reason why the provider
couldn't be loaded. Hence it is quite important for debugging.

Reviewed-by: Matthias St. Pierre <Matthias.St.Pierre@ncp-e.com>
Reviewed-by: Paul Dale <pauli@openssl.org>
(Merged from #14818)
@paulidale
Copy link
Contributor

Merged to master, thanks for the contribution.

@paulidale paulidale closed this Apr 13, 2021
@gotthardp gotthardp deleted the print_provider_load_errors branch April 15, 2021 08:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
approval: done This pull request has the required number of approvals branch: master Merge to master branch
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants