Skip to content

Commit ae7881c

Browse files
committed
Set true as the default value for strict setting
1 parent 447a201 commit ae7881c

2 files changed

Lines changed: 5 additions & 6 deletions

File tree

src/Saml2/Settings.php

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -45,7 +45,7 @@ class Settings
4545
*
4646
* @var bool
4747
*/
48-
private $_strict = false;
48+
private $_strict = true;
4949

5050
/**
5151
* Activate debug mode

tests/src/OneLogin/Saml2/SettingsTest.php

Lines changed: 4 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -1004,17 +1004,16 @@ public function testSetStrict()
10041004
{
10051005
$settingsDir = TEST_ROOT .'/settings/';
10061006
include $settingsDir.'settings1.php';
1007-
$settingsInfo['strict'] = false;
10081007

10091008
$settings = new Settings($settingsInfo);
10101009
$this->assertFalse($settings->isStrict());
10111010

1012-
$settings->setStrict(true);
1013-
$this->assertTrue($settings->isStrict());
1014-
10151011
$settings->setStrict(false);
10161012
$this->assertFalse($settings->isStrict());
10171013

1014+
$settings->setStrict(true);
1015+
$this->assertTrue($settings->isStrict());
1016+
10181017
try {
10191018
$settings->setStrict('a');
10201019
$this->fail('Exception was not raised');
@@ -1035,7 +1034,7 @@ public function testIsStrict()
10351034
unset($settingsInfo['strict']);
10361035

10371036
$settings = new Settings($settingsInfo);
1038-
$this->assertFalse($settings->isStrict());
1037+
$this->assertTrue($settings->isStrict());
10391038

10401039
$settingsInfo['strict'] = false;
10411040
$settings2 = new Settings($settingsInfo);

0 commit comments

Comments
 (0)