)]}'
{"cinder/volume/drivers/coprhd/helpers/authentication.py":[{"author":{"_account_id":6491,"name":"xing-yang","email":"xingyang105@gmail.com","username":"xing-yang"},"change_message_id":"cee663231f48cc155dd7762a6ec1e86f090b2e90","unresolved":false,"context_lines":[{"line_number":64,"context_line":"        try:"},{"line_number":65,"context_line":"            if self.port \u003d\u003d APISVC_PORT:"},{"line_number":66,"context_line":"                login_response \u003d requests.get("},{"line_number":67,"context_line":"                    url, headers\u003dself.HEADERS, verify\u003dFalse,  # nosec"},{"line_number":68,"context_line":"                    # should be verified by driver implementer"},{"line_number":69,"context_line":"                    auth\u003d(username, password), cookies\u003dcookiejar,"},{"line_number":70,"context_line":"                    allow_redirects\u003dFalse, timeout\u003dcommon.TIMEOUT_SEC)"}],"source_content_type":"text/x-python","patch_set":1,"id":"3a461143_1f62973e","line":67,"updated":"2017-01-27 22:10:14.000000000","message":"CoprHD is fixing this in this patch: https://review.openstack.org/#/c/416236/","commit_id":"0013aa143eb63b12ed3459ca067b4c6bcf2fa673"},{"author":{"_account_id":6491,"name":"xing-yang","email":"xingyang105@gmail.com","username":"xing-yang"},"change_message_id":"cee663231f48cc155dd7762a6ec1e86f090b2e90","unresolved":false,"context_lines":[{"line_number":65,"context_line":"            if self.port \u003d\u003d APISVC_PORT:"},{"line_number":66,"context_line":"                login_response \u003d requests.get("},{"line_number":67,"context_line":"                    url, headers\u003dself.HEADERS, verify\u003dFalse,  # nosec"},{"line_number":68,"context_line":"                    # should be verified by driver implementer"},{"line_number":69,"context_line":"                    auth\u003d(username, password), cookies\u003dcookiejar,"},{"line_number":70,"context_line":"                    allow_redirects\u003dFalse, timeout\u003dcommon.TIMEOUT_SEC)"},{"line_number":71,"context_line":"                if login_response.status_code \u003d\u003d SEC_REDIRECT:"}],"source_content_type":"text/x-python","patch_set":1,"id":"3a461143_9fdda7e2","line":68,"updated":"2017-01-27 22:10:14.000000000","message":"I don\u0027t think this comment is necessary.  Driver sets this to False mostly because they don\u0027t have this support on the storage backend yet.","commit_id":"0013aa143eb63b12ed3459ca067b4c6bcf2fa673"},{"author":{"_account_id":23674,"name":"Rohan Arora","email":"ra271w@att.com","username":"rarora"},"change_message_id":"adcf2380b2111833d21ea3cd3f7f2b881cc92e7b","unresolved":false,"context_lines":[{"line_number":65,"context_line":"            if self.port \u003d\u003d APISVC_PORT:"},{"line_number":66,"context_line":"                login_response \u003d requests.get("},{"line_number":67,"context_line":"                    url, headers\u003dself.HEADERS, verify\u003dFalse,  # nosec"},{"line_number":68,"context_line":"                    # should be verified by driver implementer"},{"line_number":69,"context_line":"                    auth\u003d(username, password), cookies\u003dcookiejar,"},{"line_number":70,"context_line":"                    allow_redirects\u003dFalse, timeout\u003dcommon.TIMEOUT_SEC)"},{"line_number":71,"context_line":"                if login_response.status_code \u003d\u003d SEC_REDIRECT:"}],"source_content_type":"text/x-python","patch_set":1,"id":"3a461143_02515ed8","line":68,"in_reply_to":"3a461143_9fdda7e2","updated":"2017-01-27 22:21:57.000000000","message":"Okay, I can remove this comment on all of the #nosec\u0027s since most legitimately do not have support for this feature.","commit_id":"0013aa143eb63b12ed3459ca067b4c6bcf2fa673"}]}
