)]}'
{"vitrage/middleware/keycloak.py":[{"author":{"_account_id":21414,"name":"Yujun Zhang","email":"zhang.yujunz@zte.com.cn","username":"yujunz"},"change_message_id":"3c143275cef0ba48f3ef9280c21aa39399ca04f2","unresolved":false,"context_lines":[{"line_number":34,"context_line":"               help\u003d\u0027Keycloak authentication server ip\u0027,),"},{"line_number":35,"context_line":"    cfg.StrOpt(\u0027insecure\u0027, default\u003dFalse,"},{"line_number":36,"context_line":"               help\u003d\u0027If True, SSL/TLS certificate verification is disabled\u0027),"},{"line_number":37,"context_line":"    cfg.StrOpt(\u0027certfile\u0027,"},{"line_number":38,"context_line":"               help\u003d\u0027Required if identity server requires client certificate\u0027),"},{"line_number":39,"context_line":"    cfg.StrOpt(\u0027keyfile\u0027,"},{"line_number":40,"context_line":"               help\u003d\u0027Required if identity server requires client certificate\u0027),"},{"line_number":41,"context_line":"    cfg.StrOpt(\u0027cafile\u0027,"},{"line_number":42,"context_line":"               help\u003d\u0027A PEM encoded Certificate Authority to use when verifying\u0027"},{"line_number":43,"context_line":"               \u0027 HTTPs connections. Defaults to system CAs.\u0027),"}],"source_content_type":"text/x-python","patch_set":1,"id":"7f287b81_0e3de02b","line":40,"range":{"start_line":37,"start_character":0,"end_line":40,"end_character":79},"updated":"2017-09-01 00:59:59.000000000","message":"Should we add more specific help text? e.g.\n\ncertfile: cerfitication file, required if ...\nkeyfile: priviate key, required if ...\n\nThis is also for consistency with help text for \u0027cafile\u0027","commit_id":"b2adbe1f7e35db973017259dee5aa9f4f3e4fdc8"},{"author":{"_account_id":19134,"name":"Eyal","email":"eyalb1@gmail.com","username":"eyalb"},"change_message_id":"3ca02d287a19565066bfce18929cdfc37ef3b437","unresolved":false,"context_lines":[{"line_number":34,"context_line":"               help\u003d\u0027Keycloak authentication server ip\u0027,),"},{"line_number":35,"context_line":"    cfg.StrOpt(\u0027insecure\u0027, default\u003dFalse,"},{"line_number":36,"context_line":"               help\u003d\u0027If True, SSL/TLS certificate verification is disabled\u0027),"},{"line_number":37,"context_line":"    cfg.StrOpt(\u0027certfile\u0027,"},{"line_number":38,"context_line":"               help\u003d\u0027Required if identity server requires client certificate\u0027),"},{"line_number":39,"context_line":"    cfg.StrOpt(\u0027keyfile\u0027,"},{"line_number":40,"context_line":"               help\u003d\u0027Required if identity server requires client certificate\u0027),"},{"line_number":41,"context_line":"    cfg.StrOpt(\u0027cafile\u0027,"},{"line_number":42,"context_line":"               help\u003d\u0027A PEM encoded Certificate Authority to use when verifying\u0027"},{"line_number":43,"context_line":"               \u0027 HTTPs connections. Defaults to system CAs.\u0027),"}],"source_content_type":"text/x-python","patch_set":1,"id":"7f287b81_6851d862","line":40,"range":{"start_line":37,"start_character":0,"end_line":40,"end_character":79},"in_reply_to":"7f287b81_0e3de02b","updated":"2017-09-03 06:20:02.000000000","message":"I just copied it from two other projects that use keycloak\nI can change the text what it your suggestion ?","commit_id":"b2adbe1f7e35db973017259dee5aa9f4f3e4fdc8"},{"author":{"_account_id":21414,"name":"Yujun Zhang","email":"zhang.yujunz@zte.com.cn","username":"yujunz"},"change_message_id":"3c143275cef0ba48f3ef9280c21aa39399ca04f2","unresolved":false,"context_lines":[{"line_number":91,"context_line":"        endpoint \u003d OPENID_CONNECT_USERINFO % (self.auth_url, realm_name)"},{"line_number":92,"context_line":"        headers \u003d {\u0027Authorization\u0027: \u0027Bearer %s\u0027 % self.token}"},{"line_number":93,"context_line":""},{"line_number":94,"context_line":"        verify \u003d None"},{"line_number":95,"context_line":"        if urllib.parse.urlparse(endpoint).scheme \u003d\u003d \"https\":"},{"line_number":96,"context_line":"            verify \u003d False if self.insecure else self.cafile"},{"line_number":97,"context_line":""},{"line_number":98,"context_line":"        cert \u003d (self.certfile, self.keyfile) if self.certfile and self.keyfile \\"},{"line_number":99,"context_line":"            else None"},{"line_number":100,"context_line":""},{"line_number":101,"context_line":"        resp \u003d requests.get(endpoint, headers\u003dheaders,"},{"line_number":102,"context_line":"                            verify\u003dverify, cert\u003dcert)"},{"line_number":103,"context_line":""},{"line_number":104,"context_line":"        resp.raise_for_status()"},{"line_number":105,"context_line":""}],"source_content_type":"text/x-python","patch_set":1,"id":"7f287b81_6ef34475","line":102,"range":{"start_line":94,"start_character":0,"end_line":102,"end_character":53},"updated":"2017-09-01 00:59:59.000000000","message":"There seems to be dependency between insecure, certfile, keyfile and cafile. If insecure is False, and no certfile, keyfile or cafile is provided, will it work normally?","commit_id":"b2adbe1f7e35db973017259dee5aa9f4f3e4fdc8"},{"author":{"_account_id":19134,"name":"Eyal","email":"eyalb1@gmail.com","username":"eyalb"},"change_message_id":"3ca02d287a19565066bfce18929cdfc37ef3b437","unresolved":false,"context_lines":[{"line_number":91,"context_line":"        endpoint \u003d OPENID_CONNECT_USERINFO % (self.auth_url, realm_name)"},{"line_number":92,"context_line":"        headers \u003d {\u0027Authorization\u0027: \u0027Bearer %s\u0027 % self.token}"},{"line_number":93,"context_line":""},{"line_number":94,"context_line":"        verify \u003d None"},{"line_number":95,"context_line":"        if urllib.parse.urlparse(endpoint).scheme \u003d\u003d \"https\":"},{"line_number":96,"context_line":"            verify \u003d False if self.insecure else self.cafile"},{"line_number":97,"context_line":""},{"line_number":98,"context_line":"        cert \u003d (self.certfile, self.keyfile) if self.certfile and self.keyfile \\"},{"line_number":99,"context_line":"            else None"},{"line_number":100,"context_line":""},{"line_number":101,"context_line":"        resp \u003d requests.get(endpoint, headers\u003dheaders,"},{"line_number":102,"context_line":"                            verify\u003dverify, cert\u003dcert)"},{"line_number":103,"context_line":""},{"line_number":104,"context_line":"        resp.raise_for_status()"},{"line_number":105,"context_line":""}],"source_content_type":"text/x-python","patch_set":1,"id":"7f287b81_68d678c6","line":102,"range":{"start_line":94,"start_character":0,"end_line":102,"end_character":53},"in_reply_to":"7f287b81_6ef34475","updated":"2017-09-03 06:20:02.000000000","message":"I copied it from two other projects who use keycloak\nI think that if you use https you must provide them","commit_id":"b2adbe1f7e35db973017259dee5aa9f4f3e4fdc8"}]}