bug fix: request field of http_metrics is being correctly populated now

Author: mhamzak008

opsgenie_sdk/rest.py

@@ -238,6 +238,8 @@ def request(self, method, url, query_params=None, headers=None,
             logger.debug("response body: %s", r.data)
 
         data = self.decodeResponse(r.data)
+        http_metrics_request = [('method', method), ('url', url), ('query_params', query_params), ('headers', headers),
+                           ('body', body), ('post_params', post_params)]
         if "message" not in data:
             self.http_metric.build_metric(transaction_id=self.configuration.metrics_transaction_id,
                                           duration=data["took"],
@@ -246,7 +248,7 @@ def request(self, method, url, query_params=None, headers=None,
                                           error=False,
                                           status=r.status,
                                           status_code=r.status,
-                                          request=query_params)
+                                          request=http_metrics_request)
         else:
             self.http_metric.build_metric(transaction_id=self.configuration.metrics_transaction_id,
                                           duration=data["took"],
@@ -255,7 +257,7 @@ def request(self, method, url, query_params=None, headers=None,
                                           error=True,
                                           status=r.status,
                                           status_code=r.status,
-                                          request=query_params)
+                                          request=http_metrics_request)
 
         should_retry = self.__checkHttpCode__(r.status)
         if should_retry:

templates/rest.mustache

@@ -229,6 +229,8 @@ class RESTClientObject(object):
             logger.debug("response body: %s", r.data)
 
         data = self.decodeResponse(r.data)
+        http_metrics_request = [('method', method), ('url', url), ('query_params', query_params), ('headers', headers),
+                           ('body', body), ('post_params', post_params)]
         if "message" not in data:
             self.http_metric.build_metric(transaction_id=self.configuration.metrics_transaction_id,
                                           duration=data["took"],
@@ -237,7 +239,7 @@ class RESTClientObject(object):
                                           error=False,
                                           status=r.status,
                                           status_code=r.status,
-                                          request=query_params)
+                                          request=http_metrics_request)
         else:
             self.http_metric.build_metric(transaction_id=self.configuration.metrics_transaction_id,
                                           duration=data["took"],
@@ -246,7 +248,7 @@ class RESTClientObject(object):
                                           error=True,
                                           status=r.status,
                                           status_code=r.status,
-                                          request=query_params)
+                                          request=http_metrics_request)
 
         should_retry = self.__checkHttpCode__(r.status)
         if should_retry: