Rename UsernameCache to CredentialsCache

diff --git a/src/auth/UserNameCache.cc b/src/auth/CredentialsCache.cc
similarity index 75%
rename from src/auth/UserNameCache.cc
rename to src/auth/CredentialsCache.cc
index bedd64601a6e74c65dc917e67a0a6d4de2fceaea..fcc60ba76fe19e65c895cf9af1445f784b38604c 100644 (file)
--- a/src/auth/UserNameCache.cc
+++ b/src/auth/CredentialsCache.cc
@@ -10,7 +10,7 @@
 
 #include "squid.h"
 #include "acl/Gadgets.h"
-#include "auth/UserNameCache.h"
+#include "auth/CredentialsCache.h"
 #include "Debug.h"
 #include "event.h"
 #include "SquidConfig.h"
@@ -18,20 +18,21 @@
 
 namespace Auth {
 
-CBDATA_CLASS_INIT(UserNameCache);
+CBDATA_CLASS_INIT(CredentialsCache);
 
-UserNameCache::UserNameCache(const char *name) :
-    cachename(name), cacheCleanupEventName("User cache cleanup: ")
+CredentialsCache::CredentialsCache(const char *name) :
+    cachename(name),
+    cacheCleanupEventName("User cache cleanup: ")
 {
     debugs(29, 5, "initializing " << name << " username cache");
     cacheCleanupEventName.append(name);
-    eventAdd(cacheCleanupEventName.c_str(), &UserNameCache::Cleanup,
+    eventAdd(cacheCleanupEventName.c_str(), &CredentialsCache::Cleanup,
              this, ::Config.authenticateGCInterval, 1);
     RegisterRunner(this);
 }
 
 Auth::User::Pointer
-UserNameCache::lookup(const SBuf &userKey) const
+CredentialsCache::lookup(const SBuf &userKey) const
 {
     debugs(29, 6, "lookup for " << userKey);
     auto p = store_.find(userKey);
@@ -41,16 +42,16 @@ UserNameCache::lookup(const SBuf &userKey) const
 }
 
 void
-UserNameCache::Cleanup(void *data)
+CredentialsCache::Cleanup(void *data)
 {
     debugs(29, 5, "checkpoint");
     // data is this in disguise
-    UserNameCache *self = static_cast<UserNameCache *>(data);
+    CredentialsCache *self = static_cast<CredentialsCache *>(data);
     self->cleanup();
 }
 
 void
-UserNameCache::cleanup()
+CredentialsCache::cleanup()
 {
     // cache entries with expiretime <= expirationTime are to be evicted
     const time_t expirationTime =  current_time.tv_sec - ::Config.authenticateTTL;
@@ -66,21 +67,21 @@ UserNameCache::cleanup()
             ++i;
         }
     }
-    eventAdd(cacheCleanupEventName.c_str(), &UserNameCache::Cleanup,
+    eventAdd(cacheCleanupEventName.c_str(), &CredentialsCache::Cleanup,
              this, ::Config.authenticateGCInterval, 1);
 }
 
 void
-UserNameCache::insert(Auth::User::Pointer anAuth_user)
+CredentialsCache::insert(Auth::User::Pointer anAuth_user)
 {
     debugs(29, 6, "adding " << anAuth_user->userKey());
     store_[anAuth_user->userKey()] = anAuth_user;
 }
 
 // generates the list of cached usernames in a format that is convenient
-// to merge with equivalent lists obtained from other UserNameCaches.
+// to merge with equivalent lists obtained from other CredentialsCaches.
 std::vector<Auth::User::Pointer>
-UserNameCache::sortedUsersList() const
+CredentialsCache::sortedUsersList() const
 {
     std::vector<Auth::User::Pointer> rv(size(), nullptr);
     std::transform(store_.begin(), store_.end(), rv.begin(),
@@ -95,15 +96,15 @@ UserNameCache::sortedUsersList() const
 }
 
 void
-UserNameCache::endingShutdown()
+CredentialsCache::endingShutdown()
 {
     debugs(29, 5, "Shutting down username cache " << cachename);
-    eventDelete(&UserNameCache::Cleanup, this);
+    eventDelete(&CredentialsCache::Cleanup, this);
     reset();
 }
 
 void
-UserNameCache::syncConfig()
+CredentialsCache::syncConfig()
 {
     debugs(29, 5, "Reconfiguring username cache " << cachename);
     for (auto i : store_) {

diff --git a/src/auth/UserNameCache.h b/src/auth/CredentialsCache.h
similarity index 80%
rename from src/auth/UserNameCache.h
rename to src/auth/CredentialsCache.h
index b498ec3a6839e3925365aea931e8f2060f1e1aa0..a9797475e20a0164e80eb3ef1ea92663592fdf64 100644 (file)
--- a/src/auth/UserNameCache.h
+++ b/src/auth/CredentialsCache.h
@@ -6,8 +6,8 @@
  * Please see the COPYING and CONTRIBUTORS files for details.
  */
 
-#ifndef SQUID_SRC_AUTH_UERNAMECACHE_H
-#define SQUID_SRC_AUTH_UERNAMECACHE_H
+#ifndef SQUID_SRC_AUTH_CREDENTIALSCACHE_H
+#define SQUID_SRC_AUTH_CREDENTIALSCACHE_H
 
 #include "auth/User.h"
 #include "base/RunnersRegistry.h"
@@ -19,21 +19,21 @@
 namespace Auth {
 
 /// Cache of Auth::User credentials, keyed by Auth::User::userKey
-class UserNameCache : public RegisteredRunner
+class CredentialsCache : public RegisteredRunner
 {
 private:
-    CBDATA_CLASS(UserNameCache);
+    CBDATA_CLASS(CredentialsCache);
 
     /// key is User::userKey(), mapped value is User::Pointer
     typedef std::unordered_map<SBuf, Auth::User::Pointer> StoreType;
 
 public:
-    UserNameCache() = delete;
-    explicit UserNameCache(const char *name);
+    CredentialsCache() = delete;
+    explicit CredentialsCache(const char *name);
 
-    ~UserNameCache() = default;
-    UserNameCache(const UserNameCache& ) = delete;
-    UserNameCache& operator=(const UserNameCache&) = delete;
+    ~CredentialsCache() = default;
+    CredentialsCache(const CredentialsCache& ) = delete;
+    CredentialsCache& operator=(const CredentialsCache&) = delete;
 
     /// obtain pointer to user if present, or Pointer(nullptr) if not
     /// \returns a pointer to cached credentials, or nil if none found
@@ -80,5 +80,5 @@ private:
 
 } /* namespace Auth */
 
-#endif /* SQUID_SRC_AUTH_UERNAMECACHE_H */
+#endif /* SQUID_SRC_AUTH_CREDENTIALSCACHE_H */
 

diff --git a/src/auth/Gadgets.cc b/src/auth/Gadgets.cc
index a157721c0417e12e21be724af83aa1a407e6d7d4..33886a197041f9b11c01a167ddea8c003f9a8c11 100644 (file)
--- a/src/auth/Gadgets.cc
+++ b/src/auth/Gadgets.cc
@@ -24,7 +24,7 @@
 #include "auth/ntlm/User.h"
 #include "auth/Scheme.h"
 #include "auth/User.h"
-#include "auth/UserNameCache.h"
+#include "auth/CredentialsCache.h"
 #include "auth/UserRequest.h"
 #include "client_side.h"
 #include "globals.h"

diff --git a/src/auth/Makefile.am b/src/auth/Makefile.am
index f12dedda6ed6669d80faa01d36b62e59708ba2c2..68c39c6200c3b98b2a77e0be9616b5f4138a7d7b 100644 (file)
--- a/src/auth/Makefile.am
+++ b/src/auth/Makefile.am
@@ -32,8 +32,8 @@ libauth_la_SOURCES = \
        State.cc \
        User.h \
        User.cc \
-       UserNameCache.h \
-       UserNameCache.cc \
+       CredentialsCache.h \
+       CredentialsCache.cc \
        UserRequest.h \
        UserRequest.cc
 

diff --git a/src/auth/User.cc b/src/auth/User.cc
index 305fbbab69fe2b9122a31e2e6af5bdccfc60e05e..a9ba096efe158e1da3ab4232775b5b62bb2b4654 100644 (file)
--- a/src/auth/User.cc
+++ b/src/auth/User.cc
@@ -14,7 +14,7 @@
 #include "auth/Config.h"
 #include "auth/Gadgets.h"
 #include "auth/User.h"
-#include "auth/UserNameCache.h"
+#include "auth/CredentialsCache.h"
 #include "auth/UserRequest.h"
 #include "event.h"
 #include "globals.h"
@@ -242,7 +242,7 @@ Auth::User::BuildUserKey(const char *username, const char *realm)
  * Dump the username cache statictics for viewing...
  */
 void
-Auth::User::UsernameCacheStats(StoreEntry *output)
+Auth::User::CredentialsCacheStats(StoreEntry *output)
 {
     auto userlist = authenticateCachedUsersList();
     storeAppendPrintf(output, "Cached Usernames: %d", static_cast<int32_t>(userlist.size()));

diff --git a/src/auth/User.h b/src/auth/User.h
index 8ff1af048c16c40120b456000f962f8575e4231f..a2645245e27869d8c1ed2bac1baa603ed0cd204d 100644 (file)
--- a/src/auth/User.h
+++ b/src/auth/User.h
@@ -26,7 +26,7 @@ namespace Auth
 {
 
 class Config;
-class UserNameCache;
+class CredentialsCache;
 
 /**
  *  \ingroup AuthAPI
@@ -80,11 +80,11 @@ public:
 
     /// add the Auth::User to the protocol-specific username cache.
     virtual void addToNameCache() = 0;
-    static void UsernameCacheStats(StoreEntry * output);
+    static void CredentialsCacheStats(StoreEntry * output);
 
     // userKey ->Auth::User::Pointer cache
     // must be reimplemented in subclasses
-    static CbcPointer<Auth::UserNameCache> Cache();
+    static CbcPointer<Auth::CredentialsCache> Cache();
 
     CredentialState credentials() const;
     void credentials(CredentialState);

diff --git a/src/auth/basic/Config.cc b/src/auth/basic/Config.cc
index 321e69907ecd4f5a6840683c8fd193a86a7d1d72..d62e093fc669f483924bc1565a1561bc0f60ff5f 100644 (file)
--- a/src/auth/basic/Config.cc
+++ b/src/auth/basic/Config.cc
@@ -19,7 +19,7 @@
 #include "auth/basic/UserRequest.h"
 #include "auth/Gadgets.h"
 #include "auth/State.h"
-#include "auth/UserNameCache.h"
+#include "auth/CredentialsCache.h"
 #include "cache_cf.h"
 #include "charset.h"
 #include "helper.h"

diff --git a/src/auth/basic/User.cc b/src/auth/basic/User.cc
index c8be6cf9b045600bfa627f4df1475850f95ad7d6..9c732a254ddf814ffc7db64c45ee347cf95b65b2 100644 (file)
--- a/src/auth/basic/User.cc
+++ b/src/auth/basic/User.cc
@@ -9,7 +9,7 @@
 #include "squid.h"
 #include "auth/basic/Config.h"
 #include "auth/basic/User.h"
-#include "auth/UserNameCache.h"
+#include "auth/CredentialsCache.h"
 #include "Debug.h"
 #include "SquidConfig.h"
 #include "SquidTime.h"
@@ -80,10 +80,10 @@ Auth::Basic::User::updateCached(Auth::Basic::User *from)
     }
 }
 
-CbcPointer<Auth::UserNameCache>
+CbcPointer<Auth::CredentialsCache>
 Auth::Basic::User::Cache()
 {
-    static CbcPointer<Auth::UserNameCache> p(new Auth::UserNameCache("basic"));
+    static CbcPointer<Auth::CredentialsCache> p(new Auth::CredentialsCache("basic"));
     return p;
 }
 

diff --git a/src/auth/basic/User.h b/src/auth/basic/User.h
index 97dc193258cea7a237a03f64755e9baf1ac4ca5e..340e7c8f311169b9e6a363aeaf68b5a7675721b3 100644 (file)
--- a/src/auth/basic/User.h
+++ b/src/auth/basic/User.h
@@ -37,7 +37,7 @@ public:
     virtual int32_t ttl() const;
 
     /* Auth::User API */
-    static CbcPointer<Auth::UserNameCache> Cache();
+    static CbcPointer<Auth::CredentialsCache> Cache();
     virtual void addToNameCache() override;
 
     char *passwd;

diff --git a/src/auth/digest/Config.cc b/src/auth/digest/Config.cc
index 90a6ff9eb834478e22101e52841314d2741c8908..eea2f30d3d2b49bce01e92143785b6e5e2dfe815 100644 (file)
--- a/src/auth/digest/Config.cc
+++ b/src/auth/digest/Config.cc
@@ -19,7 +19,7 @@
 #include "auth/digest/UserRequest.h"
 #include "auth/Gadgets.h"
 #include "auth/State.h"
-#include "auth/UserNameCache.h"
+#include "auth/CredentialsCache.h"
 #include "base/LookupTable.h"
 #include "base64.h"
 #include "cache_cf.h"

diff --git a/src/auth/digest/User.cc b/src/auth/digest/User.cc
index a6d85a26df5a9610a95cb7589b00a9b07064e8b8..81788f10fbd680e2bbb226e9ba038b324cc44abe 100644 (file)
--- a/src/auth/digest/User.cc
+++ b/src/auth/digest/User.cc
@@ -9,7 +9,7 @@
 #include "squid.h"
 #include "auth/digest/Config.h"
 #include "auth/digest/User.h"
-#include "auth/UserNameCache.h"
+#include "auth/CredentialsCache.h"
 #include "Debug.h"
 #include "dlink.h"
 #include "SquidConfig.h"
@@ -73,10 +73,10 @@ Auth::Digest::User::currentNonce()
     return nonce;
 }
 
-CbcPointer<Auth::UserNameCache>
+CbcPointer<Auth::CredentialsCache>
 Auth::Digest::User::Cache()
 {
-    static CbcPointer<Auth::UserNameCache> p(new Auth::UserNameCache("digest"));
+    static CbcPointer<Auth::CredentialsCache> p(new Auth::CredentialsCache("digest"));
     return p;
 }
 

diff --git a/src/auth/digest/User.h b/src/auth/digest/User.h
index 0ba69db2f8ff5b752eef5567a2a86cd3b9aea6ee..f823bb5256e0701821e9323b420e490c13a678ff 100644 (file)
--- a/src/auth/digest/User.h
+++ b/src/auth/digest/User.h
@@ -30,7 +30,7 @@ public:
     virtual int32_t ttl() const;
 
     /* Auth::User API */
-    static CbcPointer<Auth::UserNameCache> Cache();
+    static CbcPointer<Auth::CredentialsCache> Cache();
     virtual void addToNameCache() override;
 
     HASH HA1;

diff --git a/src/auth/negotiate/User.cc b/src/auth/negotiate/User.cc
index d97489c77273a6a28e90f224c32fbd5f8d4b618f..3a264d86c9f17287e0dde2c3bdef0e01ea7cc26b 100644 (file)
--- a/src/auth/negotiate/User.cc
+++ b/src/auth/negotiate/User.cc
@@ -9,7 +9,7 @@
 #include "squid.h"
 #include "auth/Config.h"
 #include "auth/negotiate/User.h"
-#include "auth/UserNameCache.h"
+#include "auth/CredentialsCache.h"
 #include "Debug.h"
 
 Auth::Negotiate::User::User(Auth::Config *aConfig, const char *aRequestRealm) :
@@ -28,10 +28,10 @@ Auth::Negotiate::User::ttl() const
     return -1; // Negotiate cannot be cached.
 }
 
-CbcPointer<Auth::UserNameCache>
+CbcPointer<Auth::CredentialsCache>
 Auth::Negotiate::User::Cache()
 {
-    static CbcPointer<Auth::UserNameCache> p(new Auth::UserNameCache("negotiate"));
+    static CbcPointer<Auth::CredentialsCache> p(new Auth::CredentialsCache("negotiate"));
     return p;
 }
 

diff --git a/src/auth/negotiate/User.h b/src/auth/negotiate/User.h
index 91ae262710804057505094ad7ea1f642e73fad7a..3df41a4eb98123c9c473f0aaf26a7661bb5445b2 100644 (file)
--- a/src/auth/negotiate/User.h
+++ b/src/auth/negotiate/User.h
@@ -30,7 +30,7 @@ public:
     virtual int32_t ttl() const;
 
     /* Auth::User API */
-    static CbcPointer<Auth::UserNameCache> Cache();
+    static CbcPointer<Auth::CredentialsCache> Cache();
     virtual void addToNameCache() override;
 
     dlink_list proxy_auth_list;

diff --git a/src/auth/negotiate/UserRequest.cc b/src/auth/negotiate/UserRequest.cc
index b5d418a445d26aebb831f02ec2cd7de808276b56..165d970a7fffdda35125106d79cef14fc0c1f2c2 100644 (file)
--- a/src/auth/negotiate/UserRequest.cc
+++ b/src/auth/negotiate/UserRequest.cc
@@ -13,7 +13,7 @@
 #include "auth/negotiate/UserRequest.h"
 #include "auth/State.h"
 #include "auth/User.h"
-#include "auth/UserNameCache.h"
+#include "auth/CredentialsCache.h"
 #include "client_side.h"
 #include "fatal.h"
 #include "format/Format.h"

diff --git a/src/auth/ntlm/User.cc b/src/auth/ntlm/User.cc
index e1a96fbb30c28dc3992f7a365b1536dde8a69a2b..bd03c96b871b071bf8e2b6d4f3335a2a8c49a0eb 100644 (file)
--- a/src/auth/ntlm/User.cc
+++ b/src/auth/ntlm/User.cc
@@ -9,7 +9,7 @@
 #include "squid.h"
 #include "auth/Config.h"
 #include "auth/ntlm/User.h"
-#include "auth/UserNameCache.h"
+#include "auth/CredentialsCache.h"
 #include "Debug.h"
 
 Auth::Ntlm::User::User(Auth::Config *aConfig, const char *aRequestRealm) :
@@ -28,10 +28,10 @@ Auth::Ntlm::User::ttl() const
     return -1; // NTLM credentials cannot be cached.
 }
 
-CbcPointer<Auth::UserNameCache>
+CbcPointer<Auth::CredentialsCache>
 Auth::Ntlm::User::Cache()
 {
-    static CbcPointer<Auth::UserNameCache> p(new Auth::UserNameCache("ntlm"));
+    static CbcPointer<Auth::CredentialsCache> p(new Auth::CredentialsCache("ntlm"));
     return p;
 }
 

diff --git a/src/auth/ntlm/User.h b/src/auth/ntlm/User.h
index d845a478bb407cb06579d936ee3b7c2475d71699..acb2cd724e235d155812f7508aceb9f2ec1df133 100644 (file)
--- a/src/auth/ntlm/User.h
+++ b/src/auth/ntlm/User.h
@@ -30,7 +30,7 @@ public:
     virtual int32_t ttl() const;
 
     /* Auth::User API */
-    static CbcPointer<Auth::UserNameCache> Cache();
+    static CbcPointer<Auth::CredentialsCache> Cache();
     virtual void addToNameCache() override;
 
     dlink_list proxy_auth_list;

diff --git a/src/auth/ntlm/UserRequest.cc b/src/auth/ntlm/UserRequest.cc
index f223e89f657e5de8af079ef4c649b04b22643c1e..cf732da6eab45e368f328441fc87569bf263207d 100644 (file)
--- a/src/auth/ntlm/UserRequest.cc
+++ b/src/auth/ntlm/UserRequest.cc
@@ -12,7 +12,7 @@
 #include "auth/ntlm/User.h"
 #include "auth/ntlm/UserRequest.h"
 #include "auth/State.h"
-#include "auth/UserNameCache.h"
+#include "auth/CredentialsCache.h"
 #include "cbdata.h"
 #include "client_side.h"
 #include "fatal.h"

diff --git a/src/stat.cc b/src/stat.cc
index cee9b4c70028d7a65e272ccc58decf1b97a69626..13d18c11058ec8d29f0386de7428edad6e240e62 100644 (file)
--- a/src/stat.cc
+++ b/src/stat.cc
@@ -1216,7 +1216,7 @@ statRegisterWithCacheManager(void)
 #if USE_AUTH
     Mgr::RegisterAction("username_cache",
                         "Active Cached Usernames",
-                        Auth::User::UsernameCacheStats, 0, 1);
+                        Auth::User::CredentialsCacheStats, 0, 1);
 #endif
 #if DEBUG_OPENFD
     Mgr::RegisterAction("openfd_objects", "Objects with Swapout files open",

diff --git a/src/tests/stub_libauth.cc b/src/tests/stub_libauth.cc
index 2f27a73a62bb02ece972c3542bbcf38bb255dc9f..dcaad79cc884b8dcc6d61baaeff1c7bbd9d09616 100644 (file)
--- a/src/tests/stub_libauth.cc
+++ b/src/tests/stub_libauth.cc
@@ -42,8 +42,7 @@ Auth::User::~User() STUB_NOP
 void Auth::User::clearIp() STUB
 void Auth::User::removeIp(Ip::Address) STUB
 void Auth::User::addIp(Ip::Address) STUB
-void Auth::User::addToNameCache() STUB
-void Auth::User::UsernameCacheStats(StoreEntry *) STUB
+void Auth::User::CredentialsCacheStats(StoreEntry *) STUB
 
 #include "auth/UserRequest.h"
 char const * Auth::UserRequest::username() const STUB_RETVAL("stub_username")