Update module to stay compatible with SimpleSAMLphp and PHP 8.3+ #63
Annotations
14 errors and 6 warnings
Psalm:
src/Auth/Source/RateLimitUserPass.php#L45
src/Auth/Source/RateLimitUserPass.php:45:19: MissingClassConstType: Class constant "SimpleSAML\Module\ratelimit\Auth\Source\RateLimitUserPass::DEFAULT_CONFIG" should have a declared type. (see https://psalm.dev/359)
|
Psalm:
src/Auth/Source/RateLimitUserPass.php#L103
src/Auth/Source/RateLimitUserPass.php:103:22: UnusedMethodCall: The call to ReflectionMethod::setAccessible is not used (see https://psalm.dev/209)
|
Psalm:
src/Limiters/IpLimiter.php#L26
src/Limiters/IpLimiter.php:26:28: MixedPropertyTypeCoercion: $this->whitelist expects 'array<array-key, string>', parent type `array<array-key, mixed>` provided (see https://psalm.dev/196)
|
Psalm:
src/Limiters/UserPassBaseLimiter.php#L17
src/Limiters/UserPassBaseLimiter.php:17:21: MissingClassConstType: Class constant "SimpleSAML\Module\ratelimit\Limiters\UserPassBaseLimiter::PREAUTH_ALLOW" should have a declared type. (see https://psalm.dev/359)
|
Psalm:
src/Limiters/UserPassBaseLimiter.php#L18
src/Limiters/UserPassBaseLimiter.php:18:21: MissingClassConstType: Class constant "SimpleSAML\Module\ratelimit\Limiters\UserPassBaseLimiter::PREAUTH_BLOCK" should have a declared type. (see https://psalm.dev/359)
|
Psalm:
src/Limiters/UserPassBaseLimiter.php#L19
src/Limiters/UserPassBaseLimiter.php:19:21: MissingClassConstType: Class constant "SimpleSAML\Module\ratelimit\Limiters\UserPassBaseLimiter::PREAUTH_CONTINUE" should have a declared type. (see https://psalm.dev/359)
|
Psalm:
src/Limiters/UserPassBaseLimiter.php#L112
src/Limiters/UserPassBaseLimiter.php:112:33: InvalidFalsableReturnType: The declared return type 'SimpleSAML\Store\StoreInterface' for SimpleSAML\Module\ratelimit\Limiters\UserPassBaseLimiter::getStore does not allow false, but 'SimpleSAML\Store\StoreInterface|false' contains false (see https://psalm.dev/143)
|
Psalm:
src/Limiters/UserPassBaseLimiter.php#L118
src/Limiters/UserPassBaseLimiter.php:118:16: FalsableReturnStatement: The declared return type 'SimpleSAML\Store\StoreInterface' for SimpleSAML\Module\ratelimit\Limiters\UserPassBaseLimiter::getStore does not allow false, but the function returns 'SimpleSAML\Store\StoreInterface|false' (see https://psalm.dev/137)
|
Psalm:
tests/src/Auth/Process/LoopDetectionTest.php#L64
tests/src/Auth/Process/LoopDetectionTest.php:64:16: UnusedMethodCall: The call to ReflectionProperty::setAccessible is not used (see https://psalm.dev/209)
|
Psalm:
tests/src/Limiters/DeviceCookieLimiterTest.php#L18
tests/src/Limiters/DeviceCookieLimiterTest.php:18:13: UndefinedDocblockClass: Docblock-defined class, interface or enum named SimpleSAML\Test\Module\ratelimit\Limiters\HTTP does not exist (see https://psalm.dev/200)
|
Psalm (testsuite):
tests/docker/public/looping-login.php#L6
tests/docker/public/looping-login.php:6:1: MissingFile: Cannot find file /home/runner/work/simplesamlphp-module-ratelimit/simplesamlphp-module-ratelimit/_include.php to include (see https://psalm.dev/107)
|
Psalm (testsuite):
tests/src/Limiters/DeviceCookieLimiterTest.php#L18
tests/src/Limiters/DeviceCookieLimiterTest.php:18:13: UndefinedDocblockClass: Docblock-defined class, interface or enum named SimpleSAML\Test\Module\ratelimit\Limiters\HTTP does not exist (see https://psalm.dev/200)
|
Psalm (testsuite):
tests/src/Limiters/DeviceCookieLimiterTest.php#L188
tests/src/Limiters/DeviceCookieLimiterTest.php:188:27: InvalidArgument: Argument 1 of SimpleSAML\Module\ratelimit\Limiters\DeviceCookieLimiter::setHttp expects SimpleSAML\Utils\HTTP|null, but PHPUnit\Framework\MockObject\MockObject|SimpleSAML\Test\Module\ratelimit\Limiters\HTTP provided (see https://psalm.dev/004)
|
Psalm (testsuite)
Process completed with exit code 2.
|
Psalm:
tests/src/Limiters/DeviceCookieLimiterTest.php#L20
tests/src/Limiters/DeviceCookieLimiterTest.php:20:13: PropertyNotSetInConstructor: Property SimpleSAML\Test\Module\ratelimit\Limiters\DeviceCookieLimiterTest::$mockHttp is not defined in constructor of SimpleSAML\Test\Module\ratelimit\Limiters\DeviceCookieLimiterTest or in any private or final methods called in the constructor (see https://psalm.dev/074)
|
Psalm (testsuite):
tests/docker/config-override.php#L5
tests/docker/config-override.php:5:1: PossiblyUndefinedGlobalVariable: Possibly undefined global variable $config, first seen on line 5 (see https://psalm.dev/126)
|
Psalm (testsuite):
tests/docker/metadata/saml20-idp-hosted.php#L9
tests/docker/metadata/saml20-idp-hosted.php:9:1: PossiblyUndefinedGlobalVariable: Possibly undefined global variable $metadata, first seen on line 9 (see https://psalm.dev/126)
|
Psalm (testsuite):
tests/docker/metadata/saml20-idp-remote.php#L3
tests/docker/metadata/saml20-idp-remote.php:3:1: PossiblyUndefinedGlobalVariable: Possibly undefined global variable $metadata, first seen on line 3 (see https://psalm.dev/126)
|
Psalm (testsuite):
tests/docker/metadata/saml20-sp-remote.php#L3
tests/docker/metadata/saml20-sp-remote.php:3:1: PossiblyUndefinedGlobalVariable: Possibly undefined global variable $metadata, first seen on line 3 (see https://psalm.dev/126)
|
Psalm (testsuite):
tests/src/Limiters/DeviceCookieLimiterTest.php#L20
tests/src/Limiters/DeviceCookieLimiterTest.php:20:13: PropertyNotSetInConstructor: Property SimpleSAML\Test\Module\ratelimit\Limiters\DeviceCookieLimiterTest::$mockHttp is not defined in constructor of SimpleSAML\Test\Module\ratelimit\Limiters\DeviceCookieLimiterTest or in any private or final methods called in the constructor (see https://psalm.dev/074)
|
Loading