)]}'
{"openstack/auth/identity/authenticator.py":[{"author":{"_account_id":8257,"name":"Brian Curtin","email":"brian@python.org","username":"briancurtin"},"change_message_id":"6f79220f25efa309ac0487659a0839be2fc276d8","unresolved":false,"context_lines":[{"line_number":18,"context_line":"class Authenticator(object):"},{"line_number":19,"context_line":""},{"line_number":20,"context_line":"    @classmethod"},{"line_number":21,"context_line":"    def create(cls, username\u003dNone, password\u003dNone, token\u003dNone, auth_url\u003dNone,"},{"line_number":22,"context_line":"               version\u003d\u00273\u0027, project_name\u003dNone):"},{"line_number":23,"context_line":"        version \u003d version.lower().replace(\u0027v\u0027, \u0027\u0027)"},{"line_number":24,"context_line":"        version \u003d version.split(\u0027.\u0027)[0]"}],"source_content_type":"text/x-python","patch_set":3,"id":"baada198_38d14c23","line":21,"updated":"2014-07-07 14:25:28.000000000","message":"Can this just be a bare function? We don\u0027t currently gain anything by this being inside a class.","commit_id":"4af4c4eecb2dd2a0d8f5f3f4eade2e6a154d02a5"},{"author":{"_account_id":7191,"name":"Jamie Lennox","email":"jamielennox@gmail.com","username":"jamielennox"},"change_message_id":"d4c99776c4f606cfa01a301cd76b9bd6475fa011","unresolved":false,"context_lines":[{"line_number":36,"context_line":"        else:"},{"line_number":37,"context_line":"            return v2.Token(auth_url, token)"},{"line_number":38,"context_line":"    msg \u003d (\"No support for identity version: %s\" % version)"},{"line_number":39,"context_line":"    raise exceptions.NoMatchingPlugin(msg)"}],"source_content_type":"text/x-python","patch_set":4,"id":"baada198_30034a11","line":39,"updated":"2014-07-14 02:37:30.000000000","message":"IMO this is a bad idea. The point of using plugins here is that you will never be able to keep up with the slew of auth plugins that become available. \n\nOff the top of my head and coming for keystoneclient include, kereberos, client cert, SAML \u0026 AD\u0027s version of SAML, federated keystone, Oauth and i\u0027m sure others.\n\n\nIt may however be useful within the examples/ directory","commit_id":"04e8b367862e009af16025b67b37709c458f8c88"}],"openstack/transport.py":[{"author":{"_account_id":8257,"name":"Brian Curtin","email":"brian@python.org","username":"briancurtin"},"change_message_id":"4db66ccbfd7ebd5928abe49755ac9e6b17fd9d3a","unresolved":false,"context_lines":[{"line_number":212,"context_line":"        return resp"},{"line_number":213,"context_line":""},{"line_number":214,"context_line":"    def _log_request(self, method, url, **kwargs):"},{"line_number":215,"context_line":"        if not _logger.isEnabledFor(logging.DEBUG):"},{"line_number":216,"context_line":"            return"},{"line_number":217,"context_line":""},{"line_number":218,"context_line":"        if \u0027params\u0027 in kwargs and kwargs[\u0027params\u0027]:"}],"source_content_type":"text/x-python","patch_set":6,"id":"baada198_aeb996ed","line":215,"updated":"2014-08-04 14:31:35.000000000","message":"Maybe needs a rebase? This was added in 109770","commit_id":"b8e268822d63f8b45ff6aa8eeb709e08c26607eb"},{"author":{"_account_id":8736,"name":"Terry Howe","email":"terrylhowe@gmail.com","username":"terrylhowe"},"change_message_id":"f37544e87a572550ad64f93b9275d4964d76f419","unresolved":false,"context_lines":[{"line_number":212,"context_line":"        return resp"},{"line_number":213,"context_line":""},{"line_number":214,"context_line":"    def _log_request(self, method, url, **kwargs):"},{"line_number":215,"context_line":"        if not _logger.isEnabledFor(logging.DEBUG):"},{"line_number":216,"context_line":"            return"},{"line_number":217,"context_line":""},{"line_number":218,"context_line":"        if \u0027params\u0027 in kwargs and kwargs[\u0027params\u0027]:"}],"source_content_type":"text/x-python","patch_set":6,"id":"baada198_9c72be48","line":215,"in_reply_to":"baada198_aeb996ed","updated":"2014-08-07 18:58:57.000000000","message":"https://review.openstack.org/#/c/109770 was merged back on the 28th.  I rebased, but I think this is good to go.","commit_id":"b8e268822d63f8b45ff6aa8eeb709e08c26607eb"}]}