)]}'
{"keystoneauth1/extras/kerberos/__init__.py":[{"author":{"_account_id":7191,"name":"Jamie Lennox","email":"jamielennox@gmail.com","username":"jamielennox"},"change_message_id":"167d5870071479a931586dfe5fa2637f517fb6d6","unresolved":false,"context_lines":[{"line_number":42,"context_line":"    _method_parameters \u003d []"},{"line_number":43,"context_line":""},{"line_number":44,"context_line":"    def __init__(self, *args, **kwargs):"},{"line_number":45,"context_line":"        super(KerberosMethod, self).__init__(**kwargs)"},{"line_number":46,"context_line":""},{"line_number":47,"context_line":"    def get_auth_data(self, session, auth, headers, request_kwargs, **kwargs):"},{"line_number":48,"context_line":"        # NOTE(jamielennox): request_kwargs is passed as a kwarg however it is"}],"source_content_type":"text/x-python","patch_set":3,"id":"bab6814e_8c071fc9","line":45,"range":{"start_line":45,"start_character":30,"end_line":45,"end_character":34},"updated":"2016-05-26 22:51:21.000000000","message":"Can you tell me what\u0027s being passed through here as *args? that doesn\u0027t seem right.","commit_id":"d7ad58ad640f3df0c22a96e4f98fdea9e2246c40"},{"author":{"_account_id":8482,"name":"Colleen Murphy","email":"colleen@gazlene.net","username":"krinkle"},"change_message_id":"e78ce4e47e845dae335d8ad7581c6e95a059df3a","unresolved":false,"context_lines":[{"line_number":42,"context_line":"    _method_parameters \u003d []"},{"line_number":43,"context_line":""},{"line_number":44,"context_line":"    def __init__(self, *args, **kwargs):"},{"line_number":45,"context_line":"        super(KerberosMethod, self).__init__(**kwargs)"},{"line_number":46,"context_line":""},{"line_number":47,"context_line":"    def get_auth_data(self, session, auth, headers, request_kwargs, **kwargs):"},{"line_number":48,"context_line":"        # NOTE(jamielennox): request_kwargs is passed as a kwarg however it is"}],"source_content_type":"text/x-python","patch_set":3,"id":"bab6814e_f265e0e0","line":45,"range":{"start_line":45,"start_character":30,"end_line":45,"end_character":34},"in_reply_to":"bab6814e_8c071fc9","updated":"2016-05-27 01:04:45.000000000","message":"Must have been running into something weird early on, I don\u0027t seem to need it any more.","commit_id":"d7ad58ad640f3df0c22a96e4f98fdea9e2246c40"},{"author":{"_account_id":7191,"name":"Jamie Lennox","email":"jamielennox@gmail.com","username":"jamielennox"},"change_message_id":"167d5870071479a931586dfe5fa2637f517fb6d6","unresolved":false,"context_lines":[{"line_number":55,"context_line":"    _auth_method_class \u003d KerberosMethod"},{"line_number":56,"context_line":""},{"line_number":57,"context_line":""},{"line_number":58,"context_line":"class MappedKerberos(federation.FederationBaseAuth):"},{"line_number":59,"context_line":"    \"\"\"Authenticate using Kerberos via the keystone federation mechanisms."},{"line_number":60,"context_line":""},{"line_number":61,"context_line":"    This uses the OS-FEDERATION extension to gain an unscoped token and then"}],"source_content_type":"text/x-python","patch_set":3,"id":"bab6814e_8cd55f1d","line":58,"range":{"start_line":58,"start_character":6,"end_line":58,"end_character":20},"updated":"2016-05-26 22:51:21.000000000","message":"I\u0027d really like this to become the default, i don\u0027t know what we call it, in ksc it was v3fedkerb i think","commit_id":"d7ad58ad640f3df0c22a96e4f98fdea9e2246c40"},{"author":{"_account_id":8482,"name":"Colleen Murphy","email":"colleen@gazlene.net","username":"krinkle"},"change_message_id":"e78ce4e47e845dae335d8ad7581c6e95a059df3a","unresolved":false,"context_lines":[{"line_number":55,"context_line":"    _auth_method_class \u003d KerberosMethod"},{"line_number":56,"context_line":""},{"line_number":57,"context_line":""},{"line_number":58,"context_line":"class MappedKerberos(federation.FederationBaseAuth):"},{"line_number":59,"context_line":"    \"\"\"Authenticate using Kerberos via the keystone federation mechanisms."},{"line_number":60,"context_line":""},{"line_number":61,"context_line":"    This uses the OS-FEDERATION extension to gain an unscoped token and then"}],"source_content_type":"text/x-python","patch_set":3,"id":"bab6814e_32ec88ba","line":58,"range":{"start_line":58,"start_character":6,"end_line":58,"end_character":20},"in_reply_to":"bab6814e_8cd55f1d","updated":"2016-05-27 01:04:45.000000000","message":"By default do you mean have v3kerberos be the entrypoint to MappedKerberos? Or should it be an additional entry point called v3fedkerb like in https://review.openstack.org/gitweb?p\u003dopenstack%2Fpython-keystoneclient-kerberos.git;a\u003dcommitdiff;h\u003d9f11208fb47762bf78cdff28e6c35c2490d2b6a1","commit_id":"d7ad58ad640f3df0c22a96e4f98fdea9e2246c40"},{"author":{"_account_id":6482,"name":"Steve Martinelli","email":"s.martinelli@gmail.com","username":"stevemar"},"change_message_id":"a73aeff3c64372786106ade56763bb374aa6be42","unresolved":false,"context_lines":[{"line_number":18,"context_line":"    not installed by default. Without the extra package an import error will"},{"line_number":19,"context_line":"    occur. The extra package can be installed using::"},{"line_number":20,"context_line":""},{"line_number":21,"context_line":"      $ pip install keystoneauth1[\u0027kerberos\u0027]"},{"line_number":22,"context_line":""},{"line_number":23,"context_line":"\"\"\""},{"line_number":24,"context_line":""}],"source_content_type":"text/x-python","patch_set":5,"id":"9abb7d3a_5d1bb51b","line":21,"range":{"start_line":21,"start_character":20,"end_line":21,"end_character":33},"updated":"2016-05-28 04:36:32.000000000","message":"nice catch :)","commit_id":"33133581aea946c82f629128d079d031bacc6b64"},{"author":{"_account_id":4,"name":"Dolph Mathews","email":"dolph.mathews@gmail.com","username":"dolph"},"change_message_id":"0104f1d8776bcffb9487a8a4988a5ffd88f19b8c","unresolved":false,"context_lines":[{"line_number":24,"context_line":"try:"},{"line_number":25,"context_line":"    import requests_kerberos"},{"line_number":26,"context_line":"except ImportError:"},{"line_number":27,"context_line":"    _HAS_DEPENDENCIES \u003d False"},{"line_number":28,"context_line":"else:"},{"line_number":29,"context_line":"    _HAS_DEPENDENCIES \u003d True"},{"line_number":30,"context_line":""}],"source_content_type":"text/x-python","patch_set":11,"id":"7aa08908_2841293a","line":27,"updated":"2016-06-10 16:02:29.000000000","message":"you don\u0027t need to use another variable for this - instead, just set requests_kerberos to None","commit_id":"b1fd91c8ccd7090e4d57680df46e509606569f4e"},{"author":{"_account_id":6482,"name":"Steve Martinelli","email":"s.martinelli@gmail.com","username":"stevemar"},"change_message_id":"23d9995aebe27a9440420c9c80e39bbf2d71e63a","unresolved":false,"context_lines":[{"line_number":47,"context_line":"Using the kerberos authentication plugin requires installation of additional"},{"line_number":48,"context_line":"packages. These can be installed with::"},{"line_number":49,"context_line":""},{"line_number":50,"context_line":"    $ pip install keystoneauth1[\u0027kerberos\u0027]"},{"line_number":51,"context_line":"\"\"\")"},{"line_number":52,"context_line":""},{"line_number":53,"context_line":""}],"source_content_type":"text/x-python","patch_set":11,"id":"7aa08908_b408cfda","line":50,"range":{"start_line":50,"start_character":32,"end_line":50,"end_character":42},"updated":"2016-06-07 04:32:33.000000000","message":"you don\u0027t use the singular quotes","commit_id":"b1fd91c8ccd7090e4d57680df46e509606569f4e"},{"author":{"_account_id":4,"name":"Dolph Mathews","email":"dolph.mathews@gmail.com","username":"dolph"},"change_message_id":"0104f1d8776bcffb9487a8a4988a5ffd88f19b8c","unresolved":false,"context_lines":[{"line_number":57,"context_line":""},{"line_number":58,"context_line":"    def __init__(self, *args, **kwargs):"},{"line_number":59,"context_line":"        super(KerberosMethod, self).__init__(*args, **kwargs)"},{"line_number":60,"context_line":"        _dependency_check()"},{"line_number":61,"context_line":""},{"line_number":62,"context_line":"    def get_auth_data(self, session, auth, headers, request_kwargs, **kwargs):"},{"line_number":63,"context_line":"        # NOTE(jamielennox): request_kwargs is passed as a kwarg however it is"}],"source_content_type":"text/x-python","patch_set":11,"id":"7aa08908_a881994f","line":60,"updated":"2016-06-10 16:02:29.000000000","message":"I\u0027d suggest doing this in the reverse order, so that you don\u0027t bother trying to initialize the parent class at all until you\u0027ve done the dependency check.","commit_id":"b1fd91c8ccd7090e4d57680df46e509606569f4e"},{"author":{"_account_id":4,"name":"Dolph Mathews","email":"dolph.mathews@gmail.com","username":"dolph"},"change_message_id":"0104f1d8776bcffb9487a8a4988a5ffd88f19b8c","unresolved":false,"context_lines":[{"line_number":79,"context_line":""},{"line_number":80,"context_line":"    def __init__(self, *args, **kwargs):"},{"line_number":81,"context_line":"        super(MappedKerberos, self).__init__(*args, **kwargs)"},{"line_number":82,"context_line":"        _dependency_check()"},{"line_number":83,"context_line":""},{"line_number":84,"context_line":"    def get_unscoped_auth_ref(self, session, **kwargs):"},{"line_number":85,"context_line":"        resp \u003d session.get(self.federated_token_url,"}],"source_content_type":"text/x-python","patch_set":11,"id":"7aa08908_c8f505ee","line":82,"updated":"2016-06-10 16:02:29.000000000","message":"Same as the above.","commit_id":"b1fd91c8ccd7090e4d57680df46e509606569f4e"},{"author":{"_account_id":6482,"name":"Steve Martinelli","email":"s.martinelli@gmail.com","username":"stevemar"},"change_message_id":"a3b73e3f003eee93315ea8e57bd93dd06f777c8d","unresolved":false,"context_lines":[{"line_number":43,"context_line":"    if requests_kerberos is None:"},{"line_number":44,"context_line":"        raise ImportError(\"\"\""},{"line_number":45,"context_line":"Using the kerberos authentication plugin requires installation of additional"},{"line_number":46,"context_line":"packages. These can be installed with::"},{"line_number":47,"context_line":""},{"line_number":48,"context_line":"    $ pip install keystoneauth1[kerberos]"},{"line_number":49,"context_line":"\"\"\")"}],"source_content_type":"text/x-python","patch_set":12,"id":"7aa08908_47a44db8","line":46,"range":{"start_line":46,"start_character":37,"end_line":46,"end_character":38},"updated":"2016-06-10 22:51:22.000000000","message":"the second colon doesn\u0027t do anything, it\u0027s not rst format here","commit_id":"fc95d25544a0f14e886810ab53c689909b22cad9"}],"keystoneauth1/extras/kerberos/_loading.py":[{"author":{"_account_id":7191,"name":"Jamie Lennox","email":"jamielennox@gmail.com","username":"jamielennox"},"change_message_id":"167d5870071479a931586dfe5fa2637f517fb6d6","unresolved":false,"context_lines":[{"line_number":24,"context_line":"        options \u003d super(Kerberos, self).get_options()"},{"line_number":25,"context_line":""},{"line_number":26,"context_line":"        options.extend(["},{"line_number":27,"context_line":"            loading.Opt(\u0027auth-url\u0027, required\u003dTrue, help\u003d\u0027Authentication URL\u0027),"},{"line_number":28,"context_line":"        ])"},{"line_number":29,"context_line":""},{"line_number":30,"context_line":"        return options"}],"source_content_type":"text/x-python","patch_set":3,"id":"bab6814e_ece4eb0d","line":27,"range":{"start_line":27,"start_character":50,"end_line":27,"end_character":51},"updated":"2016-05-26 22:51:21.000000000","message":"don\u0027t we need project_id and other scoping params here?\n\nBaseIdentityLoader would get you auth-url, but i think you want BaseV3Loader.","commit_id":"d7ad58ad640f3df0c22a96e4f98fdea9e2246c40"},{"author":{"_account_id":8482,"name":"Colleen Murphy","email":"colleen@gazlene.net","username":"krinkle"},"change_message_id":"e78ce4e47e845dae335d8ad7581c6e95a059df3a","unresolved":false,"context_lines":[{"line_number":24,"context_line":"        options \u003d super(Kerberos, self).get_options()"},{"line_number":25,"context_line":""},{"line_number":26,"context_line":"        options.extend(["},{"line_number":27,"context_line":"            loading.Opt(\u0027auth-url\u0027, required\u003dTrue, help\u003d\u0027Authentication URL\u0027),"},{"line_number":28,"context_line":"        ])"},{"line_number":29,"context_line":""},{"line_number":30,"context_line":"        return options"}],"source_content_type":"text/x-python","patch_set":3,"id":"bab6814e_922d7400","line":27,"range":{"start_line":27,"start_character":50,"end_line":27,"end_character":51},"in_reply_to":"bab6814e_ece4eb0d","updated":"2016-05-27 01:04:45.000000000","message":"Done","commit_id":"d7ad58ad640f3df0c22a96e4f98fdea9e2246c40"}],"keystoneauth1/tests/unit/extras/kerberos/test_kerberos_loading.py":[{"author":{"_account_id":17860,"name":"Samuel de Medeiros Queiroz","email":"samueldmq@gmail.com","username":"samueldmq"},"change_message_id":"783d163f3cb4726744fd0501e28bb103cdfb9114","unresolved":false,"context_lines":[{"line_number":30,"context_line":"                        \u0027auth-url\u0027,"},{"line_number":31,"context_line":"                        ]"},{"line_number":32,"context_line":""},{"line_number":33,"context_line":"        self.assertEqual(set(allowed_opts), set(opts))"},{"line_number":34,"context_line":"        self.assertEqual(len(allowed_opts), len(opts))"}],"source_content_type":"text/x-python","patch_set":11,"id":"7aa08908_4c890926","line":34,"range":{"start_line":33,"start_character":8,"end_line":34,"end_character":54},"updated":"2016-06-07 13:07:05.000000000","message":"you may replace this by assertItemsEqual","commit_id":"b1fd91c8ccd7090e4d57680df46e509606569f4e"}],"setup.cfg":[{"author":{"_account_id":7191,"name":"Jamie Lennox","email":"jamielennox@gmail.com","username":"jamielennox"},"change_message_id":"167d5870071479a931586dfe5fa2637f517fb6d6","unresolved":false,"context_lines":[{"line_number":48,"context_line":"    v3oidcauthcode \u003d keystoneauth1.loading._plugins.identity.v3:OpenIDConnectAuthorizationCode"},{"line_number":49,"context_line":"    v3oidcaccesstoken \u003d keystoneauth1.loading._plugins.identity.v3:OpenIDConnectAccessToken"},{"line_number":50,"context_line":"    v3oauth1 \u003d keystoneauth1.extras.oauth1._loading:V3OAuth1"},{"line_number":51,"context_line":"    kerberos \u003d keystoneauth1.extras.kerberos._loading:Kerberos"},{"line_number":52,"context_line":""},{"line_number":53,"context_line":"[build_sphinx]"},{"line_number":54,"context_line":"source-dir \u003d doc/source"}],"source_content_type":"text/x-ttcn-cfg","patch_set":3,"id":"bab6814e_6ca63bc2","line":51,"range":{"start_line":51,"start_character":15,"end_line":51,"end_character":28},"updated":"2016-05-26 22:51:21.000000000","message":"\u0027v3kerberos\u0027 because it expects a v3 auth_url to be passed.\n\nFeel free to subclass the Generic plugin for a \u0027kerberos\u0027 if you want it.","commit_id":"d7ad58ad640f3df0c22a96e4f98fdea9e2246c40"},{"author":{"_account_id":8482,"name":"Colleen Murphy","email":"colleen@gazlene.net","username":"krinkle"},"change_message_id":"e78ce4e47e845dae335d8ad7581c6e95a059df3a","unresolved":false,"context_lines":[{"line_number":48,"context_line":"    v3oidcauthcode \u003d keystoneauth1.loading._plugins.identity.v3:OpenIDConnectAuthorizationCode"},{"line_number":49,"context_line":"    v3oidcaccesstoken \u003d keystoneauth1.loading._plugins.identity.v3:OpenIDConnectAccessToken"},{"line_number":50,"context_line":"    v3oauth1 \u003d keystoneauth1.extras.oauth1._loading:V3OAuth1"},{"line_number":51,"context_line":"    kerberos \u003d keystoneauth1.extras.kerberos._loading:Kerberos"},{"line_number":52,"context_line":""},{"line_number":53,"context_line":"[build_sphinx]"},{"line_number":54,"context_line":"source-dir \u003d doc/source"}],"source_content_type":"text/x-ttcn-cfg","patch_set":3,"id":"bab6814e_7230d056","line":51,"range":{"start_line":51,"start_character":15,"end_line":51,"end_character":28},"in_reply_to":"bab6814e_6ca63bc2","updated":"2016-05-27 01:04:45.000000000","message":"Done","commit_id":"d7ad58ad640f3df0c22a96e4f98fdea9e2246c40"}]}