)]}'
{"zaqar/tests/unit/transport/websocket/v2/test_auth.py":[{"author":{"_account_id":6484,"name":"Feilong Wang","email":"hustemb@gmail.com","username":"flwang"},"change_message_id":"273d57b9e9d685804806dd6f3105d81005bf5f67","unresolved":false,"context_lines":[{"line_number":145,"context_line":"        # try to authenticate again, \"onMessage\" should not return 403 because"},{"line_number":146,"context_line":"        # that the _auth_app was cleaned after auth failure."},{"line_number":147,"context_line":"        self.protocol.onMessage(req, False)"},{"line_number":148,"context_line":""},{"line_number":149,"context_line":"    @ddt.data(True, False)"},{"line_number":150,"context_line":"    def test_auth_response_serialization_format(self, in_binary):"},{"line_number":151,"context_line":"        dumps, loads, create_req \u003d test_utils.get_pack_tools(binary\u003din_binary)"}],"source_content_type":"text/x-python","patch_set":3,"id":"3aaa91ec_e429178b","line":148,"updated":"2016-06-22 17:29:14.000000000","message":"As we discussed in IRC, it would be nice if you can add a check for the response. Thanks.","commit_id":"54142233d9dbd8783c73d2c60a5324e2a305eaa3"}],"zaqar/transport/websocket/protocol.py":[{"author":{"_account_id":6484,"name":"Feilong Wang","email":"hustemb@gmail.com","username":"flwang"},"change_message_id":"1e78a3719ba6e3bc17bde28a290d67e2b3856e6e","unresolved":false,"context_lines":[{"line_number":171,"context_line":"        code \u003d int(status.split()[0])"},{"line_number":172,"context_line":"        req \u003d self._handler.create_request({\u0027action\u0027: \u0027authenticate\u0027})"},{"line_number":173,"context_line":"        if code !\u003d 200:"},{"line_number":174,"context_line":"            # we should clean up the _auth_app after the authentication"},{"line_number":175,"context_line":"            # failure. So that we could authenticate again."},{"line_number":176,"context_line":"            self._auth_app \u003d None"},{"line_number":177,"context_line":"            body \u003d {\u0027error\u0027: \u0027Authentication failed.\u0027}"},{"line_number":178,"context_line":"            resp \u003d self._handler.create_response(code, body, req)"}],"source_content_type":"text/x-python","patch_set":1,"id":"7aa08908_e1617706","line":175,"range":{"start_line":174,"start_character":12,"end_line":175,"end_character":59},"updated":"2016-06-08 17:10:21.000000000","message":"(nit): can you reword it as below?\n\n# NOTE(wangxiyuan): _auth_app should be cleaned up the after the authentication failure so that the client can be authenticated again.","commit_id":"1b86838c4896884d2c23e35c269f850da92eb84a"}]}