Refactored actions for authentication, I/O, storedir and pconn modules to Cache Manager

singleton API.

diff --git a/src/AuthConfig.cc b/src/AuthConfig.cc
index 0f2e043853e0ecaf6c83727b2fff5512514b0526..dd250fb027067dd1f5531f140b1eb03bb4c2ccdf 100644 (file)
--- a/src/AuthConfig.cc
+++ b/src/AuthConfig.cc
@@ -77,5 +77,5 @@ AuthConfig::Find(const char *proxy_auth)
 
 /* Default behaviour is to expose nothing */
 void
-AuthConfig::registerWithCacheManager(CacheManager & manager)
+AuthConfig::registerWithCacheManager(void)
 {}

diff --git a/src/AuthConfig.h b/src/AuthConfig.h
index a528ca98e003299628e6f79c51e555a70abb3ace..6f675e3b374e0833ca50b46f991c639ea65f6459 100644 (file)
--- a/src/AuthConfig.h
+++ b/src/AuthConfig.h
@@ -114,7 +114,7 @@ public:
     /** prepare to handle requests */
     virtual void init(AuthConfig *) = 0;
     /** expose any/all statistics to a CacheManager */
-    virtual void registerWithCacheManager(CacheManager & manager);
+    virtual void registerWithCacheManager(void);
     /** parse config options */
     virtual void parse(AuthConfig *, int, char *) = 0;
     /** the http string id */

diff --git a/src/DiskIO/DiskDaemon/DiskDaemonDiskIOModule.cc b/src/DiskIO/DiskDaemon/DiskDaemonDiskIOModule.cc
index a0aad76a16cfa50406a979eb681821e76051866b..4f1848a278638ae8234164054959ad3fa231a2c1 100644 (file)
--- a/src/DiskIO/DiskDaemon/DiskDaemonDiskIOModule.cc
+++ b/src/DiskIO/DiskDaemon/DiskDaemonDiskIOModule.cc
@@ -72,9 +72,9 @@ DiskDaemonDiskIOModule::init()
 }
 
 void
-DiskDaemonDiskIOModule::registerWithCacheManager(CacheManager & manager)
+DiskDaemonDiskIOModule::registerWithCacheManager(void)
 {
-    manager.registerAction("diskd", "DISKD Stats", Stats, 0, 1);
+    CacheManager::GetInstance()->registerAction("diskd", "DISKD Stats", Stats, 0, 1);
 }
 
 void

diff --git a/src/DiskIO/DiskDaemon/DiskDaemonDiskIOModule.h b/src/DiskIO/DiskDaemon/DiskDaemonDiskIOModule.h
index 233812cd434f5288dd2f872340e77fc289ed95ce..a47395ed5a7d85a3f4d5a829bf5a080a1f3c965a 100644 (file)
--- a/src/DiskIO/DiskDaemon/DiskDaemonDiskIOModule.h
+++ b/src/DiskIO/DiskDaemon/DiskDaemonDiskIOModule.h
@@ -43,7 +43,7 @@ public:
     static DiskDaemonDiskIOModule &GetInstance();
     DiskDaemonDiskIOModule();
     virtual void init();
-    virtual void registerWithCacheManager(CacheManager & manager);
+    virtual void registerWithCacheManager(void);
     virtual void shutdown();
     virtual char const *type () const;
     virtual DiskIOStrategy* createStrategy();

diff --git a/src/DiskIO/DiskIOModule.cc b/src/DiskIO/DiskIOModule.cc
index 866ebbcfd0915e25c6f8014018fef532382bda97..71e1c4e3e0bc0b034f7efc90bb8229c8fc631c75 100644 (file)
--- a/src/DiskIO/DiskIOModule.cc
+++ b/src/DiskIO/DiskIOModule.cc
@@ -49,10 +49,10 @@ DiskIOModule::DiskIOModule()
 }
 
 void
-DiskIOModule::RegisterAllModulesWithCacheManager(CacheManager & manager)
+DiskIOModule::RegisterAllModulesWithCacheManager(void)
 {
     for (iterator i = GetModules().begin(); i != GetModules().end(); ++i)
-        (*i)->registerWithCacheManager(manager);
+        (*i)->registerWithCacheManager();
 }
 
 void
