)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":17685,"name":"Elod Illes","email":"elod.illes@est.tech","username":"elod.illes"},"change_message_id":"26238557b899eeab40b31b67550427f539a88c8a","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"7f0fe247_9facbe05","updated":"2022-08-30 13:02:13.000000000","message":"Seems to be a proper fix for a trivial issue.","commit_id":"6c35b366e3c8c6d7f47471b93f5315582301c5ef"},{"author":{"_account_id":15554,"name":"Bence Romsics","email":"bence.romsics@gmail.com","username":"ebenrom","status":"working for Ericsson, UTC+1 (+DST)"},"change_message_id":"4c261bd296893b6851d8265a7f526cbb68f0ed0f","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"20e46cc5_9e48263f","updated":"2022-09-07 12:18:01.000000000","message":"recheck bug 1988986","commit_id":"6c35b366e3c8c6d7f47471b93f5315582301c5ef"}],"keystone/api/ec2tokens.py":[{"author":{"_account_id":17685,"name":"Elod Illes","email":"elod.illes@est.tech","username":"elod.illes"},"change_message_id":"26238557b899eeab40b31b67550427f539a88c8a","unresolved":false,"context_lines":[{"line_number":42,"context_line":"            # NOTE(vish): Some client libraries don\u0027t use the port when"},{"line_number":43,"context_line":"            # signing requests, so try again without the port."},{"line_number":44,"context_line":"            elif \u0027:\u0027 in credentials[\u0027host\u0027]:"},{"line_number":45,"context_line":"                hostname, _port \u003d credentials.split(\u0027:\u0027)"},{"line_number":46,"context_line":"                credentials[\u0027host\u0027] \u003d hostname"},{"line_number":47,"context_line":"                # NOTE(davechen): we need to reinitialize \u0027signer\u0027 to avoid"},{"line_number":48,"context_line":"                # contaminated status of signature, this is similar with"}],"source_content_type":"text/x-python","patch_set":1,"id":"ea49e2aa_30f6727f","side":"PARENT","line":45,"range":{"start_line":45,"start_character":34,"end_line":45,"end_character":45},"updated":"2022-08-30 13:02:13.000000000","message":"Yes, the intention was here to split credentials[\u0027host\u0027], in my opinion, too.","commit_id":"051aca8e8a488efc51817463dab8e4daafbbbf59"},{"author":{"_account_id":17685,"name":"Elod Illes","email":"elod.illes@est.tech","username":"elod.illes"},"change_message_id":"26238557b899eeab40b31b67550427f539a88c8a","unresolved":false,"context_lines":[{"line_number":45,"context_line":"            # signing requests, so try again without the port."},{"line_number":46,"context_line":"            elif \u0027:\u0027 in credentials[\u0027host\u0027]:"},{"line_number":47,"context_line":"                parsed \u003d urllib.parse.urlsplit(\u0027//\u0027 + credentials[\u0027host\u0027])"},{"line_number":48,"context_line":"                credentials[\u0027host\u0027] \u003d parsed.hostname"},{"line_number":49,"context_line":"                # NOTE(davechen): we need to reinitialize \u0027signer\u0027 to avoid"},{"line_number":50,"context_line":"                # contaminated status of signature, this is similar with"},{"line_number":51,"context_line":"                # other programming language libraries, JAVA for example."}],"source_content_type":"text/x-python","patch_set":1,"id":"4e15fee1_4a181a11","line":48,"updated":"2022-08-30 13:02:13.000000000","message":"this works as intended for every case, even for ipv6 hostname + port.","commit_id":"6c35b366e3c8c6d7f47471b93f5315582301c5ef"}]}
