wip: checking more config items

UPC · May 9, 2023 · d4b19d8 · d4b19d8
1 parent 0944519
commit d4b19d8
Show file tree

Hide file tree

Showing 3 changed files with 4 additions and 1 deletion.
diff --git a/lib/Ravada/Auth/LDAP.pm b/lib/Ravada/Auth/LDAP.pm
@@ -428,6 +428,8 @@ sub remove_group {
 sub search_group {
     my %args = @_;
 
+    return if !exists $$CONFIG->{ldap} || !$$CONFIG->{ldap};
+
     my $name = delete $args{name} or confess "Error: missing name";
     my $base = ( delete $args{base} or $$CONFIG->{ldap}->{groups_base} or "ou=groups,"._dc_base() );
     my $ldap = ( delete $args{ldap} or _init_ldap_admin());

diff --git a/script/rvd_back b/script/rvd_back
@@ -224,7 +224,7 @@ my $RVD_BACK;
 #
 
 sub _do_start_logging {
-    $LOG_FILENAME = $Ravada::CONFIG->{"log"} if (! $LOG_FILENAME);
+    $LOG_FILENAME = $Ravada::CONFIG->{"log"} if (! $LOG_FILENAME && exists $Ravada::CONFIG->{"log"});
     if ($LOG_FILENAME)
     {
         File::Path::make_path(File::Basename::dirname($LOG_FILENAME));

diff --git a/t/vm/20_base.t b/t/vm/20_base.t
@@ -95,6 +95,7 @@ sub test_display {
     # only test this for Void, it will fail on real VMs
     return if $vm_name ne 'Void';
 
+    unlock_hash(%$Ravada::CONFIG);
     $Ravada::CONFIG->{display_ip} = $DISPLAY_IP;
     eval { $display = $domain->display( user_admin ) };
     is($@,'');