Fabio/ddclient
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

Check, don't assume, that --use* names a firewall

Browse source
This commit is contained in:
Richard Hansen 2024-03-16 15:07:37 -07:00
parent 474cc76587
commit ee5bb2de90
1 changed files with 18 additions and 12 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

30
ddclient.in
View file

@ -2893,15 +2893,14 @@ sub get_ip {
## This is a no-op... Do not get an IP address for this host/service ## This is a no-op... Do not get an IP address for this host/service
$reply = ''; $reply = '';
} else { } elsif ($use eq 'fw' || defined(my $fw = $builtinfw{$use})) {
# Note that --use=firewallname uses --fw=arg, not --firewallname=arg. # Note that --use=firewallname uses --fw=arg, not --firewallname=arg.
$arg = opt('fw', $h) // ''; $arg = opt('fw', $h) // '';
$url = $arg; $url = $arg;
$skip = opt('fw-skip', $h); $skip = opt('fw-skip', $h);
if ($fw) {
if (exists $builtinfw{$use}) { $skip //= $fw->{'skip'};
$skip //= $builtinfw{$use}->{'skip'}; $url = "http://$url$fw->{'url'}" unless $url =~ /\//;
$url = "http://${url}" . $builtinfw{$use}->{'url'} unless $url =~ /\//;
} }
if ($url) { if ($url) {
@ -2913,6 +2912,9 @@ sub get_ip {
ssl_validate => opt('fw-ssl-validate', $h), ssl_validate => opt('fw-ssl-validate', $h),
) // ''; ) // '';
} }
} else {
warning("ignoring unsupported '--use=$use'");
} }
if (!defined $reply) { if (!defined $reply) {
$reply = ''; $reply = '';
@ -3321,7 +3323,7 @@ sub get_ipv4 {
## This is a no-op... Do not get an IPv4 address for this host/service ## This is a no-op... Do not get an IPv4 address for this host/service
$reply = ''; $reply = '';
} else { } elsif ($usev4 eq 'fwv4' || defined(my $fw = $builtinfw{$usev4})) {
warning("'--fw' is deprecated for '--usev4=$usev4'; use '--fwv4' instead") warning("'--fw' is deprecated for '--usev4=$usev4'; use '--fwv4' instead")
if (!defined(opt('fwv4', $h)) && defined(opt('fw', $h))); if (!defined(opt('fwv4', $h)) && defined(opt('fw', $h)));
warning("'--fw-skip' is deprecated for '--usev4=$usev4'; use '--fwv4-skip' instead") warning("'--fw-skip' is deprecated for '--usev4=$usev4'; use '--fwv4-skip' instead")
@ -3330,10 +3332,9 @@ sub get_ipv4 {
$arg = opt('fwv4', $h) // opt('fw', $h) // ''; $arg = opt('fwv4', $h) // opt('fw', $h) // '';
$url = $arg; $url = $arg;
$skip = opt('fwv4-skip', $h) // opt('fw-skip', $h); $skip = opt('fwv4-skip', $h) // opt('fw-skip', $h);
if ($fw) {
if (exists $builtinfw{$usev4}) { $skip //= $fw->{'skip'};
$skip //= $builtinfw{$usev4}->{'skip'}; $url = "http://$url$fw->{'url'}" unless $url =~ /\//;
$url = "http://${url}" . $builtinfw{$usev4}->{'url'} unless $url =~ /\//;
} }
if ($url) { if ($url) {
$reply = geturl( $reply = geturl(
@ -3345,6 +3346,9 @@ sub get_ipv4 {
ssl_validate => opt('fw-ssl-validate', $h), ssl_validate => opt('fw-ssl-validate', $h),
) // ''; ) // '';
} }
} else {
warning("ignoring unsupported '--usev4=$usev4'");
} }
## Set to loopback address if no text set yet ## Set to loopback address if no text set yet
@ -3438,9 +3442,11 @@ sub get_ipv6 {
} elsif ($usev6 eq 'disabled') { } elsif ($usev6 eq 'disabled') {
$reply = ''; $reply = '';
} else { } elsif ($usev6 eq 'fwv6' || defined($builtinfw{$usev6})) {
warning("'--usev6=%s' is not implemented and does nothing", $usev6); warning("'--usev6=%s' is not implemented and does nothing", $usev6);
$reply = '';
} else {
warning("ignoring unsupported '--usev6=$usev6'");
} }