Skip to content

Commit

Permalink
tests: Use correct arguments to curl.
Browse files Browse the repository at this point in the history
The commit in the fixes tag erroneously changed the
`--disable-epsv` argument to `--disable-eps`, which does
not exist.

Curiously the version of curl in the GitHub Actions images
appears to accept the abbreviated option, whereas more recent
versions of curl will fail with a hard error code 2.

Fixes: 249a9b5 ("system-traffic: Fix syntax errors in FTP and IPv6 curl calls.")
Signed-off-by: Frode Nordahl <[email protected]>
Signed-off-by: Eelco Chaudron <[email protected]>
  • Loading branch information
fnordahl authored and chaudron committed Feb 17, 2025
1 parent a1f23e0 commit 8303441
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion tests/system-common-macros.at
Original file line number Diff line number Diff line change
Expand Up @@ -280,7 +280,7 @@ m4_define([OVS_GET_HTTP],
#
m4_define([OVS_GET_FTP],
[curl ftp://$1 --retry 3 --max-time 1 --retry-connrefused \
--disable-eps -v $2]
--disable-epsv -v $2]
)

# OVS_GET_FTP_ACTIVE([url], [optional_curl_arguments])
Expand Down

0 comments on commit 8303441

Please sign in to comment.