Skip to content
Snippets Groups Projects
Commit 18885b31 authored by Tim van Dijen's avatar Tim van Dijen
Browse files

Fix Scrutinizer issues

parent d0ed6ff9
No related branches found
No related tags found
No related merge requests found
......@@ -13,6 +13,7 @@ use SimpleSAML\Module;
use SimpleSAML\Module\saml\IdP\SAML2 as SAML2_IdP;
use SimpleSAML\Utils;
use Symfony\Component\HttpFoundation\Request;
use Symfony\Component\HttpFoundation\Response;
use function strpos;
use function strrpos;
......@@ -73,7 +74,7 @@ class Metadata
}
// check if valid local session exists
if ($config->getOptionalBoolean('admin.protectmetadata', false)) {
if ($this->config->getOptionalBoolean('admin.protectmetadata', false)) {
return new RunnableResponse([$this->authUtils, 'requireAdmin']);
}
......
......@@ -89,7 +89,7 @@ class SingleLogout
{
Logger::info('SAML2.0 - IdP.initSLO: Accessing SAML 2.0 IdP endpoint init Single Logout');
if ($this->config->getOptionalBoolean('enable.saml20-idp') === false || !Module::isModuleEnabled('saml')) {
if ($this->config->getBoolean('enable.saml20-idp') === false || !Module::isModuleEnabled('saml')) {
throw new Error\Error('NOACCESS', null, 403);
}
......
......@@ -4,6 +4,7 @@ declare(strict_types=1);
namespace SimpleSAML\Module\saml\Controller;
use Exception;
use SAML2\Exception\Protocol\UnsupportedBindingException;
use SAML2\ArtifactResolve;
use SAML2\ArtifactResponse;
......
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