Skip to content

Commit

Permalink
Revert BC breaking Webmozart assertions
Browse files Browse the repository at this point in the history
  • Loading branch information
tvdijen committed May 29, 2019
1 parent c1bfbbf commit 4704703
Show file tree
Hide file tree
Showing 2 changed files with 4 additions and 5 deletions.
3 changes: 1 addition & 2 deletions composer.json
Original file line number Diff line number Diff line change
Expand Up @@ -29,8 +29,7 @@
},
"require": {
"php": ">=5.6",
"simplesamlphp/composer-module-installer": "~1.1",
"webmozart/assert": "~1.4"
"simplesamlphp/composer-module-installer": "~1.1"
},
"require-dev": {
"simplesamlphp/simplesamlphp": "^1.17",
Expand Down
6 changes: 3 additions & 3 deletions lib/Auth/Process/Authorize.php
Original file line number Diff line number Diff line change
Expand Up @@ -58,7 +58,7 @@ public function __construct($config, $reserved)
{
parent::__construct($config, $reserved);

Assert::isArray($config);
assert(is_array($config));

// Check for the deny option
// Must be bool specifically, if not, it might be for an attrib filter below
Expand Down Expand Up @@ -109,8 +109,8 @@ public function __construct($config, $reserved)
*/
public function process(&$request)
{
Assert::isArray($request);
Assert::keyExists($request, 'Attributes');
assert(is_array($request));
assert(array_key_exists('Attributes', $request));

$authorize = $this->deny;
$attributes = &$request['Attributes'];
Expand Down

0 comments on commit 4704703

Please sign in to comment.