dnsexit2: Whitespace fixes

This commit is contained in:
Richard Hansen 2024-05-27 18:10:55 -04:00
parent a91ca7a199
commit 0040fc9608

View file

@ -4176,14 +4176,14 @@ sub nic_dnsexit2_update {
for my $h (@_) { for my $h (@_) {
# All the known status # All the known status
my %status = ( my %status = (
'0' => [ 'good', 'Success! Actions got executed successfully.' ], '0' => ['good', 'Success! Actions got executed successfully.'],
'1' => [ 'warning', 'Some execution problems. May not indicate actions failures. Some action may got executed fine and some may have problems.' ], '1' => ['warning', 'Some execution problems. May not indicate actions failures. Some action may got executed fine and some may have problems.'],
'2' => [ 'badauth', 'API Key Authentication Error. The API Key is missing or wrong.' ], '2' => ['badauth', 'API Key Authentication Error. The API Key is missing or wrong.'],
'3' => [ 'error', 'Missing Required Definitions. Your JSON file may missing some required definitions.' ], '3' => ['error', 'Missing Required Definitions. Your JSON file may missing some required definitions.'],
'4' => [ 'error', 'JSON Data Syntax Error. Your JSON file has syntax error.' ], '4' => ['error', 'JSON Data Syntax Error. Your JSON file has syntax error.'],
'5' => [ 'error', 'JSON Defined Record Type not Supported. Your JSON may try to update some record type not supported by our system.' ], '5' => ['error', 'JSON Defined Record Type not Supported. Your JSON may try to update some record type not supported by our system.'],
'6' => [ 'error', 'System Error. Our system problem. May not be your problem. Contact our support if you got such error.' ], '6' => ['error', 'System Error. Our system problem. May not be your problem. Contact our support if you got such error.'],
'7' => [ 'error', 'Error getting post data. Our server has problem to receive your JSON posting.' ], '7' => ['error', 'Error getting post data. Our server has problem to receive your JSON posting.'],
); );
my $ipv4 = delete $config{$h}{'wantipv4'}; my $ipv4 = delete $config{$h}{'wantipv4'};
my $ipv6 = delete $config{$h}{'wantipv6'}; my $ipv6 = delete $config{$h}{'wantipv6'};
@ -4191,7 +4191,7 @@ sub nic_dnsexit2_update {
# Updates for ipv4 and ipv6 need to be combined in a single API call, create Hash of Arrays for tracking # Updates for ipv4 and ipv6 need to be combined in a single API call, create Hash of Arrays for tracking
my %total_payload; my %total_payload;
for my $ip ($ipv4, $ipv6){ for my $ip ($ipv4, $ipv6) {
next if (!$ip); next if (!$ip);
my $ipv = ($ip eq ($ipv6 // '')) ? '6' : '4'; my $ipv = ($ip eq ($ipv6 // '')) ? '6' : '4';
my $type = ($ip eq ($ipv6 // '')) ? 'AAAA' : 'A'; my $type = ($ip eq ($ipv6 // '')) ? 'AAAA' : 'A';
@ -4212,7 +4212,7 @@ sub nic_dnsexit2_update {
my $header = "Content-Type: application/json\nAccept: application/json"; my $header = "Content-Type: application/json\nAccept: application/json";
# Set the zone if empty # Set the zone if empty
if ( not defined $config{$h}{'zone'}){ if (not defined $config{$h}{'zone'}) {
debug("Zone not defined, setting to default hostname: %s", $h); debug("Zone not defined, setting to default hostname: %s", $h);
$config{$h}{'zone'} = $h $config{$h}{'zone'} = $h
} else { } else {
@ -4237,7 +4237,7 @@ sub nic_dnsexit2_update {
); );
# No reply, declare as failed # No reply, declare as failed
unless ($reply && header_ok($h, $reply)){ unless ($reply && header_ok($h, $reply)) {
failed("updating %s: Could not connect to %s%s.", $h, $config{$h}{'server'}, $config{$h}{'path'}); failed("updating %s: Could not connect to %s%s.", $h, $config{$h}{'server'}, $config{$h}{'path'});
last; last;
}; };
@ -4249,7 +4249,7 @@ sub nic_dnsexit2_update {
debug("HTTP response code: %s", $http_status); debug("HTTP response code: %s", $http_status);
# If not 200, bail # If not 200, bail
if ( $http_status ne '200' ){ if ($http_status ne '200') {
failed("Failed to update Host\n%s", $h); failed("Failed to update Host\n%s", $h);
failed("HTTP response code\n%s", $http_status); failed("HTTP response code\n%s", $http_status);
failed("Full reply\n%s", $reply) unless opt('verbose'); failed("Full reply\n%s", $reply) unless opt('verbose');
@ -4262,8 +4262,7 @@ sub nic_dnsexit2_update {
debug("%s", $strip_status); debug("%s", $strip_status);
if ($strip_status) { if ($strip_status) {
debug("HTTP headers are stripped."); debug("HTTP headers are stripped.");
} } else {
else {
warning("Unexpected: no HTTP headers stripped!"); warning("Unexpected: no HTTP headers stripped!");
} }
@ -4274,16 +4273,16 @@ sub nic_dnsexit2_update {
if (defined($response->{'code'}) and defined($response->{'message'})) { if (defined($response->{'code'}) and defined($response->{'message'})) {
if (exists $status{$response->{'code'}}) { if (exists $status{$response->{'code'}}) {
# Add the server response data to the applicable array # Add the server response data to the applicable array
push( @{ $status {$response->{'code'} } }, $response->{'message'}); push(@{$status{$response->{'code'}}}, $response->{'message'});
if (defined($response->{'details'})) { if (defined($response->{'details'})) {
push ( @{ $status {$response->{'code'} } }, $response->{'details'}[0]); push(@{$status{$response->{'code'}}}, $response->{'details'}[0]);
} else { } else {
# Keep it symmetrical for simplicity # Keep it symmetrical for simplicity
push ( @{ $status {$response->{'code'} } }, "no details received"); push(@{$status{$response->{'code'}}}, "no details received");
} }
# Set data from array # Set data from array
my ($status, $message, $srv_message, $srv_details) = @{ $status {$response->{'code'} } }; my ($status, $message, $srv_message, $srv_details) = @{$status{$response->{'code'}}};
info("Status: %s -- Message: %s", $status, $message); info("Status: %s -- Message: %s", $status, $message);
info("Server Message: %s -- Server Details: %s", $srv_message, $srv_details); info("Server Message: %s -- Server Details: %s", $srv_message, $srv_details);
$config{$h}{'status-ipv4'} = $status if $ipv4; $config{$h}{'status-ipv4'} = $status if $ipv4;
@ -4293,7 +4292,7 @@ sub nic_dnsexit2_update {
if ($status eq 'good') { if ($status eq 'good') {
$config{$h}{'mtime'} = $now; $config{$h}{'mtime'} = $now;
my $tracked_ipv; my $tracked_ipv;
for $tracked_ipv ( keys %total_payload ){ for $tracked_ipv (keys %total_payload) {
$config{$h}{"ipv$tracked_ipv"} = $total_payload{$tracked_ipv}{content}; $config{$h}{"ipv$tracked_ipv"} = $total_payload{$tracked_ipv}{content};
$config{$h}{"status-ipv$tracked_ipv"} = 'good'; $config{$h}{"status-ipv$tracked_ipv"} = 'good';
success("%s", $message); success("%s", $message);
@ -4318,6 +4317,7 @@ sub nic_dnsexit2_update {
} }
} }
} }
###################################################################### ######################################################################
## nic_noip_update ## nic_noip_update
## Note: uses same features as nic_dyndns2_update, less return codes ## Note: uses same features as nic_dyndns2_update, less return codes