)]}'
{"osc_lib/clientmanager.py":[{"author":{"_account_id":7191,"name":"Jamie Lennox","email":"jamielennox@gmail.com","username":"jamielennox"},"change_message_id":"4130bc43123da5eff491c187251a5e97e8b7b573","unresolved":false,"context_lines":[{"line_number":229,"context_line":"            self.setup_auth()"},{"line_number":230,"context_line":"            LOG.debug(\"Get auth_ref\")"},{"line_number":231,"context_line":"            self._auth_ref \u003d self.auth.get_access(self.session)"},{"line_number":232,"context_line":"        return self._auth_ref"},{"line_number":233,"context_line":""},{"line_number":234,"context_line":"    def is_service_available(self, service_type):"},{"line_number":235,"context_line":"        \"\"\"Check if a service type is in the current Service Catalog\"\"\""}],"source_content_type":"text/x-python","patch_set":1,"id":"dfeb2761_8392719d","line":232,"range":{"start_line":232,"start_character":15,"end_line":232,"end_character":19},"updated":"2017-04-03 23:07:50.000000000","message":"so you\u0027re correct - get_access will memoize this so it doesn\u0027t get called twice. But a better way to do this would just be: \n\n    self.setup_auth()\n    return self.auth.get_access(session)\n\nbecause setup_auth already has a guard to prevent it being run twice and get_access will make sure it\u0027s only fetched as needed. You can then remove the self._auth_ref param as well.","commit_id":"ed6129665cef2082c1abd39952b13a71f246792e"},{"author":{"_account_id":8124,"name":"cbrandily","email":"zzelle@gmail.com","username":"cbrandily"},"change_message_id":"102d25f68fbe77893d9d0bd18d6486e77f2c0b23","unresolved":false,"context_lines":[{"line_number":229,"context_line":"            self.setup_auth()"},{"line_number":230,"context_line":"            LOG.debug(\"Get auth_ref\")"},{"line_number":231,"context_line":"            self._auth_ref \u003d self.auth.get_access(self.session)"},{"line_number":232,"context_line":"        return self._auth_ref"},{"line_number":233,"context_line":""},{"line_number":234,"context_line":"    def is_service_available(self, service_type):"},{"line_number":235,"context_line":"        \"\"\"Check if a service type is in the current Service Catalog\"\"\""}],"source_content_type":"text/x-python","patch_set":1,"id":"dfeb2761_6c9e3455","line":232,"range":{"start_line":232,"start_character":15,"end_line":232,"end_character":19},"in_reply_to":"dfeb2761_8392719d","updated":"2017-04-04 08:32:32.000000000","message":"Done","commit_id":"ed6129665cef2082c1abd39952b13a71f246792e"}]}