@@ -132,5 +132,5 @@ DiskIOModule::FindDefault()
 
 /* disk modules dont export anything by default */
 void
-DiskIOModule::registerWithCacheManager(CacheManager & manager)
+DiskIOModule::registerWithCacheManager(void)
 {}

diff --git a/src/DiskIO/DiskIOModule.h b/src/DiskIO/DiskIOModule.h
index fe3837315f4c8c59b4605b01c1c6b1a0b9be3255..a0b33322f32a3e2227f34952beab87a3f17e6645 100644 (file)
--- a/src/DiskIO/DiskIOModule.h
+++ b/src/DiskIO/DiskIOModule.h
@@ -47,7 +47,7 @@ class DiskIOModule
 {
 
 public:
-    static void RegisterAllModulesWithCacheManager(CacheManager & manager);
+    static void RegisterAllModulesWithCacheManager(void);
     static void SetupAllModules();
     static void ModuleAdd(DiskIOModule &);
     static void FreeAllModules();
@@ -63,7 +63,7 @@ public:
     virtual ~DiskIOModule(){}
 
     virtual void init() = 0;
-    virtual void registerWithCacheManager(CacheManager & manager);
+    virtual void registerWithCacheManager(void);
     virtual void shutdown() = 0;
     virtual DiskIOStrategy *createStrategy() = 0;
 

diff --git a/src/DiskIO/DiskThreads/DiskThreadsDiskIOModule.cc b/src/DiskIO/DiskThreads/DiskThreadsDiskIOModule.cc
index 4fc53ba0899b558f1f6039976b328952d0a86972..2dff881a028cf439160e7af211ed9826ba963951 100644 (file)
--- a/src/DiskIO/DiskThreads/DiskThreadsDiskIOModule.cc
+++ b/src/DiskIO/DiskThreads/DiskThreadsDiskIOModule.cc
@@ -54,9 +54,9 @@ DiskThreadsDiskIOModule::init()
 }
 
 void
-DiskThreadsDiskIOModule::registerWithCacheManager(CacheManager & manager)
+DiskThreadsDiskIOModule::registerWithCacheManager(void)
 {
-    DiskThreadsIOStrategy::Instance.registerWithCacheManager(manager);
+    DiskThreadsIOStrategy::Instance.registerWithCacheManager();
 }
 
 void

diff --git a/src/DiskIO/DiskThreads/DiskThreadsDiskIOModule.h b/src/DiskIO/DiskThreads/DiskThreadsDiskIOModule.h
index 6386e1a4ae0cfa5dc608fcccb19282981d224174..0435841d730173ba61ed8cff0c5f2f37c8b3e09d 100644 (file)
--- a/src/DiskIO/DiskThreads/DiskThreadsDiskIOModule.h
+++ b/src/DiskIO/DiskThreads/DiskThreadsDiskIOModule.h
@@ -43,7 +43,7 @@ public:
     static DiskThreadsDiskIOModule &GetInstance();
     DiskThreadsDiskIOModule();
     virtual void init();
-    virtual void registerWithCacheManager(CacheManager & manager);
+    virtual void registerWithCacheManager(void);
     virtual void shutdown();
     virtual char const *type () const;
     virtual DiskIOStrategy* createStrategy();

diff --git a/src/DiskIO/DiskThreads/DiskThreadsIOStrategy.cc b/src/DiskIO/DiskThreads/DiskThreadsIOStrategy.cc
index 39343f7fcc9ce2bda7f879a22bc98847e0976f47..ab1f101b8aa9ebcef22533db65271e1fdf79ad77 100644 (file)
--- a/src/DiskIO/DiskThreads/DiskThreadsIOStrategy.cc
+++ b/src/DiskIO/DiskThreads/DiskThreadsIOStrategy.cc
@@ -61,10 +61,11 @@ DiskThreadsIOStrategy::init(void)
 }
 
 void
-DiskThreadsIOStrategy::registerWithCacheManager(CacheManager & manager)
+DiskThreadsIOStrategy::registerWithCacheManager(void)
 {
-    manager.registerAction("squidaio_counts", "Async IO Function Counters",
-                           aioStats, 0, 1);
+    CacheManager::GetInstance()->
+        registerAction("squidaio_counts", "Async IO Function Counters",
+                       aioStats, 0, 1);
 }
 
 void

