)]}'
{"cinderclient/client.py":[{"author":{"_account_id":33431,"name":"Fábio Oliveira","email":"fabioaurelio1269@gmail.com","username":"fabiooliveira1"},"change_message_id":"e63a8c3e3cbcc555c1f45779341e2219880e773b","unresolved":true,"context_lines":[{"line_number":360,"context_line":"        self._logger.debug(\"\\nREQ: %s\\n\" % \"\".join(string_parts))"},{"line_number":361,"context_line":""},{"line_number":362,"context_line":"    def http_log_resp(self, resp):"},{"line_number":363,"context_line":"        if not self.http_log_debug:"},{"line_number":364,"context_line":"            return"},{"line_number":365,"context_line":"        self._logger.debug("},{"line_number":366,"context_line":"            \"RESP: [%s] %s\\nRESP BODY: %s\\n\","},{"line_number":367,"context_line":"            resp.status_code,"},{"line_number":368,"context_line":"            resp.headers,"}],"source_content_type":"text/x-python","patch_set":6,"id":"144045e2_3fd75c67","line":365,"range":{"start_line":363,"start_character":8,"end_line":365,"end_character":27},"updated":"2021-08-25 15:39:05.000000000","message":"since we are taking a look at this, wouldn\u0027t it be better if we put the log inside if with the desired condition instead of forcing an empty return?","commit_id":"0439a44707788f9400e5aba716001f4e4bb55d81"},{"author":{"_account_id":5314,"name":"Brian Rosmaita","email":"rosmaita.fossdev@gmail.com","username":"brian-rosmaita"},"change_message_id":"057a6ba5050991cc5e46f7b8f99d586f5bb7ad06","unresolved":true,"context_lines":[{"line_number":360,"context_line":"        self._logger.debug(\"\\nREQ: %s\\n\" % \"\".join(string_parts))"},{"line_number":361,"context_line":""},{"line_number":362,"context_line":"    def http_log_resp(self, resp):"},{"line_number":363,"context_line":"        if not self.http_log_debug:"},{"line_number":364,"context_line":"            return"},{"line_number":365,"context_line":"        self._logger.debug("},{"line_number":366,"context_line":"            \"RESP: [%s] %s\\nRESP BODY: %s\\n\","},{"line_number":367,"context_line":"            resp.status_code,"},{"line_number":368,"context_line":"            resp.headers,"}],"source_content_type":"text/x-python","patch_set":6,"id":"a960d0e2_752b2a1e","line":365,"range":{"start_line":363,"start_character":8,"end_line":365,"end_character":27},"in_reply_to":"144045e2_3fd75c67","updated":"2021-08-25 20:03:01.000000000","message":"Just want to acknowledge this.  I can go either way.  We tend to use the check-and-get-out-of-here-immediately pattern a lot, so it\u0027s not unusual to see something like this.","commit_id":"0439a44707788f9400e5aba716001f4e4bb55d81"},{"author":{"_account_id":33431,"name":"Fábio Oliveira","email":"fabioaurelio1269@gmail.com","username":"fabiooliveira1"},"change_message_id":"b9c9c0a013368eceac6aecef729e6e3db69d6490","unresolved":false,"context_lines":[{"line_number":360,"context_line":"        self._logger.debug(\"\\nREQ: %s\\n\" % \"\".join(string_parts))"},{"line_number":361,"context_line":""},{"line_number":362,"context_line":"    def http_log_resp(self, resp):"},{"line_number":363,"context_line":"        if not self.http_log_debug:"},{"line_number":364,"context_line":"            return"},{"line_number":365,"context_line":"        self._logger.debug("},{"line_number":366,"context_line":"            \"RESP: [%s] %s\\nRESP BODY: %s\\n\","},{"line_number":367,"context_line":"            resp.status_code,"},{"line_number":368,"context_line":"            resp.headers,"}],"source_content_type":"text/x-python","patch_set":6,"id":"ee6344a5_f8785015","line":365,"range":{"start_line":363,"start_character":8,"end_line":365,"end_character":27},"in_reply_to":"a960d0e2_752b2a1e","updated":"2021-08-25 20:18:29.000000000","message":"Ack. Thank you.","commit_id":"0439a44707788f9400e5aba716001f4e4bb55d81"}]}