)]}'
{"keystone/tests/test_v3.py":[{"author":{"_account_id":6486,"name":"Brant Knudson","email":"blk@acm.org","username":"blk-u"},"change_message_id":"070d9728532404558e1f18ac7b05b447dbd60d4f","unresolved":false,"context_lines":[{"line_number":1038,"context_line":"            auth_data[\u0027scope\u0027] \u003d self.build_auth_scope(**kwargs)"},{"line_number":1039,"context_line":"        return {\u0027auth\u0027: auth_data}"},{"line_number":1040,"context_line":""},{"line_number":1041,"context_line":"    def build_external_auth_request(self, user, auth_data\u003dNone):"},{"line_number":1042,"context_line":"        context \u003d {\u0027REMOTE_USER\u0027: user}"},{"line_number":1043,"context_line":"        if not auth_data:"},{"line_number":1044,"context_line":"            auth_data \u003d self.build_authentication_request()[\u0027auth\u0027]"}],"source_content_type":"text/x-python","patch_set":2,"id":"AAAATn%2F%2F0BM%3D","line":1041,"updated":"2013-10-28 16:31:38.000000000","message":"suggest change user to remote_user_name","commit_id":"4b179e88266bb8ae04d96179f3b9fc1aef544a01"},{"author":{"_account_id":91,"name":"Alvaro","email":"aloga@ifca.unican.es","username":"aloga"},"change_message_id":"a2ee2bce254725f11c689836d6413957fd884e3b","unresolved":false,"context_lines":[{"line_number":1038,"context_line":"            auth_data[\u0027scope\u0027] \u003d self.build_auth_scope(**kwargs)"},{"line_number":1039,"context_line":"        return {\u0027auth\u0027: auth_data}"},{"line_number":1040,"context_line":""},{"line_number":1041,"context_line":"    def build_external_auth_request(self, user, auth_data\u003dNone):"},{"line_number":1042,"context_line":"        context \u003d {\u0027REMOTE_USER\u0027: user}"},{"line_number":1043,"context_line":"        if not auth_data:"},{"line_number":1044,"context_line":"            auth_data \u003d self.build_authentication_request()[\u0027auth\u0027]"}],"source_content_type":"text/x-python","patch_set":2,"id":"AAAATn%2F%2FxaY%3D","line":1041,"in_reply_to":"AAAATn%2F%2F0BM%3D","updated":"2013-10-29 09:47:14.000000000","message":"Done","commit_id":"4b179e88266bb8ae04d96179f3b9fc1aef544a01"},{"author":{"_account_id":6486,"name":"Brant Knudson","email":"blk@acm.org","username":"blk-u"},"change_message_id":"070d9728532404558e1f18ac7b05b447dbd60d4f","unresolved":false,"context_lines":[{"line_number":1042,"context_line":"        context \u003d {\u0027REMOTE_USER\u0027: user}"},{"line_number":1043,"context_line":"        if not auth_data:"},{"line_number":1044,"context_line":"            auth_data \u003d self.build_authentication_request()[\u0027auth\u0027]"},{"line_number":1045,"context_line":"        auth_info \u003d auth.controllers.AuthInfo(None, auth_data)"},{"line_number":1046,"context_line":"        auth_context \u003d {\u0027extras\u0027: {}, \u0027method_names\u0027: []}"},{"line_number":1047,"context_line":"        return context, auth_info, auth_context"},{"line_number":1048,"context_line":""}],"source_content_type":"text/x-python","patch_set":2,"id":"AAAATn%2F%2F0BY%3D","line":1045,"updated":"2013-10-28 16:31:38.000000000","message":"give a name to the None parameter so that reviewers know what it is..\n\n no_context \u003d None\n auth_info \u003d auth.controllers.AuthInfo(no_context, auth_data)\n\nAlternatively, could pass in context instead. Looks like it wouldn\u0027t have any effect.","commit_id":"4b179e88266bb8ae04d96179f3b9fc1aef544a01"},{"author":{"_account_id":91,"name":"Alvaro","email":"aloga@ifca.unican.es","username":"aloga"},"change_message_id":"a2ee2bce254725f11c689836d6413957fd884e3b","unresolved":false,"context_lines":[{"line_number":1042,"context_line":"        context \u003d {\u0027REMOTE_USER\u0027: user}"},{"line_number":1043,"context_line":"        if not auth_data:"},{"line_number":1044,"context_line":"            auth_data \u003d self.build_authentication_request()[\u0027auth\u0027]"},{"line_number":1045,"context_line":"        auth_info \u003d auth.controllers.AuthInfo(None, auth_data)"},{"line_number":1046,"context_line":"        auth_context \u003d {\u0027extras\u0027: {}, \u0027method_names\u0027: []}"},{"line_number":1047,"context_line":"        return context, auth_info, auth_context"},{"line_number":1048,"context_line":""}],"source_content_type":"text/x-python","patch_set":2,"id":"AAAATn%2F%2FxaU%3D","line":1045,"in_reply_to":"AAAATn%2F%2F0BY%3D","updated":"2013-10-29 09:47:14.000000000","message":"Done.\n\nI prefer the former, otherwise it seems that a non empty context might be needed.","commit_id":"4b179e88266bb8ae04d96179f3b9fc1aef544a01"},{"author":{"_account_id":4,"name":"Dolph Mathews","email":"dolph.mathews@gmail.com","username":"dolph"},"change_message_id":"fe0c242d410cc2d9308d30dc84a5bbb5fceed5af","unresolved":false,"context_lines":[{"line_number":1044,"context_line":"            auth_data \u003d self.build_authentication_request()[\u0027auth\u0027]"},{"line_number":1045,"context_line":"        no_context \u003d None"},{"line_number":1046,"context_line":"        auth_info \u003d auth.controllers.AuthInfo(no_context, auth_data)"},{"line_number":1047,"context_line":"        auth_context \u003d {\u0027extras\u0027: {}, \u0027method_names\u0027: []}"},{"line_number":1048,"context_line":"        return context, auth_info, auth_context"},{"line_number":1049,"context_line":""},{"line_number":1050,"context_line":""}],"source_content_type":"text/x-python","patch_set":3,"id":"AAAATn%2F%2Fl4Y%3D","line":1047,"updated":"2013-10-31 18:38:21.000000000","message":"Extras?","commit_id":"6a5af32df45f478553c66a0f5ecf3f72699296a3"}]}
