Skip to content

Commit

Permalink
Merge pull request #224 from OpenConext/feature/1-make-tiqr-less-cryptic
Browse files Browse the repository at this point in the history 
1. Make the Tiqr Configuration validation less cryptic
  • Loading branch information
@pmeulen
pmeulen authored Nov 29, 2024
2 parents ad9afb7 + 9ffe9b7 commit c835e4f
Showing 1 changed file with 96 additions and 23 deletions.
119 changes: 96 additions & 23 deletions src/Tiqr/TiqrConfiguration.php
View file
Original file line number Diff line number Diff line change
Expand Up @@ -36,82 +36,155 @@ class TiqrConfiguration implements TiqrConfigurationInterface
* @param array<string, array<string, mixed>> $tiqrConfiguration
*
* @throws \Assert\AssertionFailedException
* @SuppressWarnings(PHPMD.ExcessiveMethodLength)
*/
public function __construct(array $tiqrConfiguration)
{
Assertion::string($tiqrConfiguration['general']['identifier']);
Assertion::string(
$tiqrConfiguration['general']['identifier'],
'TiqrConfiguration: general -> identifier must be of type string'
);
$this->options['identifier'] = $tiqrConfiguration['general']['identifier'];

Assertion::string($tiqrConfiguration['general']['name']);
Assertion::string(
$tiqrConfiguration['general']['name'],
'TiqrConfiguration: general -> name must be of type string'
);
$this->options['name'] = $tiqrConfiguration['general']['name'];

Assertion::string($tiqrConfiguration['general']['auth_protocol']);
Assertion::string(
$tiqrConfiguration['general']['auth_protocol'],
'TiqrConfiguration: general -> auth_protocol must be of type string'
);
$this->options['auth.protocol'] = $tiqrConfiguration['general']['auth_protocol'];

Assertion::string($tiqrConfiguration['general']['enroll_protocol']);
Assertion::string(
$tiqrConfiguration['general']['enroll_protocol'],
'TiqrConfiguration: general -> enroll_protocol must be of type string'
);
$this->options['enroll.protocol'] = $tiqrConfiguration['general']['enroll_protocol'];

Assertion::string($tiqrConfiguration['general']['ocra_suite']);
Assertion::string(
$tiqrConfiguration['general']['ocra_suite'],
'TiqrConfiguration: general -> ocra_suite must be of type string'
);
$this->options['ocra.suite'] = $tiqrConfiguration['general']['ocra_suite'];

Assertion::string($tiqrConfiguration['general']['logoUrl']);
Assertion::string(
$tiqrConfiguration['general']['logoUrl'],
'TiqrConfiguration: general -> logoUrl must be of type string'
);
$this->options['logoUrl'] = $tiqrConfiguration['general']['logoUrl'];

Assertion::string($tiqrConfiguration['general']['infoUrl']);
Assertion::string(
$tiqrConfiguration['general']['infoUrl'],
'TiqrConfiguration: general -> infoUrl must be of type string'
);
$this->options['infoUrl'] = $tiqrConfiguration['general']['infoUrl'];

if (isset($tiqrConfiguration['library']['apns'])) {
Assertion::string($tiqrConfiguration['library']['apns']['certificate']);
Assertion::string(
$tiqrConfiguration['library']['apns']['certificate'],
'TiqrConfiguration: library -> apns -> certificate must be of type string'
);
$this->options['apns.certificate'] = $tiqrConfiguration['library']['apns']['certificate'];
Assertion::string($tiqrConfiguration['library']['apns']['environment']);
Assertion::string(
$tiqrConfiguration['library']['apns']['environment'],
'TiqrConfiguration: library -> apns -> environment must be of type string'
);
$this->options['apns.environment'] = $tiqrConfiguration['library']['apns']['environment'];
}

if (isset($tiqrConfiguration['library']['firebase']) && is_array($tiqrConfiguration['library']['firebase'])) {
Assertion::string($tiqrConfiguration['library']['firebase']['projectId']);
Assertion::string(
$tiqrConfiguration['library']['firebase']['projectId'],
'TiqrConfiguration: library -> firebase -> projectId must be of type string'
);
$this->options['firebase.projectId'] = $tiqrConfiguration['library']['firebase']['projectId'];
Assertion::string($tiqrConfiguration['library']['firebase']['credentialsFile']);
Assertion::string(
$tiqrConfiguration['library']['firebase']['credentialsFile'],
'TiqrConfiguration: library -> firebase -> credentialsFile must be of type string'
);
$this->options['firebase.credentialsFile'] = $tiqrConfiguration['library']['firebase']['credentialsFile'];
Assertion::boolean($tiqrConfiguration['library']['firebase']['cacheTokens']);
Assertion::boolean(
$tiqrConfiguration['library']['firebase']['cacheTokens'],
'TiqrConfiguration: library -> firebase -> cacheTokens must be of type boolean'
);
$this->options['firebase.cacheTokens'] = $tiqrConfiguration['library']['firebase']['cacheTokens'];
Assertion::string($tiqrConfiguration['library']['firebase']['tokenCacheDir']);
Assertion::string(
$tiqrConfiguration['library']['firebase']['tokenCacheDir'],
'TiqrConfiguration: library -> firebase -> tokenCacheDir must be of type string'
);
$this->options['firebase.tokenCacheDir'] = $tiqrConfiguration['library']['firebase']['tokenCacheDir'];
}

if (isset($tiqrConfiguration['accountblocking'][self::MAX_ATTEMPTS])) {
Assertion::digit($tiqrConfiguration['accountblocking'][self::MAX_ATTEMPTS]);
Assertion::greaterThan($tiqrConfiguration['accountblocking'][self::MAX_ATTEMPTS], 0);
Assertion::digit(
$tiqrConfiguration['accountblocking'][self::MAX_ATTEMPTS],
'TiqrConfiguration: accountblocking -> maxAttempts must be of type digit'
);
Assertion::greaterThan(
$tiqrConfiguration['accountblocking'][self::MAX_ATTEMPTS],
0,
'TiqrConfiguration: accountblocking -> maxAttempts must be greater than 0'
);
$this->options[self::MAX_ATTEMPTS] = $tiqrConfiguration['accountblocking'][self::MAX_ATTEMPTS];
}

if (isset($tiqrConfiguration['accountblocking'][self::TEMPORARILY_BLOCK_DURATION])) {
Assertion::digit($tiqrConfiguration['accountblocking'][self::TEMPORARILY_BLOCK_DURATION]);
Assertion::greaterThan($tiqrConfiguration['accountblocking'][self::TEMPORARILY_BLOCK_DURATION], 0);
Assertion::digit(
$tiqrConfiguration['accountblocking'][self::TEMPORARILY_BLOCK_DURATION],
'TiqrConfiguration: accountblocking -> temporarilyBlockDuration must be of type digit'
);
Assertion::greaterThan(
$tiqrConfiguration['accountblocking'][self::TEMPORARILY_BLOCK_DURATION],
0,
'TiqrConfiguration: accountblocking -> temporarilyBlockDuration must be greater than 0'
);
$this->options[self::TEMPORARILY_BLOCK_DURATION] =
$tiqrConfiguration['accountblocking'][self::TEMPORARILY_BLOCK_DURATION];
}

if (isset($tiqrConfiguration['accountblocking'][self::MAX_TEMPORARILY_BLOCKS])) {
Assertion::digit($tiqrConfiguration['accountblocking'][self::MAX_TEMPORARILY_BLOCKS]);
Assertion::greaterThan($tiqrConfiguration['accountblocking'][self::MAX_TEMPORARILY_BLOCKS], 0);
Assertion::digit(
$tiqrConfiguration['accountblocking'][self::MAX_TEMPORARILY_BLOCKS],
'TiqrConfiguration: accountblocking -> maxTemporarilyBlocks must be of type digit'
);
Assertion::greaterThan(
$tiqrConfiguration['accountblocking'][self::MAX_TEMPORARILY_BLOCKS],
0,
'TiqrConfiguration: accountblocking -> maxTemporarilyBlocks must be greater than 0'
);
$this->options[self::MAX_TEMPORARILY_BLOCKS] = $tiqrConfiguration['accountblocking'][self::MAX_TEMPORARILY_BLOCKS];
}

$this->options['statestorage']['type'] = $tiqrConfiguration['storage']['statestorage']['type'];
Assertion::isArray($tiqrConfiguration['storage']['statestorage']['arguments']);
Assertion::isArray(
$tiqrConfiguration['storage']['statestorage']['arguments'],
'TiqrConfiguration: storage -> statestorage -> arguments must be of type array'
);
$this->options['statestorage'] += $tiqrConfiguration['storage']['statestorage']['arguments'];

$this->options['userstorage']['type'] = $tiqrConfiguration['storage']['userstorage']['type'];
Assertion::isArray($tiqrConfiguration['storage']['userstorage']['arguments']);
Assertion::isArray(
$tiqrConfiguration['storage']['userstorage']['arguments'],
'TiqrConfiguration: storage -> userstorage -> arguments must be of type array'
);
$this->options['userstorage'] += $tiqrConfiguration['storage']['userstorage']['arguments'];

$this->options['devicestorage']['type'] = $tiqrConfiguration['storage']['devicestorage']['type'];
Assertion::isArray($tiqrConfiguration['storage']['devicestorage']['arguments']);
Assertion::isArray(
$tiqrConfiguration['storage']['devicestorage']['arguments'],
'TiqrConfiguration: storage -> devicestorage -> arguments must be of type array'
);
$this->options['devicestorage'] += $tiqrConfiguration['storage']['devicestorage']['arguments'];

if (isset($tiqrConfiguration['storage']['usersecretstorage'])) {
$this->options['usersecretstorage']['type'] = $tiqrConfiguration['storage']['usersecretstorage']['type'];
Assertion::isArray($tiqrConfiguration['storage']['usersecretstorage']['arguments']);
Assertion::isArray(
$tiqrConfiguration['storage']['usersecretstorage']['arguments'],
'TiqrConfiguration: storage -> usersecretstorage -> arguments must be of type array'
);
$this->options['usersecretstorage'] += $tiqrConfiguration['storage']['usersecretstorage']['arguments'];
}
}
Expand Down

0 comments on commit c835e4f

Please sign in to comment.