)]}'
{"jenkins/__init__.py":[{"author":{"_account_id":1054,"name":"Darragh Bailey","email":"daragh.bailey@gmail.com","username":"dbailey-k"},"change_message_id":"dbab75cd7992c5e056ffae4b41e35f3ab237b189","unresolved":false,"context_lines":[{"line_number":338,"context_line":"        by the fact that a redirected POST is transformed into a GET."},{"line_number":339,"context_line":"        \u0027\u0027\u0027"},{"line_number":340,"context_line":"        try:"},{"line_number":341,"context_line":"            r \u003d requests.head(self.server,"},{"line_number":342,"context_line":"                              allow_redirects\u003dFalse,"},{"line_number":343,"context_line":"                              timeout\u003dself.timeout)"},{"line_number":344,"context_line":"            if r.status_code in [300, 301, 302, 303]:"},{"line_number":345,"context_line":"                new_url \u003d r.headers[\u0027Location\u0027]"},{"line_number":346,"context_line":"                warnings.warn("}],"source_content_type":"text/x-python","patch_set":7,"id":"5f7c97a3_61074a83","line":343,"range":{"start_line":341,"start_character":16,"end_line":343,"end_character":51},"updated":"2018-06-19 18:21:04.000000000","message":"should this still use the session object to make the call?","commit_id":"a740dee0e3ef49b8d17da76febd933b4f72134b9"},{"author":{"_account_id":24162,"name":"Sorin Sbârnea","display_name":"zbr","email":"ssbarnea@redhat.com","username":"ssbarnea","status":"do not feed the troll 🥕"},"change_message_id":"aa1a65ebe9c5c8ce9ab9133dce15b749edc045e7","unresolved":false,"context_lines":[{"line_number":338,"context_line":"        by the fact that a redirected POST is transformed into a GET."},{"line_number":339,"context_line":"        \u0027\u0027\u0027"},{"line_number":340,"context_line":"        try:"},{"line_number":341,"context_line":"            r \u003d requests.head(self.server,"},{"line_number":342,"context_line":"                              allow_redirects\u003dFalse,"},{"line_number":343,"context_line":"                              timeout\u003dself.timeout)"},{"line_number":344,"context_line":"            if r.status_code in [300, 301, 302, 303]:"},{"line_number":345,"context_line":"                new_url \u003d r.headers[\u0027Location\u0027]"},{"line_number":346,"context_line":"                warnings.warn("}],"source_content_type":"text/x-python","patch_set":7,"id":"5f7c97a3_788f3476","line":343,"range":{"start_line":341,"start_character":16,"end_line":343,"end_character":51},"in_reply_to":"5f7c97a3_61074a83","updated":"2018-07-12 09:18:57.000000000","message":"Done","commit_id":"a740dee0e3ef49b8d17da76febd933b4f72134b9"},{"author":{"_account_id":1054,"name":"Darragh Bailey","email":"daragh.bailey@gmail.com","username":"dbailey-k"},"change_message_id":"dbab75cd7992c5e056ffae4b41e35f3ab237b189","unresolved":false,"context_lines":[{"line_number":347,"context_line":"                    \"Redirection from %s to %s detected, you may want to update your frontend url.\" % ("},{"line_number":348,"context_line":"                        self.server, new_url))"},{"line_number":349,"context_line":"                self.server \u003d new_url"},{"line_number":350,"context_line":"        except (JenkinsException, ConnectionError):"},{"line_number":351,"context_line":"            # for backwards compatibility on:"},{"line_number":352,"context_line":"            # - test suite"},{"line_number":353,"context_line":"            # - clients that may not expect a HTTP exception to occur so soon"}],"source_content_type":"text/x-python","patch_set":7,"id":"5f7c97a3_4129eeef","line":350,"range":{"start_line":350,"start_character":16,"end_line":350,"end_character":32},"updated":"2018-06-19 18:21:04.000000000","message":"Is this possible? or rather should it be? I\u0027m guessing this is from mock side_effects which suggests the \u0027_resolve_api\u0027 method should probably be mocked out in the test code except for specific tests checking for this behaviour.","commit_id":"a740dee0e3ef49b8d17da76febd933b4f72134b9"},{"author":{"_account_id":24162,"name":"Sorin Sbârnea","display_name":"zbr","email":"ssbarnea@redhat.com","username":"ssbarnea","status":"do not feed the troll 🥕"},"change_message_id":"aa1a65ebe9c5c8ce9ab9133dce15b749edc045e7","unresolved":false,"context_lines":[{"line_number":347,"context_line":"                    \"Redirection from %s to %s detected, you may want to update your frontend url.\" % ("},{"line_number":348,"context_line":"                        self.server, new_url))"},{"line_number":349,"context_line":"                self.server \u003d new_url"},{"line_number":350,"context_line":"        except (JenkinsException, ConnectionError):"},{"line_number":351,"context_line":"            # for backwards compatibility on:"},{"line_number":352,"context_line":"            # - test suite"},{"line_number":353,"context_line":"            # - clients that may not expect a HTTP exception to occur so soon"}],"source_content_type":"text/x-python","patch_set":7,"id":"5f7c97a3_d8e3e839","line":350,"range":{"start_line":350,"start_character":16,"end_line":350,"end_character":32},"in_reply_to":"5f7c97a3_4129eeef","updated":"2018-07-12 09:18:57.000000000","message":"Done","commit_id":"a740dee0e3ef49b8d17da76febd933b4f72134b9"},{"author":{"_account_id":1054,"name":"Darragh Bailey","email":"daragh.bailey@gmail.com","username":"dbailey-k"},"change_message_id":"dbab75cd7992c5e056ffae4b41e35f3ab237b189","unresolved":false,"context_lines":[{"line_number":352,"context_line":"            # - test suite"},{"line_number":353,"context_line":"            # - clients that may not expect a HTTP exception to occur so soon"},{"line_number":354,"context_line":"            pass"},{"line_number":355,"context_line":"        finally:"},{"line_number":356,"context_line":"            # tests expect crumb ping-pong to happen on first api call."},{"line_number":357,"context_line":"            self.crumb \u003d None"},{"line_number":358,"context_line":""},{"line_number":359,"context_line":"    def _get_encoded_params(self, params):"},{"line_number":360,"context_line":"        for k, v in params.items():"}],"source_content_type":"text/x-python","patch_set":7,"id":"5f7c97a3_0133f663","line":357,"range":{"start_line":355,"start_character":8,"end_line":357,"end_character":29},"updated":"2018-06-19 18:21:04.000000000","message":"I think that\u0027s because we\u0027re not mocking this method out by default so anything with a side_effect/requests_mock has an extra call that it\u0027s not handling by default.","commit_id":"a740dee0e3ef49b8d17da76febd933b4f72134b9"},{"author":{"_account_id":24162,"name":"Sorin Sbârnea","display_name":"zbr","email":"ssbarnea@redhat.com","username":"ssbarnea","status":"do not feed the troll 🥕"},"change_message_id":"aa1a65ebe9c5c8ce9ab9133dce15b749edc045e7","unresolved":false,"context_lines":[{"line_number":352,"context_line":"            # - test suite"},{"line_number":353,"context_line":"            # - clients that may not expect a HTTP exception to occur so soon"},{"line_number":354,"context_line":"            pass"},{"line_number":355,"context_line":"        finally:"},{"line_number":356,"context_line":"            # tests expect crumb ping-pong to happen on first api call."},{"line_number":357,"context_line":"            self.crumb \u003d None"},{"line_number":358,"context_line":""},{"line_number":359,"context_line":"    def _get_encoded_params(self, params):"},{"line_number":360,"context_line":"        for k, v in params.items():"}],"source_content_type":"text/x-python","patch_set":7,"id":"5f7c97a3_f8e82416","line":357,"range":{"start_line":355,"start_character":8,"end_line":357,"end_character":29},"in_reply_to":"5f7c97a3_0133f663","updated":"2018-07-12 09:18:57.000000000","message":"Done","commit_id":"a740dee0e3ef49b8d17da76febd933b4f72134b9"},{"author":{"_account_id":1054,"name":"Darragh Bailey","email":"daragh.bailey@gmail.com","username":"dbailey-k"},"change_message_id":"3987a5633042424b38db88b12d56045e70e00479","unresolved":false,"context_lines":[{"line_number":1330,"context_line":"            \u0027POST\u0027, url, data\u003ddata))"},{"line_number":1331,"context_line":""},{"line_number":1332,"context_line":"        if not result.endswith(magic_str):"},{"line_number":1333,"context_line":"            logging.error(result)"},{"line_number":1334,"context_line":"            raise JenkinsException(result)"},{"line_number":1335,"context_line":""},{"line_number":1336,"context_line":"        return result[:result.rfind(\u0027\\n\u0027)]"}],"source_content_type":"text/x-python","patch_set":9,"id":"5f7c97a3_c368a789","line":1333,"range":{"start_line":1333,"start_character":12,"end_line":1333,"end_character":33},"updated":"2018-07-14 14:02:35.000000000","message":"did you meant to make this change here?","commit_id":"47a43a71945957e235657e498af6dd84064e14e6"},{"author":{"_account_id":24162,"name":"Sorin Sbârnea","display_name":"zbr","email":"ssbarnea@redhat.com","username":"ssbarnea","status":"do not feed the troll 🥕"},"change_message_id":"27ca2f7b505c85cdc068b6565099e66adaaca04c","unresolved":false,"context_lines":[{"line_number":1330,"context_line":"            \u0027POST\u0027, url, data\u003ddata))"},{"line_number":1331,"context_line":""},{"line_number":1332,"context_line":"        if not result.endswith(magic_str):"},{"line_number":1333,"context_line":"            logging.error(result)"},{"line_number":1334,"context_line":"            raise JenkinsException(result)"},{"line_number":1335,"context_line":""},{"line_number":1336,"context_line":"        return result[:result.rfind(\u0027\\n\u0027)]"}],"source_content_type":"text/x-python","patch_set":9,"id":"5f7c97a3_98f0c294","line":1333,"range":{"start_line":1333,"start_character":12,"end_line":1333,"end_character":33},"in_reply_to":"5f7c97a3_c368a789","updated":"2018-07-16 16:56:07.000000000","message":"Done","commit_id":"47a43a71945957e235657e498af6dd84064e14e6"}]}