diff --git a/src/DiskIO/DiskThreads/DiskThreadsIOStrategy.h b/src/DiskIO/DiskThreads/DiskThreadsIOStrategy.h
index 2526f63881267437680576dc25b61ee2d706165e..1ff47da0c9414a70407e3b5e140e85b8c5ccfceb 100644 (file)
--- a/src/DiskIO/DiskThreads/DiskThreadsIOStrategy.h
+++ b/src/DiskIO/DiskThreads/DiskThreadsIOStrategy.h
@@ -58,7 +58,7 @@ public:
     virtual int callback();
     virtual void sync();
     virtual void init();
-    virtual void registerWithCacheManager(CacheManager & manager);
+    virtual void registerWithCacheManager(void);
     void done();
     /* Todo: add access limitations */
     bool initialised;

diff --git a/src/SquidString.h b/src/SquidString.h
index b561bcedffee15b164b352b341dae82aa4fa1147..905a4f607e0b593087a90e0b8701b6b60c8c2c75 100644 (file)
--- a/src/SquidString.h
+++ b/src/SquidString.h
@@ -60,7 +60,7 @@ public:
     void add
         (String const *);
 
-    void registerWithCacheManager(CacheManager & manager);
+    void registerWithCacheManager(void);
 
     void remove
         (String const *);

diff --git a/src/StoreFileSystem.cc b/src/StoreFileSystem.cc
index e25e3ff32cef0e07fd54c4024ce09df351bd0f40..db74d3e9926887c9562014e298bdeeebaebdbbd5 100644 (file)
--- a/src/StoreFileSystem.cc
+++ b/src/StoreFileSystem.cc
@@ -43,7 +43,7 @@ void
 StoreFileSystem::RegisterAllFsWithCacheManager(CacheManager & manager)
 {
     for (iterator i = GetFileSystems().begin(); i != GetFileSystems().end(); ++i)
-        (*i)->registerWithCacheManager(manager);
+        (*i)->registerWithCacheManager();
 }
 
 void
@@ -98,5 +98,5 @@ StoreFileSystem::FreeAllFs()
 
 /* no filesystem is required to export statistics */
 void
-StoreFileSystem::registerWithCacheManager(CacheManager & manager)
+StoreFileSystem::registerWithCacheManager(void)
 {}

diff --git a/src/StoreFileSystem.h b/src/StoreFileSystem.h
index ddd87a092807c355877d31f0ece3aff4bbfc562a..c4e532abeb930c20c5ee22f2556b3cd6bc626907 100644 (file)
--- a/src/StoreFileSystem.h
+++ b/src/StoreFileSystem.h
@@ -131,7 +131,7 @@ public:
     virtual char const *type () const = 0;
     virtual SwapDir *createSwapDir() = 0;
     virtual void done() = 0;
-    virtual void registerWithCacheManager(CacheManager & manager);
+    virtual void registerWithCacheManager(void);
     virtual void setup() = 0;
     // Not implemented
     StoreFileSystem(StoreFileSystem const &);

diff --git a/src/String.cc b/src/String.cc
index dab7b7d16add7db765566fe5db6d5f8a45c81b46..102403be7b335cade3a59bde4c2914ea4a9af2d1 100644 (file)
--- a/src/String.cc
+++ b/src/String.cc
@@ -254,9 +254,9 @@ ptrcmp(C const &lhs, C const &rhs)
 }
 
 void
-StringRegistry::registerWithCacheManager(CacheManager & manager)
+StringRegistry::registerWithCacheManager(void)
 {
-    manager.registerAction("strings",
+    CacheManager::GetInstance()->registerAction("strings",
                            "Strings in use in squid", Stat, 0, 1);
 }
 

diff --git a/src/auth/basic/auth_basic.cc b/src/auth/basic/auth_basic.cc
index 60c773a8433f1792f8ee95e54911518d5f3dfcc2..3a693635f880beb3c73764183f31ee2f24d877c8 100644 (file)
--- a/src/auth/basic/auth_basic.cc
+++ b/src/auth/basic/auth_basic.cc
@@ -622,9 +622,10 @@ AuthBasicConfig::init(AuthConfig * scheme)
 }
 
 void
