)]}'
{"openstack/transport.py":[{"author":{"_account_id":8257,"name":"Brian Curtin","email":"brian@python.org","username":"briancurtin"},"change_message_id":"ff338843493fcea994cfc2603559156ea4065e39","unresolved":false,"context_lines":[{"line_number":283,"context_line":"        # API. See: https://en.wikipedia.org/wiki/Post/Redirect/Get"},{"line_number":284,"context_line":""},{"line_number":285,"context_line":"        resp \u003d super(Transport, self).request(method, url, **kwargs)"},{"line_number":286,"context_line":""},{"line_number":287,"context_line":"        self._log_response(resp)"},{"line_number":288,"context_line":""},{"line_number":289,"context_line":"        if resp.status_code in self.REDIRECT_STATUSES:"}],"source_content_type":"text/x-python","patch_set":1,"id":"9a80dd14_c977969e","side":"PARENT","line":286,"updated":"2015-03-09 21:42:00.000000000","message":"I think this would be the one to remove if we only want one response logged. However, removing it breaks the case where we get into redirects and would want to have the redirect chain logged. Instead, I think we should move this call into the redirect loop so it only does logs when there\u0027s actually a redirect.","commit_id":"fa1ad8dd55f4f363c9b3c005a56f1876b6b46986"},{"author":{"_account_id":12000,"name":"Ian Cordasco","email":"sigmavirus24@gmail.com","username":"sigmavirus24"},"change_message_id":"1d5cd81ad9846a7792cc790f8be5b417cbf99120","unresolved":false,"context_lines":[{"line_number":283,"context_line":"        # API. See: https://en.wikipedia.org/wiki/Post/Redirect/Get"},{"line_number":284,"context_line":""},{"line_number":285,"context_line":"        resp \u003d super(Transport, self).request(method, url, **kwargs)"},{"line_number":286,"context_line":""},{"line_number":287,"context_line":"        self._log_response(resp)"},{"line_number":288,"context_line":""},{"line_number":289,"context_line":"        if resp.status_code in self.REDIRECT_STATUSES:"}],"source_content_type":"text/x-python","patch_set":1,"id":"9a80dd14_79672526","side":"PARENT","line":286,"in_reply_to":"9a80dd14_c977969e","updated":"2015-03-10 19:05:06.000000000","message":"+1","commit_id":"fa1ad8dd55f4f363c9b3c005a56f1876b6b46986"},{"author":{"_account_id":8257,"name":"Brian Curtin","email":"brian@python.org","username":"briancurtin"},"change_message_id":"ff338843493fcea994cfc2603559156ea4065e39","unresolved":false,"context_lines":[{"line_number":302,"context_line":"                        \"Redirection from %s failed, no location provided\","},{"line_number":303,"context_line":"                        resp.url,"},{"line_number":304,"context_line":"                    )"},{"line_number":305,"context_line":"                else:"},{"line_number":306,"context_line":"                    new_resp \u003d self._send_request("},{"line_number":307,"context_line":"                        method,"},{"line_number":308,"context_line":"                        location,"}],"source_content_type":"text/x-python","patch_set":1,"id":"9a80dd14_89b83e2b","side":"PARENT","line":305,"updated":"2015-03-09 21:42:00.000000000","message":"Moving the call from line 287 to after this one is probably the best move.","commit_id":"fa1ad8dd55f4f363c9b3c005a56f1876b6b46986"},{"author":{"_account_id":12000,"name":"Ian Cordasco","email":"sigmavirus24@gmail.com","username":"sigmavirus24"},"change_message_id":"1d5cd81ad9846a7792cc790f8be5b417cbf99120","unresolved":false,"context_lines":[{"line_number":302,"context_line":"                        \"Redirection from %s failed, no location provided\","},{"line_number":303,"context_line":"                        resp.url,"},{"line_number":304,"context_line":"                    )"},{"line_number":305,"context_line":"                else:"},{"line_number":306,"context_line":"                    new_resp \u003d self._send_request("},{"line_number":307,"context_line":"                        method,"},{"line_number":308,"context_line":"                        location,"}],"source_content_type":"text/x-python","patch_set":1,"id":"9a80dd14_d97571cd","side":"PARENT","line":305,"in_reply_to":"9a80dd14_89b83e2b","updated":"2015-03-10 19:05:06.000000000","message":"+1","commit_id":"fa1ad8dd55f4f363c9b3c005a56f1876b6b46986"},{"author":{"_account_id":8736,"name":"Terry Howe","email":"terrylhowe@gmail.com","username":"terrylhowe"},"change_message_id":"b8bf1f64b3ddcb708582b658c12b62de959a68b9","unresolved":false,"context_lines":[{"line_number":278,"context_line":"        # POSTs as GETs for certain statuses which is not want we want for an"},{"line_number":279,"context_line":"        # API. See: https://en.wikipedia.org/wiki/Post/Redirect/Get"},{"line_number":280,"context_line":""},{"line_number":281,"context_line":"        self._log_request(method, url, **kwargs)"},{"line_number":282,"context_line":""},{"line_number":283,"context_line":"        resp \u003d super(Transport, self).request(method, url, **kwargs)"},{"line_number":284,"context_line":""}],"source_content_type":"text/x-python","patch_set":2,"id":"9a80dd14_ed726fe2","line":281,"updated":"2015-03-10 20:42:01.000000000","message":"In the redirect case, we\u0027ll log the request multiple times.","commit_id":"3c9af53809d859ecec1331928032f9a7277a9357"},{"author":{"_account_id":1112,"name":"Everett Toews","email":"everett.toews@rackspace.com","username":"everett-toews"},"change_message_id":"f636a2bbd209054fa28ddc75738bbf2ecdc522f0","unresolved":false,"context_lines":[{"line_number":278,"context_line":"        # POSTs as GETs for certain statuses which is not want we want for an"},{"line_number":279,"context_line":"        # API. See: https://en.wikipedia.org/wiki/Post/Redirect/Get"},{"line_number":280,"context_line":""},{"line_number":281,"context_line":"        self._log_request(method, url, **kwargs)"},{"line_number":282,"context_line":""},{"line_number":283,"context_line":"        resp \u003d super(Transport, self).request(method, url, **kwargs)"},{"line_number":284,"context_line":""}],"source_content_type":"text/x-python","patch_set":2,"id":"9a80dd14_c3e8f318","line":281,"in_reply_to":"9a80dd14_ed726fe2","updated":"2015-03-11 15:54:51.000000000","message":"That\u0027s what we want. A new request is made to a new location and we need to log that so the logs match up with what\u0027s really happening.","commit_id":"3c9af53809d859ecec1331928032f9a7277a9357"}]}
