Hello Dan Burkert,
I'd like you to do a code review. Please visit
http://gerrit.cloudera.org:8080/5039
to review the following change.
Change subject: sasl: don't use the 'client_mech_list' SASL option
......................................................................
sasl: don't use the 'client_mech_list' SASL option
This option allows the client to specify which mechanisms it's willing
to negotiate. It turns out the option was added in Cyrus SASL 2.1.25,
which isn't available on el6.
While that option is useful, it's not doing anything magic: it just
filters the list of mechanisms that we pass from the server. This patch
just implements this filtering on our side.
This fixes the TestSaslRpc.TestNoMatchingMechanisms test case on el6.
Change-Id: I570a91e176803fdd0e4324e6f4443b8297ad0395
---
M src/kudu/rpc/sasl_client.cc
M src/kudu/rpc/sasl_helper.cc
2 files changed, 32 insertions(+), 34 deletions(-)
git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/39/5039/1
--
To view, visit http://gerrit.cloudera.org:8080/5039
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings
Gerrit-MessageType: newchange
Gerrit-Change-Id: I570a91e176803fdd0e4324e6f4443b8297ad0395
Gerrit-PatchSet: 1
Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-Owner: Todd Lipcon <[email protected]>
Gerrit-Reviewer: Dan Burkert <[email protected]>
