auth: introduce DummyAuthSessionHandler.

Signed-off-by: Radoslaw Zarzynski <rzarzyns@redhat.com>

diff --git a/src/auth/AuthSessionHandler.h b/src/auth/AuthSessionHandler.h
index 341db88e72153d0f9e4324ecbf7b9f60bd3e1c9a..a2e688ffe8c94f2c030208f88e0165298b57dfb3 100644 (file)
--- a/src/auth/AuthSessionHandler.h
+++ b/src/auth/AuthSessionHandler.h
@@ -41,6 +41,21 @@ struct AuthSessionHandler {
   }
 };
 
+struct DummyAuthSessionHandler : AuthSessionHandler {
+  int sign_message(Message*) final {
+    return 0;
+  }
+  int check_message_signature(Message*) final {
+    return 0;
+  }
+  int encrypt_message(Message*) final {
+    return 0;
+  }
+  int decrypt_message(Message*) final {
+    return 0;
+  }
+};
+
 extern AuthSessionHandler *get_auth_session_handler(
   CephContext *cct, int protocol,
   const CryptoKey& key,

diff --git a/src/auth/krb/KrbSessionHandler.hpp b/src/auth/krb/KrbSessionHandler.hpp
index 9f715730a56d1abc7a3084022f89227b982dfc27..ee80d7909333a0fc62176862f1f196c1d5bed36d 100644 (file)
--- a/src/auth/krb/KrbSessionHandler.hpp
+++ b/src/auth/krb/KrbSessionHandler.hpp
@@ -29,21 +29,7 @@
  
 #define dout_subsys ceph_subsys_auth
 
-class Message;
-
-class KrbSessionHandler : public AuthSessionHandler {
-
-  public:
-    KrbSessionHandler() = default;
-    ~KrbSessionHandler() override = default; 
-
-    // No security
-    int sign_message(Message* msg) override { return 0; }
-    int check_message_signature(Message* msg) override { return 0; }
-    int encrypt_message(Message* msg) override { return 0; }
-    int decrypt_message(Message* msg) override { return 0; }
-
-  private:
+struct KrbSessionHandler : DummyAuthSessionHandler {
 };
 
 #endif    //-- KRB_SESSION_HANDLER_HPP

diff --git a/src/auth/none/AuthNoneSessionHandler.h b/src/auth/none/AuthNoneSessionHandler.h
index 2c738191f5edbd804385062fb86875ae2c8e1798..ca1451f797a8bcbb5533e8d6ca797356de3deda1 100644 (file)
--- a/src/auth/none/AuthNoneSessionHandler.h
+++ b/src/auth/none/AuthNoneSessionHandler.h
@@ -13,31 +13,7 @@
  */
 
 #include "auth/AuthSessionHandler.h"
-#include "msg/Message.h"
-
-class AuthNoneSessionHandler : public AuthSessionHandler {
-public:
-  AuthNoneSessionHandler() = default;
-  ~AuthNoneSessionHandler() override = default;
-  
-  // The None suite neither signs nor encrypts messages, so these functions just return success.
-  // Since nothing was signed or encrypted, don't increment the stats.  PLR
-
-  int sign_message(Message *m) override {
-    return 0;
-  }
-
-  int check_message_signature(Message *m) override {
-    return 0;
-  }
-
-  int encrypt_message(Message *m) override {
-    return 0;
-  }
-
-  int decrypt_message(Message *m) override {
-    return 0;
-  }
 
+struct AuthNoneSessionHandler : DummyAuthSessionHandler {
 };
 

diff --git a/src/auth/unknown/AuthUnknownSessionHandler.h b/src/auth/unknown/AuthUnknownSessionHandler.h
index 9bf901004a2123e880b68f51143cf91da9139830..7230663da29b8413a7029facda897c4e5fbbc776 100644 (file)
--- a/src/auth/unknown/AuthUnknownSessionHandler.h
+++ b/src/auth/unknown/AuthUnknownSessionHandler.h
@@ -13,33 +13,7 @@
  */
 
 #include "auth/AuthSessionHandler.h"
-#include "msg/Message.h"
-
-#define dout_subsys ceph_subsys_auth
-
-class AuthUnknownSessionHandler : public AuthSessionHandler {
-public:
-  AuthUnknownSessionHandler() = default;
-  ~AuthUnknownSessionHandler() override = default;
-  
-  // The Unknown suite neither signs nor encrypts messages, so these functions just return success.
-  // Since nothing was signed or encrypted, don't increment the stats.  PLR
-
-  int sign_message(Message *m) override {
-    return 0;
-  }
-
-  int check_message_signature(Message *m) override {
-    return 0;
-  }
-
-  int encrypt_message(Message *m) override {
-    return 0;
-  }
-
-  int decrypt_message(Message *m) override {
-    return 0;
-  }
 
+struct AuthUnknownSessionHandler : DummyAuthSessionHandler {
 };