)]}'
{"mistralclient/api/httpclient.py":[{"author":{"_account_id":7700,"name":"Nikolay Mahotkin","email":"nikolay.makhotkin@gmail.com","username":"nmakhotkin"},"change_message_id":"98ad764b98eea2edc182e0ac7aee530490da9932","unresolved":false,"context_lines":[{"line_number":57,"context_line":"    def get(self, url, headers\u003dNone):"},{"line_number":58,"context_line":"        headers \u003d self._update_headers(headers)"},{"line_number":59,"context_line":""},{"line_number":60,"context_line":"        options \u003d copy.deepcopy(self.ssl_options)"},{"line_number":61,"context_line":"        options[\u0027headers\u0027] \u003d headers"},{"line_number":62,"context_line":""},{"line_number":63,"context_line":"        return requests.get(self.base_url + url, **options)"},{"line_number":64,"context_line":""}],"source_content_type":"text/x-python","patch_set":1,"id":"7af24918_252af02c","line":61,"range":{"start_line":60,"start_character":8,"end_line":61,"end_character":36},"updated":"2016-03-02 07:35:13.000000000","message":"Could it be inserted in _update_headers method? Or, put it under another method to call inside _update_headers? That way we can avoid duplication of these 2 lines in every request method.","commit_id":"592adc97bffd1930ba3932af7326ca8e0a061376"},{"author":{"_account_id":6732,"name":"Lingxian Kong","email":"anlin.kong@gmail.com","username":"kong"},"change_message_id":"4e75010f919195171e74044e90c54d3ef0d1b39e","unresolved":false,"context_lines":[{"line_number":57,"context_line":"    def get(self, url, headers\u003dNone):"},{"line_number":58,"context_line":"        headers \u003d self._update_headers(headers)"},{"line_number":59,"context_line":""},{"line_number":60,"context_line":"        options \u003d copy.deepcopy(self.ssl_options)"},{"line_number":61,"context_line":"        options[\u0027headers\u0027] \u003d headers"},{"line_number":62,"context_line":""},{"line_number":63,"context_line":"        return requests.get(self.base_url + url, **options)"},{"line_number":64,"context_line":""}],"source_content_type":"text/x-python","patch_set":1,"id":"7af24918_e03593da","line":61,"range":{"start_line":60,"start_character":8,"end_line":61,"end_character":36},"in_reply_to":"7af24918_252af02c","updated":"2016-03-02 08:52:00.000000000","message":"Yes, I am inclined to agree with Nekolay.","commit_id":"592adc97bffd1930ba3932af7326ca8e0a061376"}],"mistralclient/api/v2/client.py":[{"author":{"_account_id":10126,"name":"manas kelshikar","email":"manas@stackstorm.com","username":"manasdk"},"change_message_id":"32f5546ac513637531a0c2f6ef9fbc748d6e73ec","unresolved":false,"context_lines":[{"line_number":64,"context_line":"            project_id,"},{"line_number":65,"context_line":"            user_id,"},{"line_number":66,"context_line":"            cacert\u003dcacert,"},{"line_number":67,"context_line":"            insecure\u003dinsecure"},{"line_number":68,"context_line":"        )"},{"line_number":69,"context_line":""},{"line_number":70,"context_line":"        # Create all resource managers."}],"source_content_type":"text/x-python","patch_set":3,"id":"7af24918_2f0c1f3a","line":67,"updated":"2016-03-03 15:11:50.000000000","message":"cacert and insecure setting will be same for both mistral-api server and keystone api server?","commit_id":"7e9a4e8a48cba6f674873563f44d79cb9cbbeef5"},{"author":{"_account_id":5558,"name":"Winson Chan","email":"wcchan@stackstorm.com","username":"winson-c-chan"},"change_message_id":"96507d6fa046b7234391d15c298679e5265697eb","unresolved":false,"context_lines":[{"line_number":64,"context_line":"            project_id,"},{"line_number":65,"context_line":"            user_id,"},{"line_number":66,"context_line":"            cacert\u003dcacert,"},{"line_number":67,"context_line":"            insecure\u003dinsecure"},{"line_number":68,"context_line":"        )"},{"line_number":69,"context_line":""},{"line_number":70,"context_line":"        # Create all resource managers."}],"source_content_type":"text/x-python","patch_set":3,"id":"7af24918_c9248d0e","line":67,"in_reply_to":"7af24918_2f0c1f3a","updated":"2016-03-03 17:49:33.000000000","message":"For our use case since we own both keystone and mistral, my assumption is that they will be setup the same.  But if there\u0027s a user requirement to separate the two, we can separate the cacert and insecure options between keystone and mistral.","commit_id":"7e9a4e8a48cba6f674873563f44d79cb9cbbeef5"},{"author":{"_account_id":10126,"name":"manas kelshikar","email":"manas@stackstorm.com","username":"manasdk"},"change_message_id":"f7de10a84698852551732add3f747ec670fca8c7","unresolved":false,"context_lines":[{"line_number":64,"context_line":"            project_id,"},{"line_number":65,"context_line":"            user_id,"},{"line_number":66,"context_line":"            cacert\u003dcacert,"},{"line_number":67,"context_line":"            insecure\u003dinsecure"},{"line_number":68,"context_line":"        )"},{"line_number":69,"context_line":""},{"line_number":70,"context_line":"        # Create all resource managers."}],"source_content_type":"text/x-python","patch_set":3,"id":"7af24918_804d329c","line":67,"in_reply_to":"7af24918_c9248d0e","updated":"2016-03-03 19:24:36.000000000","message":"Not a requirement for my use-case. I suspect in most cases there will be a proper web server front ending both the APIs. If other deployments have this usecase then it can be addressed at that point.\n\nAlso, I assume most folks either properly install the certs on the box or do not use self signed therefore this is definitely a corner case.","commit_id":"7e9a4e8a48cba6f674873563f44d79cb9cbbeef5"}]}