)]}'
{"openstack_auth/utils.py":[{"author":{"_account_id":5623,"name":"David Lyle","email":"dklyle0@gmail.com","username":"david-lyle"},"change_message_id":"2273cf0db5e75d8495111975ca359a4966964f4a","unresolved":false,"context_lines":[{"line_number":280,"context_line":"        if get_keystone_version() \u003e\u003d 3:"},{"line_number":281,"context_line":"            auth_url \u003d url_path_append(auth_url, \"/v3\")"},{"line_number":282,"context_line":"        else:"},{"line_number":283,"context_line":"            auth_url \u003d url_path_append(auth_url, \"/v2.0\")"},{"line_number":284,"context_line":""},{"line_number":285,"context_line":"    if get_keystone_version() \u003e\u003d 3:"},{"line_number":286,"context_line":"        if has_in_url_path(auth_url, \"/v2.0\"):"}],"source_content_type":"text/x-python","patch_set":5,"id":"7aa08908_2a50f3d3","line":283,"updated":"2016-06-16 17:32:15.000000000","message":"could this now result in a \"//\" e.g., //v3 or //v2.0 ? Not entirely sure that\u0027s problematic, but it\u0027s a change from the previous version.","commit_id":"a5ad815233e8af029add2cc4c89a955515817c20"},{"author":{"_account_id":8040,"name":"Timur Sufiev","email":"tsufiev@gmail.com","username":"tsufiev"},"change_message_id":"5b662f895f5cf77b97a1c0aaec5aa1c46dc13ca0","unresolved":false,"context_lines":[{"line_number":280,"context_line":"        if get_keystone_version() \u003e\u003d 3:"},{"line_number":281,"context_line":"            auth_url \u003d url_path_append(auth_url, \"/v3\")"},{"line_number":282,"context_line":"        else:"},{"line_number":283,"context_line":"            auth_url \u003d url_path_append(auth_url, \"/v2.0\")"},{"line_number":284,"context_line":""},{"line_number":285,"context_line":"    if get_keystone_version() \u003e\u003d 3:"},{"line_number":286,"context_line":"        if has_in_url_path(auth_url, \"/v2.0\"):"}],"source_content_type":"text/x-python","patch_set":5,"id":"7aa08908_ca116f5f","line":283,"in_reply_to":"7aa08908_2a50f3d3","updated":"2016-06-16 17:47:17.000000000","message":"Double slash is dealt with inside url_path_append(), line 259 - otherwise the tests complaint.","commit_id":"a5ad815233e8af029add2cc4c89a955515817c20"},{"author":{"_account_id":5623,"name":"David Lyle","email":"dklyle0@gmail.com","username":"david-lyle"},"change_message_id":"2273cf0db5e75d8495111975ca359a4966964f4a","unresolved":false,"context_lines":[{"line_number":307,"context_line":"    # there. Keystone version needs to be added to endpoint because as of Kilo,"},{"line_number":308,"context_line":"    # the identity URLs returned by Keystone might no longer contain API"},{"line_number":309,"context_line":"    # versions, leaving the version choice up to the user."},{"line_number":310,"context_line":"    if not has_in_url_path(auth_url, [\"/v2.0\", \"/v3\"]):"},{"line_number":311,"context_line":"        if get_keystone_version() \u003e\u003d 3:"},{"line_number":312,"context_line":"            auth_url \u003d url_path_append(auth_url, \"/v3\")"},{"line_number":313,"context_line":"        else:"}],"source_content_type":"text/x-python","patch_set":5,"id":"7aa08908_eaf26b9a","line":310,"updated":"2016-06-16 17:32:15.000000000","message":"could this piece be moved into a separate method to be more DRY?","commit_id":"a5ad815233e8af029add2cc4c89a955515817c20"},{"author":{"_account_id":8040,"name":"Timur Sufiev","email":"tsufiev@gmail.com","username":"tsufiev"},"change_message_id":"5b662f895f5cf77b97a1c0aaec5aa1c46dc13ca0","unresolved":false,"context_lines":[{"line_number":307,"context_line":"    # there. Keystone version needs to be added to endpoint because as of Kilo,"},{"line_number":308,"context_line":"    # the identity URLs returned by Keystone might no longer contain API"},{"line_number":309,"context_line":"    # versions, leaving the version choice up to the user."},{"line_number":310,"context_line":"    if not has_in_url_path(auth_url, [\"/v2.0\", \"/v3\"]):"},{"line_number":311,"context_line":"        if get_keystone_version() \u003e\u003d 3:"},{"line_number":312,"context_line":"            auth_url \u003d url_path_append(auth_url, \"/v3\")"},{"line_number":313,"context_line":"        else:"}],"source_content_type":"text/x-python","patch_set":5,"id":"7aa08908_4d3479dd","line":310,"in_reply_to":"7aa08908_eaf26b9a","updated":"2016-06-16 17:47:17.000000000","message":"Moving the whole block on lines 304-314 to a separate helper function.","commit_id":"a5ad815233e8af029add2cc4c89a955515817c20"},{"author":{"_account_id":5623,"name":"David Lyle","email":"dklyle0@gmail.com","username":"david-lyle"},"change_message_id":"2273cf0db5e75d8495111975ca359a4966964f4a","unresolved":false,"context_lines":[{"line_number":308,"context_line":"    # the identity URLs returned by Keystone might no longer contain API"},{"line_number":309,"context_line":"    # versions, leaving the version choice up to the user."},{"line_number":310,"context_line":"    if not has_in_url_path(auth_url, [\"/v2.0\", \"/v3\"]):"},{"line_number":311,"context_line":"        if get_keystone_version() \u003e\u003d 3:"},{"line_number":312,"context_line":"            auth_url \u003d url_path_append(auth_url, \"/v3\")"},{"line_number":313,"context_line":"        else:"},{"line_number":314,"context_line":"            auth_url \u003d url_path_append(auth_url, \"/v2.0\")"}],"source_content_type":"text/x-python","patch_set":5,"id":"7aa08908_8a2be732","line":311,"updated":"2016-06-16 17:32:15.000000000","message":"same question here","commit_id":"a5ad815233e8af029add2cc4c89a955515817c20"}]}