From 0ed297085240d9738d04bc32ac9b0324c3a44347 Mon Sep 17 00:00:00 2001 From: Richard Hansen Date: Sat, 13 Jul 2024 18:13:45 -0400 Subject: [PATCH] keysystems: Honor `http:` or `https:` scheme in `server` variable or fall back to the value of the `ssl` variable if neither `http:` nor `https:` is present. --- ChangeLog.md | 6 +++--- ddclient.in | 5 +---- 2 files changed, 4 insertions(+), 7 deletions(-) diff --git a/ChangeLog.md b/ChangeLog.md index ca430c7..30deeb5 100644 --- a/ChangeLog.md +++ b/ChangeLog.md @@ -62,9 +62,9 @@ repository history](https://github.com/ddclient/ddclient/commits/master). address changes. [#654](https://github.com/ddclient/ddclient/pull/654) * `he.net`: Added support for updating Hurricane Electric records. [#682](https://github.com/ddclient/ddclient/pull/682) - * `dyndns2`, `domeneshop`, `dnsmadeeasy`: The `server` option can now include - `http://` or `https://` to control the use of TLS. If omitted, the value of - the `ssl` option is used to determine the scheme. + * `dyndns2`, `domeneshop`, `dnsmadeeasy`, `keysystems`: The `server` option + can now include `http://` or `https://` to control the use of TLS. If + omitted, the value of the `ssl` option is used to determine the scheme. [#703](https://github.com/ddclient/ddclient/pull/703) ### Bug fixes diff --git a/ddclient.in b/ddclient.in index 3963c0d..1e2cd18 100755 --- a/ddclient.in +++ b/ddclient.in @@ -7654,10 +7654,7 @@ sub nic_keysystems_update { for my $h (@_) { my $ip = delete $config{$h}{'wantip'}; info("KEYSYSTEMS setting IP address to %s for %s", $ip, $h); - - my $url = "http://$config{$h}{'server'}/update.php?hostname=$h&password=$config{$h}{'password'}&ip=$ip"; - - # Try to get URL + my $url = "$config{$h}{'server'}/update.php?hostname=$h&password=$config{$h}{'password'}&ip=$ip"; my $reply = geturl(proxy => opt('proxy'), url => $url) // ''; # No response, declare as failed