-AuthBasicConfig::registerWithCacheManager(CacheManager & manager)
+AuthBasicConfig::registerWithCacheManager(void)
 {
-    manager.registerAction("basicauthenticator",
+    CacheManager::GetInstance()->
+            registerAction("basicauthenticator",
                            "Basic User Authenticator Stats",
                            authenticateBasicStats, 0, 1);
 }

diff --git a/src/auth/basic/auth_basic.h b/src/auth/basic/auth_basic.h
index c1d335d24de04db0a0eab702f6e14348887ae85c..22c13c580e5c1d1bd6fbb68a62e3e002240a2891 100644 (file)
--- a/src/auth/basic/auth_basic.h
+++ b/src/auth/basic/auth_basic.h
@@ -124,7 +124,7 @@ public:
     virtual void fixHeader(AuthUserRequest *, HttpReply *, http_hdr_type, HttpRequest *);
     virtual void init(AuthConfig *);
     virtual void parse(AuthConfig *, int, char *);
-    virtual void registerWithCacheManager(CacheManager & manager);
+    virtual void registerWithCacheManager(void);
     virtual const char * type() const;
     int authenticateChildren;
     int authenticateConcurrency;

diff --git a/src/auth/digest/auth_digest.cc b/src/auth/digest/auth_digest.cc
index 32f6dbe25f67fa0bd404947b52aa072a80c63c69..bbecfda780090c0d5e71a03ff9b6a3e097447236 100644 (file)
--- a/src/auth/digest/auth_digest.cc
+++ b/src/auth/digest/auth_digest.cc
@@ -888,9 +888,10 @@ AuthDigestConfig::init(AuthConfig * scheme)
 }
 
 void
-AuthDigestConfig::registerWithCacheManager(CacheManager & manager)
+AuthDigestConfig::registerWithCacheManager(void)
 {
-    manager.registerAction("digestauthenticator",
+    CacheManager::GetInstance()->
+            registerAction("digestauthenticator",
                            "Digest User Authenticator Stats",
                            authenticateDigestStats, 0, 1);
 }

diff --git a/src/auth/digest/auth_digest.h b/src/auth/digest/auth_digest.h
index 73b05086f29f781f5f149b24f9ef29e77f075074..90d029973a37f465a3a11fa6ff991d01505f3c1e 100644 (file)
--- a/src/auth/digest/auth_digest.h
+++ b/src/auth/digest/auth_digest.h
@@ -153,7 +153,7 @@ public:
     virtual void fixHeader(AuthUserRequest *, HttpReply *, http_hdr_type, HttpRequest *);
     virtual void init(AuthConfig *);
     virtual void parse(AuthConfig *, int, char *);
-    virtual void registerWithCacheManager(CacheManager & manager);
+    virtual void registerWithCacheManager(void);
     virtual const char * type() const;
     int authenticateChildren;
     char *digestAuthRealm;

diff --git a/src/auth/negotiate/auth_negotiate.cc b/src/auth/negotiate/auth_negotiate.cc
index c85842cd424f76c7a86c15074945ddbbd1b178b2..36ff22f34b203348aeeff26520d658f42d21d97c 100644 (file)
--- a/src/auth/negotiate/auth_negotiate.cc
+++ b/src/auth/negotiate/auth_negotiate.cc
@@ -212,9 +212,10 @@ AuthNegotiateConfig::init(AuthConfig * scheme)
 }
 
 void
-AuthNegotiateConfig::registerWithCacheManager(CacheManager & manager)
+AuthNegotiateConfig::registerWithCacheManager(void)
 {
-    manager.registerAction("negotiateauthenticator",
+    CacheManager::GetInstance()->
+            registerAction("negotiateauthenticator",
                            "Negotiate User Authenticator Stats",
                            authenticateNegotiateStats, 0, 1);
 }

diff --git a/src/auth/negotiate/auth_negotiate.h b/src/auth/negotiate/auth_negotiate.h
index a9bc53d6165431839e40a5598d24d01857e9420c..e46604afa62a6cc358625ea970e3617f18e8381a 100644 (file)
--- a/src/auth/negotiate/auth_negotiate.h
+++ b/src/auth/negotiate/auth_negotiate.h
@@ -125,7 +125,7 @@ public:
     virtual void fixHeader(AuthUserRequest *, HttpReply *, http_hdr_type, HttpRequest *);
     virtual void init(AuthConfig *);
     virtual void parse(AuthConfig *, int, char *);
-    virtual void registerWithCacheManager(CacheManager & manager);
+    virtual void registerWithCacheManager(void);
     virtual const char * type() const;
     int authenticateChildren;
     int keep_alive;

diff --git a/src/auth/ntlm/auth_ntlm.cc b/src/auth/ntlm/auth_ntlm.cc
index c9956f2ddd43e9dd357818fceb042940a355514c..edeb966d0d93f23882d011ef22b71baacaf3f296 100644 (file)
--- a/src/auth/ntlm/auth_ntlm.cc
+++ b/src/auth/ntlm/auth_ntlm.cc
@@ -198,9 +198,10 @@ AuthNTLMConfig::init(AuthConfig * scheme)
 }
 
 void
