)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":20190,"name":"zhufl","email":"zhu.fanglei@zte.com.cn","username":"zhufl"},"change_message_id":"69118c1f586681b8eade9363bfc2318c901146be","unresolved":false,"context_lines":[{"line_number":9,"context_line":"If the VNC is running at a default port and the address returned"},{"line_number":10,"context_line":"is an address which does not contain a port, then the VNC tests"},{"line_number":11,"context_line":"fail to connect because they attempt to call a socket connect"},{"line_number":12,"context_line":"without an IP listed."},{"line_number":13,"context_line":""},{"line_number":14,"context_line":"In addition, this makes sure that the Host header does not include"},{"line_number":15,"context_line":"a port if we are hitting the default port so things are clean."}],"source_content_type":"text/x-gerrit-commit-message","patch_set":6,"id":"df7087c5_04af377d","line":12,"range":{"start_line":12,"start_character":11,"end_line":12,"end_character":13},"updated":"2018-03-14 05:58:54.000000000","message":"Do you mean port?","commit_id":"aa5dd9a3cb19f9ddcd8d6b7a8c63bd0443e08a11"}],"tempest/common/compute.py":[{"author":{"_account_id":6167,"name":"Ken\u0027ichi Ohmichi","email":"ken1ohmichi@gmail.com","username":"oomichi"},"change_message_id":"a54d208bb330ba65a54f505a9574e86442f72ffa","unresolved":false,"context_lines":[{"line_number":292,"context_line":""},{"line_number":293,"context_line":"    # NOTE(mnaser): It is possible that there is no port specified, so fall"},{"line_number":294,"context_line":"    #               back to the default port based on the scheme."},{"line_number":295,"context_line":"    port \u003d url.port or (443 if url.scheme \u003d\u003d \u0027https\u0027 else 80)"},{"line_number":296,"context_line":""},{"line_number":297,"context_line":"    for res in socket.getaddrinfo(url.hostname, port,"},{"line_number":298,"context_line":"                                  socket.AF_UNSPEC, socket.SOCK_STREAM):"}],"source_content_type":"text/x-python","patch_set":6,"id":"df7087c5_d79a2160","line":295,"updated":"2018-03-15 16:57:26.000000000","message":"I guessed this kind thing can be covered on the library side of urlparse, but the library doesn\u0027t like\n\n \u003e\u003e\u003e import urlparse\n \u003e\u003e\u003e url \u003d urlparse.urlparse(\u0027http://localhost.com\u0027)\n \u003e\u003e\u003e url.hostname\n \u0027localhost.com\u0027\n \u003e\u003e\u003e url.port\n \u003e\u003e\u003e\n\nSo LGTM","commit_id":"aa5dd9a3cb19f9ddcd8d6b7a8c63bd0443e08a11"}]}