)]}'
{"keystoneclient/session.py":[{"author":{"_account_id":13055,"name":"Alexander Makarov","email":"ajiekcahdp.makapob@gmail.com","username":"amakarov"},"change_message_id":"ce37d0b5a212dbb4c2491c6081fc7367a2982c17","unresolved":false,"context_lines":[{"line_number":139,"context_line":"            # debug log."},{"line_number":140,"context_line":"            return"},{"line_number":141,"context_line":""},{"line_number":142,"context_line":"        string_parts \u003d [\u0027REQ: curl -g -i\u0027]"},{"line_number":143,"context_line":""},{"line_number":144,"context_line":"        # NOTE(jamielennox): None means let requests do its default validation"},{"line_number":145,"context_line":"        # so we need to actually check that this is False."}],"source_content_type":"text/x-python","patch_set":2,"id":"5a890539_a7ca4318","line":142,"updated":"2014-11-27 10:13:06.000000000","message":"-g from man page: \"When you set this option, you can specify URLs that contain the letters {}[] without having them being interpreted by curl itself\"\nIs there any validity check? If necessary, of course.","commit_id":"c859cb71c2236a8aa3a59b9ce07ce8e60d763254"},{"author":{"_account_id":10257,"name":"Sridhar Gaddam","email":"sgaddam@redhat.com","username":"sridhargaddam"},"change_message_id":"5198ef008d810231d7f776a8e358c92dee925c43","unresolved":false,"context_lines":[{"line_number":139,"context_line":"            # debug log."},{"line_number":140,"context_line":"            return"},{"line_number":141,"context_line":""},{"line_number":142,"context_line":"        string_parts \u003d [\u0027REQ: curl -g -i\u0027]"},{"line_number":143,"context_line":""},{"line_number":144,"context_line":"        # NOTE(jamielennox): None means let requests do its default validation"},{"line_number":145,"context_line":"        # so we need to actually check that this is False."}],"source_content_type":"text/x-python","patch_set":2,"id":"5a890539_e775f9cc","line":142,"in_reply_to":"5a890539_274aa155","updated":"2014-12-01 02:17:42.000000000","message":"Agree DNS can be used, and with DNS this problem will not be seen. However, since we also allow users to use IP addresses, this change would be necessary.","commit_id":"c859cb71c2236a8aa3a59b9ce07ce8e60d763254"},{"author":{"_account_id":10257,"name":"Sridhar Gaddam","email":"sgaddam@redhat.com","username":"sridhargaddam"},"change_message_id":"00ceaecb7cf206c323accc6bb971dccf056884dc","unresolved":false,"context_lines":[{"line_number":139,"context_line":"            # debug log."},{"line_number":140,"context_line":"            return"},{"line_number":141,"context_line":""},{"line_number":142,"context_line":"        string_parts \u003d [\u0027REQ: curl -g -i\u0027]"},{"line_number":143,"context_line":""},{"line_number":144,"context_line":"        # NOTE(jamielennox): None means let requests do its default validation"},{"line_number":145,"context_line":"        # so we need to actually check that this is False."}],"source_content_type":"text/x-python","patch_set":2,"id":"5a890539_ba3fba86","line":142,"in_reply_to":"5a890539_a7ca4318","updated":"2014-11-27 10:30:59.000000000","message":"@Alexander, when we configure OpenStack services to bind on IPv6 addresses and execute any client commands (ie.., keystone, neutron etc) with --debug option, as you know, keystone displays a \"debug\" curl equivalent message as below.\n\nkeystone --debug tenant-list\nDEBUG:keystoneclient.auth.identity.v2:Making authentication request to http://[::1]:5000/v2.0/tokens\nINFO:urllib3.connectionpool:Starting new HTTP connection (1): ::1\nDEBUG:urllib3.connectionpool:Setting read timeout to 600.0\nDEBUG:urllib3.connectionpool:\"POST /v2.0/tokens HTTP/1.1\" 200 3360\nDEBUG:keystoneclient.session:REQ: curl -i -X GET http://[::1]:35357/v2.0/tenants -H \"User-Agent: python-keystoneclient\" -H \"X-Auth-Token: {SHA1}ffe380043d26eb631ddd19e38fb62bb45adccc69\"\n\nNow, if we run the curl statement, curl displays the following error message.\n#curl -i -X GET http://[::1]:35357/v2.0/tenants -H \"User-Agent: python-keystoneclient\ncurl: (3) [globbing] bd range specification in column 9\n\nSince Curl is not used directly by Keystone/other-clients and is only used for displaying the http-request it would be an overkill to parse the URL for \u0027{}/[]\u0027 and use \"-g\" in the debug log. Also there is no harm in including -g option even for IPv4 urls. Hence including -g option is an optimal solution IMHO.","commit_id":"c859cb71c2236a8aa3a59b9ce07ce8e60d763254"},{"author":{"_account_id":6486,"name":"Brant Knudson","email":"blk@acm.org","username":"blk-u"},"change_message_id":"40bdcd76bde3ab5fcf88d8a374fa2884194262dc","unresolved":false,"context_lines":[{"line_number":139,"context_line":"            # debug log."},{"line_number":140,"context_line":"            return"},{"line_number":141,"context_line":""},{"line_number":142,"context_line":"        string_parts \u003d [\u0027REQ: curl -g -i\u0027]"},{"line_number":143,"context_line":""},{"line_number":144,"context_line":"        # NOTE(jamielennox): None means let requests do its default validation"},{"line_number":145,"context_line":"        # so we need to actually check that this is False."}],"source_content_type":"text/x-python","patch_set":2,"id":"5a890539_274aa155","line":142,"in_reply_to":"5a890539_ba3fba86","updated":"2014-11-30 23:55:00.000000000","message":"why use IPv6 addresses? Isn\u0027t this what DNS is for?","commit_id":"c859cb71c2236a8aa3a59b9ce07ce8e60d763254"}]}