)]}'
{"keystone/resource/core.py":[{"author":{"_account_id":6482,"name":"Steve Martinelli","email":"s.martinelli@gmail.com","username":"stevemar"},"change_message_id":"7a4f2383a609561e9b30ac9a1b39f0b891f7ae48","unresolved":false,"context_lines":[{"line_number":196,"context_line":"                                                     project[\u0027name\u0027])"},{"line_number":197,"context_line":""},{"line_number":198,"context_line":"        project.setdefault(\u0027enabled\u0027, True)"},{"line_number":199,"context_line":"        project[\u0027name\u0027] \u003d project[\u0027name\u0027].strip()"},{"line_number":200,"context_line":"        project.setdefault(\u0027description\u0027, \u0027\u0027)"},{"line_number":201,"context_line":""},{"line_number":202,"context_line":"        # For regular projects, the controller will ensure we have a valid"}],"source_content_type":"text/x-python","patch_set":2,"id":"bacf61ea_58c74912","line":199,"range":{"start_line":199,"start_character":42,"end_line":199,"end_character":49},"updated":"2016-07-28 05:22:37.000000000","message":"i wonder what the history is here, don\u0027t we sanitize all data before the controller? if not, we should.","commit_id":"295cfde14cefb5fd3e5a820432a1dc0892d77bcd"},{"author":{"_account_id":13063,"name":"WEI CHEN","email":"dave.jungler@gmail.com","username":"wei.d.chen"},"change_message_id":"d236fc0d2c94ea0b18a32327a67eddcd48f4093b","unresolved":false,"context_lines":[{"line_number":196,"context_line":"                                                     project[\u0027name\u0027])"},{"line_number":197,"context_line":""},{"line_number":198,"context_line":"        project.setdefault(\u0027enabled\u0027, True)"},{"line_number":199,"context_line":"        project[\u0027name\u0027] \u003d project[\u0027name\u0027].strip()"},{"line_number":200,"context_line":"        project.setdefault(\u0027description\u0027, \u0027\u0027)"},{"line_number":201,"context_line":""},{"line_number":202,"context_line":"        # For regular projects, the controller will ensure we have a valid"}],"source_content_type":"text/x-python","patch_set":2,"id":"bacf61ea_a43a9bcb","line":199,"range":{"start_line":199,"start_character":42,"end_line":199,"end_character":49},"in_reply_to":"bacf61ea_58c74912","updated":"2016-07-28 08:31:53.000000000","message":"Agreed, this is the only logic from clean.py that is not supported by jsonschema. The code is kept here since there was a test that expect to call resource_api.create_project(...) and strip the whitespace.\n\npls see https://github.com/openstack/keystone/blob/master/keystone/tests/unit/resource/test_backends.py#L102-L110","commit_id":"295cfde14cefb5fd3e5a820432a1dc0892d77bcd"},{"author":{"_account_id":6482,"name":"Steve Martinelli","email":"s.martinelli@gmail.com","username":"stevemar"},"change_message_id":"02db12aa7a0c86d728d545ee192e65e9c6790c6f","unresolved":false,"context_lines":[{"line_number":196,"context_line":"                                                     project[\u0027name\u0027])"},{"line_number":197,"context_line":""},{"line_number":198,"context_line":"        project.setdefault(\u0027enabled\u0027, True)"},{"line_number":199,"context_line":"        project[\u0027name\u0027] \u003d project[\u0027name\u0027].strip()"},{"line_number":200,"context_line":"        project.setdefault(\u0027description\u0027, \u0027\u0027)"},{"line_number":201,"context_line":""},{"line_number":202,"context_line":"        # For regular projects, the controller will ensure we have a valid"}],"source_content_type":"text/x-python","patch_set":2,"id":"bacf61ea_f02282b8","line":199,"range":{"start_line":199,"start_character":42,"end_line":199,"end_character":49},"in_reply_to":"bacf61ea_a43a9bcb","updated":"2016-07-28 11:51:23.000000000","message":"yeah, i remember seeing that test fail when Tin removed the call to clean the project name. it\u0027s just funny we validate it for *just* projects?","commit_id":"295cfde14cefb5fd3e5a820432a1dc0892d77bcd"},{"author":{"_account_id":11022,"name":"Rodrigo Duarte Sousa","email":"rodrigodsousa@gmail.com","username":"rodrigods"},"change_message_id":"b01cef1f52ce7530366abec91a63831768caa0f6","unresolved":false,"context_lines":[{"line_number":196,"context_line":"                                                     project[\u0027name\u0027])"},{"line_number":197,"context_line":""},{"line_number":198,"context_line":"        project.setdefault(\u0027enabled\u0027, True)"},{"line_number":199,"context_line":"        project[\u0027name\u0027] \u003d project[\u0027name\u0027].strip()"},{"line_number":200,"context_line":"        project.setdefault(\u0027description\u0027, \u0027\u0027)"},{"line_number":201,"context_line":""},{"line_number":202,"context_line":"        # For regular projects, the controller will ensure we have a valid"}],"source_content_type":"text/x-python","patch_set":2,"id":"bacf61ea_eb7457d0","line":199,"range":{"start_line":199,"start_character":42,"end_line":199,"end_character":49},"in_reply_to":"bacf61ea_f02282b8","updated":"2016-07-28 12:11:34.000000000","message":"+1\n\nThose inconsistencies are bad :(","commit_id":"295cfde14cefb5fd3e5a820432a1dc0892d77bcd"}]}