fix status test

silinternational · Jun 20, 2024 · 3bdc9d4 · 3bdc9d4
1 parent a2159a9
commit 3bdc9d4
Show file tree

Hide file tree

Showing 3 changed files with 8 additions and 5 deletions.
diff --git a/behat.yml b/behat.yml
@@ -28,6 +28,6 @@ default:
 #        - '%paths.base%//features//Sp3Idp1Sp1Idp1Sp2Idp2.feature'
 #        - '%paths.base%//features//WwwMetadataCept.feature'
 #        - '%paths.base%//features//ZSp1Idp1BetaSp1Idp3.feature'
-#    status_features:
-#      paths:    [ '%paths.base%//features//status.feature' ]
-#      contexts: [ 'StatusContext' ]
+    status_features:
+      paths:    [ '%paths.base%//features//status.feature' ]
+      contexts: [ 'StatusContext' ]
diff --git a/features/bootstrap/StatusContext.php b/features/bootstrap/StatusContext.php
@@ -39,7 +39,7 @@ public function iRequestTheInitialLoginPageOfThisModule()
             'cookies' => true,
             'http_errors' => false,
         ]);
-        $response = $client->get('http://ssp-idp1.local/module.php/core/authenticate.php?as=silauth');
+        $response = $client->get('http://ssp-idp1.local/module.php/admin/test/silauth');
         $this->responseCode = $response->getStatusCode();
     }
 

diff --git a/modules/silauth/src/Auth/Source/system/System.php b/modules/silauth/src/Auth/Source/system/System.php
@@ -29,7 +29,7 @@ protected function isDatabaseOkay(): bool
             FailedLoginIpAddress::getMostRecentFailedLoginFor('');
             return true;
         } catch (Throwable $t) {
-            $this->logError($t->getMessage());
+            $this->logError('isDatabaseOkay database check returned error: ' . $t->getMessage());
             return false;
         }
     }
@@ -47,6 +47,9 @@ protected function isRequiredConfigPresent(): bool
          */
         $baseURL = $globalConfig->getString('baseurlpath', '');
         $avoidsSecurityHole = (preg_match('#^https?://.*/$#D', $baseURL) === 1);
+        if (!$avoidsSecurityHole) {
+            $this->logError('isRequiredConfigPresent failed: baseurlpath (' . $baseURL . ') does not meet requirements');
+        }
         return $avoidsSecurityHole;
     }