)]}'
{"glance/common/auth.py":[{"author":{"_account_id":706,"name":"Brian Lamar","email":"brian.lamar@rackspace.com","username":"blamar"},"change_message_id":"71d06393339976ad5aabbc6f8698aff0bb4e7010","unresolved":false,"context_lines":[{"line_number":192,"context_line":"            only one -- successful match in the catalog,"},{"line_number":193,"context_line":"            otherwise we will raise an exception."},{"line_number":194,"context_line":"            \"\"\""},{"line_number":195,"context_line":"            # FIXME(sirp): for now just use the public url."},{"line_number":196,"context_line":"            endpoint \u003d None"},{"line_number":197,"context_line":"            region \u003d self.creds.get(\u0027region\u0027)"},{"line_number":198,"context_line":"            for service in service_catalog:"}],"source_content_type":"text/x-python","patch_set":1,"id":"AAAAEn%2F%2F0Ec%3D","line":195,"updated":"2012-02-29 23:28:02.000000000","message":"Is this note still valid? I see this method returning the entire endpoint and not just the publicURL.","commit_id":"9f25238766786a9677c78908e85ce1d45ef49fca"},{"author":{"_account_id":706,"name":"Brian Lamar","email":"brian.lamar@rackspace.com","username":"blamar"},"change_message_id":"71d06393339976ad5aabbc6f8698aff0bb4e7010","unresolved":false,"context_lines":[{"line_number":205,"context_line":"                            endpoint \u003d ep"},{"line_number":206,"context_line":"            if endpoint is None:"},{"line_number":207,"context_line":"                raise exception.NoServiceEndpoint()"},{"line_number":208,"context_line":"            return endpoint"},{"line_number":209,"context_line":""},{"line_number":210,"context_line":"        creds \u003d self.creds"},{"line_number":211,"context_line":""}],"source_content_type":"text/x-python","patch_set":1,"id":"AAAAEn%2F%2F0EU%3D","line":208,"updated":"2012-02-29 23:28:02.000000000","message":"Should this be endpoint[\"publicURL\"] ?","commit_id":"9f25238766786a9677c78908e85ce1d45ef49fca"},{"author":{"_account_id":2284,"name":"Eoghan Glynn","email":"eglynn@redhat.com","username":"eglynn"},"change_message_id":"d1a4cf35eec3a40d598bdd79912cad1443b5ac47","unresolved":false,"context_lines":[{"line_number":205,"context_line":"                            endpoint \u003d ep"},{"line_number":206,"context_line":"            if endpoint is None:"},{"line_number":207,"context_line":"                raise exception.NoServiceEndpoint()"},{"line_number":208,"context_line":"            return endpoint"},{"line_number":209,"context_line":""},{"line_number":210,"context_line":"        creds \u003d self.creds"},{"line_number":211,"context_line":""}],"source_content_type":"text/x-python","patch_set":1,"id":"AAAAEn%2F%2F0CU%3D","line":208,"in_reply_to":"AAAAEn%2F%2F0EU%3D","updated":"2012-03-01 00:12:27.000000000","message":"Great catch, you\u0027re absolutely right about the endpioint[\u0027publicURL\u0027] \n\nReturning the entire dict caused the glance client to fail in a nasty way, fixed up in patch set 2.\n\nI\u0027m just trying to get Stuart\u0027s patch knocked into shape for E4, so I\u0027m at a bit of disadvantge in terms of explaining how that issue wasn\u0027t picked up in testing.\n\nI suspect it was falling back to v1 auth in the tests.","commit_id":"9f25238766786a9677c78908e85ce1d45ef49fca"}]}