Added method to the name for status code based API metrics.

regunathb · Apr 18, 2017 · 7e0787c · 7e0787c
1 parent 6ab9e4f
commit 7e0787c
Showing 1 changed file with 4 additions and 1 deletion.
diff --git a/container/src/main/java/com/flipkart/poseidon/filters/HystrixContextFilter.java b/container/src/main/java/com/flipkart/poseidon/filters/HystrixContextFilter.java
@@ -18,6 +18,7 @@
 
 import com.flipkart.poseidon.api.Configuration;
 import com.flipkart.poseidon.api.HeaderConfiguration;
+import com.flipkart.poseidon.constants.RequestConstants;
 import com.flipkart.poseidon.core.RequestContext;
 import com.flipkart.poseidon.handlers.http.utils.StringUtils;
 import com.flipkart.poseidon.metrics.Metrics;
@@ -70,7 +71,9 @@ private void ingestResponseBasedMetrics(ServletResponse response) {
         // Ingest API response status codes for HttpServletResponse
         if (response instanceof HttpServletResponse && !StringUtils.isNullOrEmpty(RequestContext.get(ENDPOINT_NAME))) {
             String status = (((HttpServletResponse) response).getStatus() / 100) + "XX";
-            Metrics.getRegistry().counter("poseidon.api." + RequestContext.get(ENDPOINT_NAME) + "." + status).inc();
+            Metrics.getRegistry()
+                    .counter("poseidon.api." + RequestContext.get(ENDPOINT_NAME) + "_" + RequestContext.get(RequestConstants.METHOD) + "." + status)
+                    .inc();
         }
     }