Merge "Canonical request/response logging"

openstack/transport.py

@@ -253,12 +253,8 @@ class Transport(requests.Session):
         if accept:
             headers.setdefault('Accept', accept)
 
-        self._log_request(method, url, **kwargs)
-
         resp = self._send_request(method, url, redirect, **kwargs)
 
-        self._log_response(resp)
-
         try:
             resp.raise_for_status()
         except requests.RequestException as e:
@@ -287,6 +283,8 @@ class Transport(requests.Session):
         # POSTs as GETs for certain statuses which is not want we want for an
         # API. See: https://en.wikipedia.org/wiki/Post/Redirect/Get
 
+        self._log_request(method, url, **kwargs)
+
         resp = super(Transport, self).request(method, url, **kwargs)
 
         self._log_response(resp)