)]}'
{"openstackclient/compute/v2/server.py":[{"author":{"_account_id":15334,"name":"Stephen Finucane","display_name":"stephenfin","email":"stephenfin@redhat.com","username":"sfinucan"},"change_message_id":"73967de49eda3bac546f3c6c6a20e1434b8762c8","unresolved":true,"context_lines":[{"line_number":776,"context_line":""},{"line_number":777,"context_line":"        if values.startswith(\u0027file://\u0027):"},{"line_number":778,"context_line":"            path \u003d urllib.parse.urlparse(values).path"},{"line_number":779,"context_line":"            with open(path) as fh:"},{"line_number":780,"context_line":"                data \u003d json.load(fh)"},{"line_number":781,"context_line":""},{"line_number":782,"context_line":"            # Validate the keys - other validation is left to later"}],"source_content_type":"text/x-python","patch_set":1,"id":"45775693_a2e7b0f3","line":779,"updated":"2021-03-05 13:17:58.000000000","message":"I\u0027m still not sure if we want to do this or bog standard paths, replacing the check above with \u0027os.path.exists\u0027. If we keep with this, perhaps I should allow remote URIs also and replace \u0027open\u0027 with \u0027urllib2.urlopen\u0027. This will currently break with e.g. \u0027file://localhost/tmp/sample.json\u0027.","commit_id":"d3bd0146ae64ebc3a7c5d7a3a2fc90efe4071495"}]}