-AuthNTLMConfig::registerWithCacheManager(CacheManager & manager)
+AuthNTLMConfig::registerWithCacheManager(void)
 {
-    manager.registerAction("ntlmauthenticator",
+    CacheManager::GetInstance()->
+            registerAction("ntlmauthenticator",
                            "NTLM User Authenticator Stats",
                            authenticateNTLMStats, 0, 1);
 }

diff --git a/src/auth/ntlm/auth_ntlm.h b/src/auth/ntlm/auth_ntlm.h
index 38351a77521143edd5ff1c4b7b68acd7df0a6a97..cb76a9c214ee625aab17b4fc7e823faa26dd4e9e 100644 (file)
--- a/src/auth/ntlm/auth_ntlm.h
+++ b/src/auth/ntlm/auth_ntlm.h
@@ -110,7 +110,7 @@ public:
     virtual void fixHeader(AuthUserRequest *, HttpReply *, http_hdr_type, HttpRequest *);
     virtual void init(AuthConfig *);
     virtual void parse(AuthConfig *, int, char *);
-    virtual void registerWithCacheManager(CacheManager & manager);
+    virtual void registerWithCacheManager(void);
     virtual const char * type() const;
     int authenticateChildren;
     int keep_alive;

diff --git a/src/authenticate.cc b/src/authenticate.cc
index 6285e4626a4d55712e4b1a43622e7d8b029b3d43..394ba765a992d8e6a86e3fe41c2e607b234344f8 100644 (file)
--- a/src/authenticate.cc
+++ b/src/authenticate.cc
@@ -90,11 +90,11 @@ authenticateInit(authConfig * config)
 }
 
 void
-authenticateRegisterWithCacheManager(authConfig * config, CacheManager & manager)
+authenticateRegisterWithCacheManager(authConfig * config)
 {
     for (authConfig::iterator i = config->begin(); i != config->end(); ++i) {
         AuthConfig *scheme = *i;
-        scheme->registerWithCacheManager(manager);
+        scheme->registerWithCacheManager();
     }
 }
 

diff --git a/src/authenticate.h b/src/authenticate.h
index 8b0dd00b387b91bc72216b1db22adc8983d73382..6bf8fc9107d92c5f05d6be0a050a67a4d42323b6 100644 (file)
--- a/src/authenticate.h
+++ b/src/authenticate.h
@@ -83,7 +83,7 @@ extern void authenticateAuthUserMerge(AuthUser *, AuthUser *);
 /// \ingroup AuthAPI
 extern void authenticateInit(authConfig *);
 /// \ingroup AuthAPI
-extern void authenticateRegisterWithCacheManager(authConfig * config, CacheManager & manager);
+extern void authenticateRegisterWithCacheManager(authConfig * config);
 /// \ingroup AuthAPI
 extern void authenticateShutdown(void);
 /// \ingroup AuthAPI

diff --git a/src/fs/coss/StoreFScoss.cc b/src/fs/coss/StoreFScoss.cc
index 74c1b5485b1a4abae506fd342f8956b14e89f2b7..6028eb1b0320ddb72cb077612aaeecd0f6dae8db 100644 (file)
--- a/src/fs/coss/StoreFScoss.cc
+++ b/src/fs/coss/StoreFScoss.cc
@@ -84,9 +84,9 @@ StoreFScoss::setup()
 }
 
 void
