-
Notifications
You must be signed in to change notification settings - Fork 89
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
Issue 6324 - Provide more information in the error message during setup_ol_tls_conn() #6325
Conversation
…up_ol_tls_conn() Description: When there's a problem with creating a new TLS context, we just fail with -1 error code. We can improve it by providing more information that can be extracted from LDAP_OPT_DIAGNOSTIC_MESSAGE or LDAP_OPT_ERROR_STRING. Use slapi_ldap_get_lderrno to get more information from the ld structure. Add a test case. Fixes: 389ds#6324 Reviewed by: ?
I'm not sure why but the test passes on my machine (and on a virtual machine). But it can't find the error message in GitHub CI. |
At least the C code looks good. |
Okay, the issue was that I tried to grab the wrong error. The error I need to check for is: But there's another error that happens earlier in the function. And they are different between GitHub CI and my virtual machine mock run. GitHub CI - Why it's different, it's another story I am not sure still. |
What version of openssl do you have? In the CI image I see openssl-3.2.2-3.fc40 |
It's the same on my machine - openssl-3.2.2-3.fc40.x86_64 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
…up_ol_tls_conn() (#6325) Description: When there's a problem with creating a new TLS context, we just fail with -1 error code. We can improve it by providing more information that can be extracted from LDAP_OPT_DIAGNOSTIC_MESSAGE or LDAP_OPT_ERROR_STRING. Use slapi_ldap_get_lderrno to get more information from the ld structure. Add a test case. Fixes: #6324 Reviewed by: @progier389 (Thanks!)
…up_ol_tls_conn() (#6325) Description: When there's a problem with creating a new TLS context, we just fail with -1 error code. We can improve it by providing more information that can be extracted from LDAP_OPT_DIAGNOSTIC_MESSAGE or LDAP_OPT_ERROR_STRING. Use slapi_ldap_get_lderrno to get more information from the ld structure. Add a test case. Fixes: #6324 Reviewed by: @progier389 (Thanks!)
…up_ol_tls_conn() (#6325) Description: When there's a problem with creating a new TLS context, we just fail with -1 error code. We can improve it by providing more information that can be extracted from LDAP_OPT_DIAGNOSTIC_MESSAGE or LDAP_OPT_ERROR_STRING. Use slapi_ldap_get_lderrno to get more information from the ld structure. Add a test case. Fixes: #6324 Reviewed by: @progier389 (Thanks!)
…up_ol_tls_conn() (#6325) Description: When there's a problem with creating a new TLS context, we just fail with -1 error code. We can improve it by providing more information that can be extracted from LDAP_OPT_DIAGNOSTIC_MESSAGE or LDAP_OPT_ERROR_STRING. Use slapi_ldap_get_lderrno to get more information from the ld structure. Add a test case. Fixes: #6324 Reviewed by: @progier389 (Thanks!)
…up_ol_tls_conn() (#6325) Description: When there's a problem with creating a new TLS context, we just fail with -1 error code. We can improve it by providing more information that can be extracted from LDAP_OPT_DIAGNOSTIC_MESSAGE or LDAP_OPT_ERROR_STRING. Use slapi_ldap_get_lderrno to get more information from the ld structure. Add a test case. Fixes: #6324 Reviewed by: @progier389 (Thanks!)
…up_ol_tls_conn() (#6325) Description: When there's a problem with creating a new TLS context, we just fail with -1 error code. We can improve it by providing more information that can be extracted from LDAP_OPT_DIAGNOSTIC_MESSAGE or LDAP_OPT_ERROR_STRING. Use slapi_ldap_get_lderrno to get more information from the ld structure. Add a test case. Fixes: #6324 Reviewed by: @progier389 (Thanks!)
…up_ol_tls_conn() (#6325) Description: When there's a problem with creating a new TLS context, we just fail with -1 error code. We can improve it by providing more information that can be extracted from LDAP_OPT_DIAGNOSTIC_MESSAGE or LDAP_OPT_ERROR_STRING. Use slapi_ldap_get_lderrno to get more information from the ld structure. Add a test case. Fixes: #6324 Reviewed by: @progier389 (Thanks!)
…up_ol_tls_conn() (#6325) Description: When there's a problem with creating a new TLS context, we just fail with -1 error code. We can improve it by providing more information that can be extracted from LDAP_OPT_DIAGNOSTIC_MESSAGE or LDAP_OPT_ERROR_STRING. Use slapi_ldap_get_lderrno to get more information from the ld structure. Add a test case. Fixes: #6324 Reviewed by: @progier389 (Thanks!)
Description: When there's a problem with creating a new TLS context, we just fail with -1 error code. We can improve it by providing more information that can be extracted from LDAP_OPT_DIAGNOSTIC_MESSAGE or LDAP_OPT_ERROR_STRING.
Use slapi_ldap_get_lderrno to get more information from the ld structure. Add a test case.
Fixes: #6324
Reviewed by: ?