)]}'
{"nova/console/websocketproxy.py":[{"author":{"_account_id":7634,"name":"Takashi Natsume","email":"takanattie@gmail.com","username":"natsumet"},"change_message_id":"b9bfbffb327c4f2e8cd87ee6e20ae65977d3192c","unresolved":false,"context_lines":[{"line_number":71,"context_line":"        new_hdr \u003d \"; \".join(c for c in cookie_hdr.split(\u0027; \u0027) if good(c))"},{"line_number":72,"context_line":"        if new_hdr !\u003d cookie_hdr:"},{"line_number":73,"context_line":"            LOG.debug(\"Changed cookie header from [%s] to [%s]\""},{"line_number":74,"context_line":"                      % (cookie_hdr, new_hdr))"},{"line_number":75,"context_line":"        return new_hdr"},{"line_number":76,"context_line":""},{"line_number":77,"context_line":"    def _get_token(self, parse):"}],"source_content_type":"text/x-python","patch_set":3,"id":"9ad45d7e_41e77612","line":74,"range":{"start_line":74,"start_character":22,"end_line":74,"end_character":23},"updated":"2016-08-08 10:43:28.000000000","message":"Use \u0027,\u0027 instead of \u0027%\u0027.\nSee http://docs.openstack.org/developer/oslo.i18n/guidelines.html#adding-variables-to-log-messages","commit_id":"ed757c310df041410822289e151868c1c4a38a82"},{"author":{"_account_id":22785,"name":"Tanvir Talukder","email":"tanvirt16@gmail.com","username":"tanvirt16"},"change_message_id":"2a9b3f21dca0a7f998196234a4616528620c9282","unresolved":false,"context_lines":[{"line_number":69,"context_line":"            return not (\u0027[\u0027 in val or \u0027]\u0027 in val)"},{"line_number":70,"context_line":""},{"line_number":71,"context_line":"        new_hdr \u003d \"; \".join(c for c in cookie_hdr.split(\u0027; \u0027) if good(c))"},{"line_number":72,"context_line":"        if new_hdr !\u003d cookie_hdr:"},{"line_number":73,"context_line":"            LOG.debug(\"Changed cookie header from [%s] to [%s]\","},{"line_number":74,"context_line":"                      cookie_hdr, new_hdr)"},{"line_number":75,"context_line":"        return new_hdr"}],"source_content_type":"text/x-python","patch_set":4,"id":"9ad45d7e_3173544f","line":72,"range":{"start_line":72,"start_character":11,"end_line":72,"end_character":32},"updated":"2016-08-09 19:34:14.000000000","message":"I recommend renaming these to \"new_header\" and \"cookie_header\". \"hdr\" makes sense in context, but I was originally confused about what it stood for (could have been \"high dynamic range\").","commit_id":"1f1ea13695058d9215005a5b8e6eeac6f76005a6"},{"author":{"_account_id":22785,"name":"Tanvir Talukder","email":"tanvirt16@gmail.com","username":"tanvirt16"},"change_message_id":"2a9b3f21dca0a7f998196234a4616528620c9282","unresolved":false,"context_lines":[{"line_number":75,"context_line":"        return new_hdr"},{"line_number":76,"context_line":""},{"line_number":77,"context_line":"    def _get_token(self, parse):"},{"line_number":78,"context_line":"        hcookie \u003d self.headers.getheader(\u0027cookie\u0027)"},{"line_number":79,"context_line":"        if (hcookie and sys.version_info \u003e\u003d (2, 7, 9)"},{"line_number":80,"context_line":"                and sys.version_info \u003c (2, 7, 10)):"},{"line_number":81,"context_line":"            hcookie \u003d self._strip_cookie_brackets(hcookie)"}],"source_content_type":"text/x-python","patch_set":4,"id":"9ad45d7e_b1ebc4af","line":78,"range":{"start_line":78,"start_character":8,"end_line":78,"end_character":15},"updated":"2016-08-09 19:34:14.000000000","message":"Could rename this to \"cookie_header\" for more clarification.","commit_id":"1f1ea13695058d9215005a5b8e6eeac6f76005a6"}]}
