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

digest_sspi: Show InitializeSecurityContext errors in verbose mode #6315

Closed
wants to merge 1 commit into from

Conversation

@jay
Copy link
Member

@jay jay commented Dec 13, 2020

The error is shown with infof rather than failf so that the user will
see the extended error message information only in verbose mode, and
will still see the standard CURLE_AUTH_ERROR message. For example:


  • schannel: InitializeSecurityContext failed: SEC_E_QOP_NOT_SUPPORTED
    (0x8009030A) - The per-message Quality of Protection is not supported by
    the security package
  • multi_done
  • Connection #1 to host 127.0.0.1 left intact
    curl: (94) An authentication function returned an error

Ref: #6302

Closes #xxxx

The error is shown with infof rather than failf so that the user will
see the extended error message information only in verbose mode, and
will still see the standard CURLE_AUTH_ERROR message. For example:

---

* schannel: InitializeSecurityContext failed: SEC_E_QOP_NOT_SUPPORTED
(0x8009030A) - The per-message Quality of Protection is not supported by
the security package
* multi_done
* Connection #1 to host 127.0.0.1 left intact
curl: (94) An authentication function returned an error

---

Ref: #6302

Closes #xxxx
@bagder
bagder approved these changes Dec 13, 2020
@jay jay closed this in c545659 Dec 14, 2020
@jay jay deleted the jay:improve_sspi_auth_errmsg branch Dec 14, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

2 participants