)]}'
{"keystoneclient/discover.py":[{"author":{"_account_id":7191,"name":"Jamie Lennox","email":"jamielennox@gmail.com","username":"jamielennox"},"change_message_id":"b9ddaf614a307300b8f180a9f7289530526d2ff5","unresolved":false,"context_lines":[{"line_number":172,"context_line":"            url \u003d auth_url"},{"line_number":173,"context_line":"        elif hasattr(session.auth, \u0027auth_url\u0027):"},{"line_number":174,"context_line":"            self._use_endpoint \u003d False"},{"line_number":175,"context_line":"            url \u003d session.auth.auth_url"},{"line_number":176,"context_line":""},{"line_number":177,"context_line":"        if not url:"},{"line_number":178,"context_line":"            raise exceptions.DiscoveryFailure("}],"source_content_type":"text/x-python","patch_set":4,"id":"da85f559_4b04867d","line":175,"updated":"2015-11-08 20:56:54.000000000","message":"if we want auth_url do session.get_endpoint(AUTH_INTERFACE) and i would need to check on what that might raise.\n\nIdeally this would work with an Adapter but this whole file is kind of awkward now.","commit_id":"a0a2fb1399fab624e229f08bd189553c8e27e41a"},{"author":{"_account_id":6482,"name":"Steve Martinelli","email":"s.martinelli@gmail.com","username":"stevemar"},"change_message_id":"385adec817d06ea27c794aa44c7ae4627900ba75","unresolved":false,"context_lines":[{"line_number":17,"context_line":"import six"},{"line_number":18,"context_line":""},{"line_number":19,"context_line":"from keystoneclient import _discover"},{"line_number":20,"context_line":"from keystoneclient.auth import base"},{"line_number":21,"context_line":"from keystoneclient import exceptions"},{"line_number":22,"context_line":"from keystoneclient.i18n import _"},{"line_number":23,"context_line":"from keystoneclient import session as client_session"}],"source_content_type":"text/x-python","patch_set":6,"id":"da85f559_13cf9057","line":20,"updated":"2015-11-09 07:21:31.000000000","message":"why not use the one from ksa, since you\u0027re pulling it in now?","commit_id":"d9031c252848d89270a543b67109a46f9c505c86"},{"author":{"_account_id":2903,"name":"Morgan Fainberg","email":"morgan.fainberg@gmail.com","username":"mdrnstm"},"change_message_id":"ad1260581597ae98fabeead244bd89f8aba1a75c","unresolved":false,"context_lines":[{"line_number":17,"context_line":"import six"},{"line_number":18,"context_line":""},{"line_number":19,"context_line":"from keystoneclient import _discover"},{"line_number":20,"context_line":"from keystoneclient.auth import base"},{"line_number":21,"context_line":"from keystoneclient import exceptions"},{"line_number":22,"context_line":"from keystoneclient.i18n import _"},{"line_number":23,"context_line":"from keystoneclient import session as client_session"}],"source_content_type":"text/x-python","patch_set":6,"id":"da85f559_68359ed8","line":20,"in_reply_to":"da85f559_13cf9057","updated":"2015-11-09 20:11:49.000000000","message":"That is a good question, but I think that could be an out-of-scope for this minor fix as it is outlined in the commit message.","commit_id":"d9031c252848d89270a543b67109a46f9c505c86"},{"author":{"_account_id":7191,"name":"Jamie Lennox","email":"jamielennox@gmail.com","username":"jamielennox"},"change_message_id":"63bffc99c26d6b3c836f40911b3edfb6f4f93f13","unresolved":false,"context_lines":[{"line_number":17,"context_line":"import six"},{"line_number":18,"context_line":""},{"line_number":19,"context_line":"from keystoneclient import _discover"},{"line_number":20,"context_line":"from keystoneclient.auth import base"},{"line_number":21,"context_line":"from keystoneclient import exceptions"},{"line_number":22,"context_line":"from keystoneclient.i18n import _"},{"line_number":23,"context_line":"from keystoneclient import session as client_session"}],"source_content_type":"text/x-python","patch_set":6,"id":"da85f559_9d091d2f","line":20,"in_reply_to":"da85f559_68359ed8","updated":"2015-11-09 23:18:53.000000000","message":"i think from keystoneclient we should continue to use values from there keystoneclient locations. Feels less confusing.\n\nI always think from keystoneclient import auth looks better than bringing in base - but whatever.","commit_id":"d9031c252848d89270a543b67109a46f9c505c86"}],"keystoneclient/tests/unit/auth/test_identity_v3.py":[{"author":{"_account_id":6482,"name":"Steve Martinelli","email":"s.martinelli@gmail.com","username":"stevemar"},"change_message_id":"67ddd1490d9742400b7e308c7642409a91415ef3","unresolved":false,"context_lines":[{"line_number":216,"context_line":"                        username\u003dself.TEST_USER,"},{"line_number":217,"context_line":"                        password\u003dself.TEST_PASS)"},{"line_number":218,"context_line":"        s \u003d session.Session(auth\u003da)"},{"line_number":219,"context_line":"        cs \u003d client.Client(session\u003ds)"},{"line_number":220,"context_line":""},{"line_number":221,"context_line":"        # As a sanity check on the auth_ref, make sure client has the"},{"line_number":222,"context_line":"        # proper user id, that it fetches the right project response"}],"source_content_type":"text/x-python","patch_set":2,"id":"fa80f949_3c754863","line":219,"updated":"2015-11-06 14:36:23.000000000","message":"this should be a new test as this test is testing something different that the new proposed code, it just so happens to be surfaced here too.","commit_id":"80354cec8bc316ca9707cc52bcef81bfd2c60548"}]}