(trafficserver) branch master updated: rate_limit plugin: Decouple metrics and queue config (#11122)

[masaori]

This is an automated email from the ASF dual-hosted git repository.

masaori pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/trafficserver.git


The following commit(s) were added to refs/heads/master by this push:
     new 0d84ded118 rate_limit plugin: Decouple metrics and queue config 
(#11122)
0d84ded118 is described below

commit 0d84ded1181603e082141c7ba3b5d403e8e38025
Author: Masaori Koshiba <masa...@apache.org>
AuthorDate: Tue Mar 5 13:27:24 2024 +0900

    rate_limit plugin: Decouple metrics and queue config (#11122)
---
 plugins/experimental/rate_limit/limiter.h | 14 +++++++-------
 1 file changed, 7 insertions(+), 7 deletions(-)

diff --git a/plugins/experimental/rate_limit/limiter.h 
b/plugins/experimental/rate_limit/limiter.h
index 6f92f7e97c..73757450f4 100644
--- a/plugins/experimental/rate_limit/limiter.h
+++ b/plugins/experimental/rate_limit/limiter.h
@@ -95,16 +95,16 @@ public:
       if (queue["max_age"]) {
         _max_age = std::chrono::seconds(queue["max_age"].as<uint32_t>());
       }
+    }
 
-      const YAML::Node &metrics = node["metrics"];
+    const YAML::Node &metrics = node["metrics"];
 
-      if (metrics) {
-        std::string prefix = metrics["prefix"] ? 
metrics["prefix"].as<std::string>() : RATE_LIMITER_METRIC_PREFIX;
-        std::string tag    = metrics["tag"] ? metrics["tag"].as<std::string>() 
: name();
+    if (metrics) {
+      std::string prefix = metrics["prefix"] ? 
metrics["prefix"].as<std::string>() : RATE_LIMITER_METRIC_PREFIX;
+      std::string tag    = metrics["tag"] ? metrics["tag"].as<std::string>() : 
name();
 
-        Dbg(dbg_ctl, "Metrics for selector rule: %s(%s, %s)", name().c_str(), 
prefix.c_str(), tag.c_str());
-        initializeMetrics(RATE_LIMITER_TYPE_SNI, prefix, tag);
-      }
+      Dbg(dbg_ctl, "Metrics for selector rule: %s(%s, %s)", name().c_str(), 
prefix.c_str(), tag.c_str());
+      initializeMetrics(RATE_LIMITER_TYPE_SNI, prefix, tag);
     }
 
     return true;