)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":1669,"name":"Julien Danjou","display_name":"jd","email":"julien@danjou.info","username":"jdanjou"},"change_message_id":"0501010398a87d1e0a28016be10000d470d1f748","unresolved":false,"context_lines":[{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Add HTTP proxy support to ceilometer client"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"There is a need for HTTP Proxy support in Ceilometer client in the case when Ceilometer API is not directly available"},{"line_number":10,"context_line":"Proxy url is taken from environment variables http_proxy or https_proxy"},{"line_number":11,"context_line":""},{"line_number":12,"context_line":"Implements: blueprint proxy-support"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"AAAATn%2F%2BnR4%3D","line":9,"updated":"2013-11-28 09:59:24.000000000","message":"Could you truncate at 80 char in commit message too? :)","commit_id":"df66bb82c0a9df06e4054fdff1167f4f4c4107e5"},{"author":{"_account_id":7729,"name":"Ilya Tyaptin","email":"ityaptin@mirantis.com","username":"ityaptin"},"change_message_id":"258e15c3205ad6c067b6d1939ea9f09763b5f10c","unresolved":false,"context_lines":[{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Add HTTP proxy support to ceilometer client"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"There is a need for HTTP Proxy support in Ceilometer client in the case when Ceilometer API is not directly available"},{"line_number":10,"context_line":"Proxy url is taken from environment variables http_proxy or https_proxy"},{"line_number":11,"context_line":""},{"line_number":12,"context_line":"Implements: blueprint proxy-support"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"AAAATn%2F%2BOrY%3D","line":9,"in_reply_to":"AAAATn%2F%2BnR4%3D","updated":"2013-11-28 15:52:14.000000000","message":"Done","commit_id":"df66bb82c0a9df06e4054fdff1167f4f4c4107e5"}],"ceilometerclient/common/http.py":[{"author":{"_account_id":7478,"name":"Nadya Shakhat","email":"nadmi4@gmail.com","username":"nprivalova"},"change_message_id":"b907247a97424dda5c913be35259e68cad33f4c5","unresolved":false,"context_lines":[{"line_number":75,"context_line":"        try:"},{"line_number":76,"context_line":""},{"line_number":77,"context_line":"            if self.proxy_url:"},{"line_number":78,"context_line":"                proxy_parts \u003d urlutils.urlparse(self.proxy_url)"},{"line_number":79,"context_line":"                return _class(proxy_parts.host, proxy_parts.host,"},{"line_number":80,"context_line":"                              **self.connection_params[2])"},{"line_number":81,"context_line":"            else:"}],"source_content_type":"text/x-python","patch_set":1,"id":"AAAATn%2F%2Bnxw%3D","line":78,"updated":"2013-11-22 10:58:15.000000000","message":"Why host is passed two times?","commit_id":"5ecba60236c90ba744c354244b7be376fe6a7e1e"},{"author":{"_account_id":7729,"name":"Ilya Tyaptin","email":"ityaptin@mirantis.com","username":"ityaptin"},"change_message_id":"7389fdec2e39db3eb6366580f41689b45f7f43bd","unresolved":false,"context_lines":[{"line_number":75,"context_line":"        try:"},{"line_number":76,"context_line":""},{"line_number":77,"context_line":"            if self.proxy_url:"},{"line_number":78,"context_line":"                proxy_parts \u003d urlutils.urlparse(self.proxy_url)"},{"line_number":79,"context_line":"                return _class(proxy_parts.host, proxy_parts.host,"},{"line_number":80,"context_line":"                              **self.connection_params[2])"},{"line_number":81,"context_line":"            else:"}],"source_content_type":"text/x-python","patch_set":1,"id":"AAAATn%2F%2BnwI%3D","line":78,"in_reply_to":"AAAATn%2F%2Bnxw%3D","updated":"2013-11-22 11:02:31.000000000","message":"Done","commit_id":"5ecba60236c90ba744c354244b7be376fe6a7e1e"},{"author":{"_account_id":7478,"name":"Nadya Shakhat","email":"nadmi4@gmail.com","username":"nprivalova"},"change_message_id":"8885c99096f55acd46c446c310fabdbe7b0e1519","unresolved":false,"context_lines":[{"line_number":142,"context_line":"        conn \u003d self.get_connection()"},{"line_number":143,"context_line":""},{"line_number":144,"context_line":"        try:"},{"line_number":145,"context_line":"            if self.proxy_url:"},{"line_number":146,"context_line":"                conn_url \u003d self.endpoint + self._make_connection_url(url)"},{"line_number":147,"context_line":"            else:"},{"line_number":148,"context_line":"                conn_url \u003d self._make_connection_url(url)"}],"source_content_type":"text/x-python","patch_set":1,"id":"AAAATn%2F%2BnuM%3D","line":145,"updated":"2013-11-22 11:09:01.000000000","message":"why self.proxy_url doesn\u0027t participate in conn_url construction?","commit_id":"5ecba60236c90ba744c354244b7be376fe6a7e1e"},{"author":{"_account_id":7729,"name":"Ilya Tyaptin","email":"ityaptin@mirantis.com","username":"ityaptin"},"change_message_id":"f89f93137310c383ef3c4ecc7b0d58b6e2db2651","unresolved":false,"context_lines":[{"line_number":142,"context_line":"        conn \u003d self.get_connection()"},{"line_number":143,"context_line":""},{"line_number":144,"context_line":"        try:"},{"line_number":145,"context_line":"            if self.proxy_url:"},{"line_number":146,"context_line":"                conn_url \u003d self.endpoint + self._make_connection_url(url)"},{"line_number":147,"context_line":"            else:"},{"line_number":148,"context_line":"                conn_url \u003d self._make_connection_url(url)"}],"source_content_type":"text/x-python","patch_set":1,"id":"AAAATn%2F%2BnsU%3D","line":145,"in_reply_to":"AAAATn%2F%2BnuM%3D","updated":"2013-11-22 11:17:25.000000000","message":"We initialized connection in get_connection() with proxy and now requests are sent to conn_url through it","commit_id":"5ecba60236c90ba744c354244b7be376fe6a7e1e"},{"author":{"_account_id":7478,"name":"Nadya Shakhat","email":"nadmi4@gmail.com","username":"nprivalova"},"change_message_id":"8885c99096f55acd46c446c310fabdbe7b0e1519","unresolved":false,"context_lines":[{"line_number":210,"context_line":"                                     \u0027application/octet-stream\u0027)"},{"line_number":211,"context_line":"        return self._http_request(url, method, **kwargs)"},{"line_number":212,"context_line":""},{"line_number":213,"context_line":"    def get_proxy_url(self, endpoint):"},{"line_number":214,"context_line":"        scheme \u003d urlutils.urlparse(endpoint).scheme"},{"line_number":215,"context_line":"        if scheme \u003d\u003d \u0027https\u0027:"},{"line_number":216,"context_line":"            return os.environ.get(\u0027https_proxy\u0027)"}],"source_content_type":"text/x-python","patch_set":1,"id":"AAAATn%2F%2Bnu0%3D","line":213,"updated":"2013-11-22 11:09:01.000000000","message":"Please take a look into get_connection_params. I think you should throw exception in case of unsupported schema","commit_id":"5ecba60236c90ba744c354244b7be376fe6a7e1e"},{"author":{"_account_id":7729,"name":"Ilya Tyaptin","email":"ityaptin@mirantis.com","username":"ityaptin"},"change_message_id":"f89f93137310c383ef3c4ecc7b0d58b6e2db2651","unresolved":false,"context_lines":[{"line_number":210,"context_line":"                                     \u0027application/octet-stream\u0027)"},{"line_number":211,"context_line":"        return self._http_request(url, method, **kwargs)"},{"line_number":212,"context_line":""},{"line_number":213,"context_line":"    def get_proxy_url(self, endpoint):"},{"line_number":214,"context_line":"        scheme \u003d urlutils.urlparse(endpoint).scheme"},{"line_number":215,"context_line":"        if scheme \u003d\u003d \u0027https\u0027:"},{"line_number":216,"context_line":"            return os.environ.get(\u0027https_proxy\u0027)"}],"source_content_type":"text/x-python","patch_set":1,"id":"AAAATn%2F%2BnsQ%3D","line":213,"in_reply_to":"AAAATn%2F%2Bnu0%3D","updated":"2013-11-22 11:17:25.000000000","message":"Done","commit_id":"5ecba60236c90ba744c354244b7be376fe6a7e1e"}]}