Move tls-flags= parser into Security::PeerOptions

It no longer needs to be called by the SSL code, so can be a private
member.

diff --git a/src/security/PeerOptions.cc b/src/security/PeerOptions.cc
index 0903979f12bb812ab9bdffeccd6a485f36ac0998..2b1899fa200b979faf26e4ee9c5e3218df4f7aa9 100644 (file)
--- a/src/security/PeerOptions.cc
+++ b/src/security/PeerOptions.cc
@@ -75,7 +75,7 @@ Security::PeerOptions::parse(const char *token)
             debugs(3, DBG_PARSE_NOTE(1), "WARNING: Overwriting flags=" << sslFlags << " with " << SBuf(token + 6));
         }
         sslFlags = SBuf(token + 6);
-        parsedFlags = Security::ParseFlags(sslFlags);
+        parsedFlags = parseFlags();
     } else if (strncmp(token, "domain=", 7) == 0) {
         sslDomain = SBuf(token + 7);
     } else {
@@ -391,10 +391,13 @@ Security::PeerOptions::parseOptions()
     return op;
 }
 
+/**
+ * Parses the TLS flags squid.conf parameter
+ */
 long
-Security::ParseFlags(const SBuf &flags)
+Security::PeerOptions::parseFlags()
 {
-    if (flags.isEmpty())
+    if (sslFlags.isEmpty())
         return 0;
 
     static struct {
@@ -413,7 +416,7 @@ Security::ParseFlags(const SBuf &flags)
         { SBuf(), 0 }
     };
 
-    ::Parser::Tokenizer tok(flags);
+    ::Parser::Tokenizer tok(sslFlags);
     static const CharacterSet delims("Flag-delimiter", ":,");
 
     long fl = 0;

diff --git a/src/security/PeerOptions.h b/src/security/PeerOptions.h
index 13e5fa8db6766f69e0ebda7b0f122163702993ec..d6f1d75636f65677c4d5c5107208522a2633aae9 100644 (file)
--- a/src/security/PeerOptions.h
+++ b/src/security/PeerOptions.h
@@ -42,6 +42,7 @@ public:
 
 private:
     long parseOptions();
+    long parseFlags();
 
 public:
     SBuf certFile;       ///< path of file containing PEM format X509 certificate
@@ -71,11 +72,6 @@ public:
 /// configuration options for DIRECT server access
 extern PeerOptions ProxyOutgoingConfig;
 
-/**
- * Parses the TLS flags squid.conf parameter
- */
-long ParseFlags(const SBuf &);
-
 } // namespace Security
 
 // parse the tls_outgoing_options directive

diff --git a/src/tests/stub_libsecurity.cc b/src/tests/stub_libsecurity.cc
index e1e80cbd901902a94f8e6a7662cb2ca1a138c86b..8c3a70d1a340c4c936221b9d02e7bea15c57e6bb 100644 (file)
--- a/src/tests/stub_libsecurity.cc
+++ b/src/tests/stub_libsecurity.cc
@@ -23,6 +23,6 @@ Security::ContextPointer Security::PeerOptions::createClientContext(bool) STUB_R
 void Security::PeerOptions::updateTlsVersionLimits() STUB
 void Security::PeerOptions::dumpCfg(Packable*, char const*) const STUB
 long Security::PeerOptions::parseOptions() STUB_RETVAL(0)
+long Security::PeerOptions::parseFlags() STUB_RETVAL(0)
 void parse_securePeerOptions(Security::PeerOptions *) STUB
-long Security::ParseFlags(const SBuf &) STUB_RETVAL(0)