fix downtime duration in rest commands (fixes #1340)

the plus sign has been url decoded as space, the remaining 1m has then been parsed
as one month instead of one minute.

Author: sni

lib/Thruk/Utils/CLI/Rest.pm

@@ -256,6 +256,7 @@ sub _set_postdata {
 
     my($key,$val) = split(/=/mx, $data, 2);
     if($src && $src eq 'local' && defined $val) {
+        $val =~ s/^\+/%2B/gmx;
         $val =~ s/\+/ /gmx;
         $val = Thruk::Request->unescape($val);
     }

t/scenarios/rest_api/t/local/rest_commands.t

@@ -8,7 +8,7 @@ BEGIN {
     import TestUtils;
 }
 
-plan tests => 26;
+plan tests => 33;
 
 ###########################################################
 # test thruks script path
@@ -41,4 +41,21 @@ TestUtils::test_command({
 TestUtils::test_command({
     cmd  => '/usr/bin/env sed -i etc/thruk/thruk_local.d/lmd.conf -e s/^.*use_lmd_core=.*/#use_lmd_core=1/g',
     like => ['/^$/'],
-});
+});
+
+###########################################################
+# rest downtime duration
+{
+    my $test = {
+        cmd  => '/usr/bin/env thruk r -d "comment_data=test" -d "end_time=+1m" /hosts/localhost/cmd/schedule_host_downtime',
+        like => ['/COMMAND/', '/Command successfully submitted/', '/SCHEDULE_HOST_DOWNTIME/'],
+    };
+    TestUtils::test_command($test);
+    my($t1, $t2) = ($test->{'stdout'} =~ m/SCHEDULE_HOST_DOWNTIME;localhost;(\d+);(\d+);/gmx);
+    if(!$t1) {
+        fail("cannot parse timestamps from stdout: ".$test->{'stdout'});
+    } else {
+        my $duration = $t2 - $t1;
+        ok($duration == 60, "downtime duration should be 60s but is ".$duration."s");
+    }
+}