Fix bug in v-add-letsencrypt-domain hestiacp#2563 (hestiacp#2744)

* Replace search_objects with an better method getting SSL_HOME var

If LE was enabled for multiple domains under the same users search_objects could return multiple times "same
same
same " 

Replacing it with get_object_value should solve the issue

* Add test for it

Author: jaapmarcus

bin/v-add-letsencrypt-domain

@@ -520,7 +520,7 @@ fi
 
 # Adding SSL
 if [ -z "$mail" ]; then
-    ssl_home=$(search_objects 'web' 'LETSENCRYPT' 'yes' 'SSL_HOME')
+    ssl_home="$(get_object_value 'web' 'DOMAIN' "$domain" '$SSL_HOME')"
     ssl_enabled="$(get_object_value 'web' 'DOMAIN' "$domain" '$SSL')"
     if [ "$ssl_enabled" = "yes" ]; then 
         $BIN/v-update-web-domain-ssl "$user" "$domain" "$ssl_dir" "$ssl_home" "updatessl" 

test/letsencrypt.bats

@@ -38,12 +38,24 @@ function setup() {
     refute_output
 }
 
+@test "[ Web ] Create 2nd web domain" {
+    run v-add-web-domain $user "hestia.$domain" $ip yes
+    assert_success
+    refute_output
+}
+
 @test "[ Web ] Request new certificate for web domain" {
     run v-add-letsencrypt-domain $user $domain "www.$domain,renewal.$domain,foobar.$domain,bar.$domain"
     assert_success
     refute_output
 }
 
+@test "[ Web ] Request 2nd new certificate for web domain" {
+    run v-add-letsencrypt-domain $user "hestia.$domain"
+    assert_success
+    refute_output
+}
+
 @test "[ Mail ] Create mail domain" {
     run v-add-mail-domain $user $domain
     assert_success