-StoreFScoss::registerWithCacheManager(CacheManager & manager)
+StoreFScoss::registerWithCacheManager()
 {
-    manager.registerAction("coss", "COSS Stats", Stats, 0, 1);
+    CacheManager::GetInstance()->registerAction("coss", "COSS Stats", Stats, 0, 1);
 }
 
 void

diff --git a/src/fs/coss/StoreFScoss.h b/src/fs/coss/StoreFScoss.h
index 2cbd335d866902ef5d47ae938f1c18028f777b44..69ac6dd98853dfa334eaa60f5f5c695435cdca09 100644 (file)
--- a/src/fs/coss/StoreFScoss.h
+++ b/src/fs/coss/StoreFScoss.h
@@ -86,7 +86,7 @@ public:
     virtual char const *type() const;
     virtual SwapDir *createSwapDir();
     virtual void done();
-    virtual void registerWithCacheManager(CacheManager & manager);
+    virtual void registerWithCacheManager(void);
     virtual void setup();
     /* Not implemented */
     StoreFScoss (StoreFScoss const &);

diff --git a/src/main.cc b/src/main.cc
index e53e98949074b3471eb344f8420f198a067b31b9..d2b78e9abb1979f76dbc9a7e9c8f4cad07db568e 100644 (file)
--- a/src/main.cc
+++ b/src/main.cc
@@ -969,7 +969,7 @@ mainInitialize(void)
         /* register the modules in the cache manager menus */
         accessLogRegisterWithCacheManager();
         asnRegisterWithCacheManager();
-        authenticateRegisterWithCacheManager(&Config.authConfiguration, *manager);
+        authenticateRegisterWithCacheManager(&Config.authConfiguration);
 #if USE_CARP
 
         carpRegisterWithCacheManager(*manager);
@@ -1002,7 +1002,7 @@ mainInitialize(void)
         DelayPools::RegisterWithCacheManager(*manager);
 #endif
 
-        DiskIOModule::RegisterAllModulesWithCacheManager(*manager);
+        DiskIOModule::RegisterAllModulesWithCacheManager();
 #if USE_DNSSERVERS
 
         dnsRegisterWithCacheManager(*manager);
@@ -1021,7 +1021,7 @@ mainInitialize(void)
         ipcacheRegisterWithCacheManager(*manager);
         Mem::RegisterWithCacheManager(*manager);
         netdbRegisterWitHCacheManager(*manager);
-        PconnModule::GetInstance()->registerWithCacheManager(*manager);
+        PconnModule::GetInstance()->registerWithCacheManager();
         redirectRegisterWithCacheManager(*manager);
         refreshRegisterWithCacheManager(*manager);
         statRegisterWithCacheManager(*manager);
@@ -1031,7 +1031,7 @@ mainInitialize(void)
         storeLogRegisterWithCacheManager(*manager);
 #if DEBUGSTRINGS
 
-        StringRegistry::Instance().registerWithCacheManager(*manager);
+        StringRegistry::Instance().registerWithCacheManager();
 #endif
 
 #if    USE_XPROF_STATS

diff --git a/src/pconn.cc b/src/pconn.cc
index 229657da8151bd353954f991f89a940b9b4ad9ea..8895a0a6979a3e74decb31584d0070b58100e2d8 100644 (file)
--- a/src/pconn.cc
+++ b/src/pconn.cc
@@ -340,11 +340,12 @@ PconnModule::GetInstance()
 }
 
 void
-PconnModule::registerWithCacheManager(CacheManager & manager)
+PconnModule::registerWithCacheManager(void)
 {
-    manager.registerAction("pconn",
-                           "Persistent Connection Utilization Histograms",
-                           DumpWrapper, 0, 1);
+    CacheManager::GetInstance()->
+        registerAction("pconn",
+                       "Persistent Connection Utilization Histograms",
+                       DumpWrapper, 0, 1);
 }
 
 void

diff --git a/src/pconn.h b/src/pconn.h
index 1991ef15628c6cc79827132c7b90d9fe502475ca..5e612c5502897439c151181dfd0413fb0c50a18d 100644 (file)
--- a/src/pconn.h
+++ b/src/pconn.h
@@ -107,7 +107,7 @@ public:
     static void DumpWrapper(StoreEntry *e);
 
     PconnModule();
-    void registerWithCacheManager(CacheManager & manager);
+    void registerWithCacheManager(void);
 
     void add(PconnPool *);