You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@qpid.apache.org by as...@apache.org on 2015/08/05 00:04:03 UTC

qpid-proton git commit: NO-JIRA: Include selected mechanism with authentication failures - This shoiuld help with the cases where GSSAPI gets selected without the user realising

Repository: qpid-proton
Updated Branches:
  refs/heads/master 9e7cb674c -> 64fa59ce2


NO-JIRA: Include selected mechanism with authentication failures
- This shoiuld help with the cases where GSSAPI gets selected
  without the user realising


Project: http://git-wip-us.apache.org/repos/asf/qpid-proton/repo
Commit: http://git-wip-us.apache.org/repos/asf/qpid-proton/commit/64fa59ce
Tree: http://git-wip-us.apache.org/repos/asf/qpid-proton/tree/64fa59ce
Diff: http://git-wip-us.apache.org/repos/asf/qpid-proton/diff/64fa59ce

Branch: refs/heads/master
Commit: 64fa59ce2cf728a2e32ab6cc97f78dea26f55b42
Parents: 9e7cb67
Author: Andrew Stitcher <as...@apache.org>
Authored: Tue Aug 4 18:02:54 2015 -0400
Committer: Andrew Stitcher <as...@apache.org>
Committed: Tue Aug 4 18:02:54 2015 -0400

----------------------------------------------------------------------
 proton-c/src/sasl/sasl.c | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/qpid-proton/blob/64fa59ce/proton-c/src/sasl/sasl.c
----------------------------------------------------------------------
diff --git a/proton-c/src/sasl/sasl.c b/proton-c/src/sasl/sasl.c
index a01cce5..83543ec 100644
--- a/proton-c/src/sasl/sasl.c
+++ b/proton-c/src/sasl/sasl.c
@@ -225,7 +225,7 @@ static void pni_post_sasl_frame(pn_transport_t *transport)
       pn_post_frame(transport, SASL_FRAME_TYPE, 0, "DL[B]", SASL_OUTCOME, sasl->outcome);
       pni_emit(transport);
       if (sasl->outcome!=PN_SASL_OK) {
-        pn_do_error(transport, "amqp:unauthorized-access", "Failed to authenticate client");
+        pn_do_error(transport, "amqp:unauthorized-access", "Failed to authenticate client [mech=%s]", transport->sasl->selected_mechanism);
         desired_state = SASL_ERROR;
       }
       break;
@@ -235,7 +235,7 @@ static void pni_post_sasl_frame(pn_transport_t *transport)
         continue;
       }
       if (sasl->outcome!=PN_SASL_OK) {
-        pn_do_error(transport, "amqp:unauthorized-access", "Authentication failed");
+        pn_do_error(transport, "amqp:unauthorized-access", "Authentication failed [mech=%s]", transport->sasl->selected_mechanism);
         desired_state = SASL_ERROR;
       }
       break;


---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@qpid.apache.org
For additional commands, e-mail: commits-help@qpid.apache.org