auth: return unknown if no supported auth is found

If NONE is supported, it will already be in the list of supported
protocols, so there's no need to default to it here. This prevents
clients that request the NONE protocol from authenticating when the
server only accepts CEPHX. Instead, they get -ENOTSUP from the
AuthMonitor.

Signed-off-by: Josh Durgin <josh.durgin@dreamhost.com>

diff --git a/src/auth/AuthSupported.cc b/src/auth/AuthSupported.cc
index ea577926564f8df3b89be3cc5179fa9cab7bbcf0..3524bcfb493221c0ed08623bdd58c3c73a48c86a 100644 (file)
--- a/src/auth/AuthSupported.cc
+++ b/src/auth/AuthSupported.cc
@@ -47,5 +47,5 @@ int AuthSupported::pick(const set<__u32>& supp)
   for (set<__u32>::const_reverse_iterator p = supp.rbegin(); p != supp.rend(); ++p)
     if (is_supported_auth(*p))
       return *p;
-  return CEPH_AUTH_NONE;
+  return CEPH_AUTH_UNKNOWN;
 }