)]}'
{"nodepool/nodeutils.py":[{"author":{"_account_id":4146,"name":"Clark Boylan","email":"cboylan@sapwetik.org","username":"cboylan"},"change_message_id":"b6852005ffd62a29640052fab56a8a9440ee23d5","unresolved":true,"context_lines":[{"line_number":89,"context_line":"            if gather_hostkeys:"},{"line_number":90,"context_line":"                t \u003d paramiko.transport.Transport(sock)"},{"line_number":91,"context_line":"                t.start_client(timeout\u003dtimeout)"},{"line_number":92,"context_line":"                t.close()"},{"line_number":93,"context_line":"            break"},{"line_number":94,"context_line":"        except socket.error as e:"},{"line_number":95,"context_line":"            if e.errno not in [errno.ECONNREFUSED, errno.EHOSTUNREACH, None]:"}],"source_content_type":"text/x-python","patch_set":1,"id":"f93db73c_fa394764","line":92,"updated":"2021-07-18 23:20:04.000000000","message":"I think it might be better to put:\n\n  if t:\n      t.close()\n\nin the finally block below rather than closing it here. That way all the error handling is consistent.","commit_id":"9c8de6b156c3fbbb9e767150d1afb8620b7f9fad"}]}