Skip to content
Snippets Groups Projects
Unverified Commit 66828779 authored by Jan Trejbal's avatar Jan Trejbal
Browse files

Fix tests for PHP 7.2

parent 25841162
No related branches found
No related tags found
No related merge requests found
...@@ -76,13 +76,15 @@ class SessionHandlerPHP extends SessionHandler ...@@ -76,13 +76,15 @@ class SessionHandlerPHP extends SessionHandler
$params = $this->getCookieParams(); $params = $this->getCookieParams();
session_set_cookie_params( if (!headers_sent()) {
$params['lifetime'], session_set_cookie_params(
$params['path'], $params['lifetime'],
$params['domain'], $params['path'],
$params['secure'], $params['domain'],
$params['httponly'] $params['secure'],
); $params['httponly']
);
}
$savepath = $config->getString('session.phpsession.savepath', null); $savepath = $config->getString('session.phpsession.savepath', null);
if (!empty($savepath)) { if (!empty($savepath)) {
......
...@@ -443,7 +443,7 @@ class sspmod_saml_IdP_SAML2 ...@@ -443,7 +443,7 @@ class sspmod_saml_IdP_SAML2
public static function processSOAPAuthnRequest(array &$state) public static function processSOAPAuthnRequest(array &$state)
{ {
if (!isset($_SERVER['PHP_AUTH_USER']) || !isset($_SERVER['PHP_AUTH_PW'])) { if (!isset($_SERVER['PHP_AUTH_USER']) || !isset($_SERVER['PHP_AUTH_PW'])) {
SimpleSAML_Logger::error("ECP AuthnRequest did not contain Basic Authentication header"); SimpleSAML\Logger::error("ECP AuthnRequest did not contain Basic Authentication header");
// TODO Throw some sort of ECP-specific exception / convert this to SOAP fault // TODO Throw some sort of ECP-specific exception / convert this to SOAP fault
throw new SimpleSAML_Error_Error("WRONGUSERPASS"); throw new SimpleSAML_Error_Error("WRONGUSERPASS");
} }
......
0% Loading or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment