)]}'
{"tempest/lib/auth.py":[{"author":{"_account_id":20190,"name":"zhufl","email":"zhu.fanglei@zte.com.cn","username":"zhufl"},"change_message_id":"812517c44872a2da272250e07854508023cab9f5","unresolved":false,"context_lines":[{"line_number":576,"context_line":"            # No matching type, keep all and try matching by region at least"},{"line_number":577,"context_line":"            filtered_catalog \u003d service_catalog"},{"line_number":578,"context_line":"        # Filter by region only if the variable has a value"},{"line_number":579,"context_line":"        if region:"},{"line_number":580,"context_line":"            # Only update filtered_catalog if an endpoint with a matching region is found"},{"line_number":581,"context_line":"            matching_region \u003d [ep for ep in filtered_catalog if"},{"line_number":582,"context_line":"                                ep[\u0027region\u0027] \u003d\u003d region]"}],"source_content_type":"text/x-python","patch_set":1,"id":"9fb8cfa7_8715309e","line":579,"range":{"start_line":579,"start_character":11,"end_line":579,"end_character":17},"updated":"2019-07-02 02:14:14.000000000","message":"I\u0027m curious when will region be without a value?","commit_id":"f3557a9134d8f6cd6807d9ee78051b3509c6f283"},{"author":{"_account_id":20190,"name":"zhufl","email":"zhu.fanglei@zte.com.cn","username":"zhufl"},"change_message_id":"b2012d2ec6407a32921e0f34f1fab3769158f525","unresolved":false,"context_lines":[{"line_number":576,"context_line":"            # No matching type, keep all and try matching by region at least"},{"line_number":577,"context_line":"            filtered_catalog \u003d service_catalog"},{"line_number":578,"context_line":"        # Filter by region only if the variable has a value"},{"line_number":579,"context_line":"        if region:"},{"line_number":580,"context_line":"            # Only update filtered_catalog if an endpoint with a matching region is found"},{"line_number":581,"context_line":"            matching_region \u003d [ep for ep in filtered_catalog if"},{"line_number":582,"context_line":"                                ep[\u0027region\u0027] \u003d\u003d region]"}],"source_content_type":"text/x-python","patch_set":1,"id":"9fb8cfa7_0c696023","line":579,"range":{"start_line":579,"start_character":8,"end_line":579,"end_character":18},"updated":"2019-07-02 08:49:47.000000000","message":"So this can be removed?\nEither no region provided or no region matched, we should use results filtered by endpoint.","commit_id":"f3557a9134d8f6cd6807d9ee78051b3509c6f283"},{"author":{"_account_id":20190,"name":"zhufl","email":"zhu.fanglei@zte.com.cn","username":"zhufl"},"change_message_id":"7901bb5fb31346ec8aaf5147a9e2d96724ad42e8","unresolved":false,"context_lines":[{"line_number":576,"context_line":"            # No matching type, keep all and try matching by region at least"},{"line_number":577,"context_line":"            filtered_catalog \u003d service_catalog"},{"line_number":578,"context_line":"        # Filter by region only if the variable has a value"},{"line_number":579,"context_line":"        if region:"},{"line_number":580,"context_line":"            # Only update filtered_catalog if an endpoint with a matching region is found"},{"line_number":581,"context_line":"            matching_region \u003d [ep for ep in filtered_catalog if"},{"line_number":582,"context_line":"                                ep[\u0027region\u0027] \u003d\u003d region]"}],"source_content_type":"text/x-python","patch_set":1,"id":"9fb8cfa7_f67e1b17","line":579,"range":{"start_line":579,"start_character":11,"end_line":579,"end_character":17},"in_reply_to":"9fb8cfa7_8715309e","updated":"2019-07-02 05:42:40.000000000","message":"And what if region is not empty but no region matched?\nIn this case shall filtered_catalog still keep the result filtered by endpoint_type?","commit_id":"f3557a9134d8f6cd6807d9ee78051b3509c6f283"}]}
