Merge pull request #8 from simplesamlphp/dependabot/github_actions/al… #123
tvdijenphp.yml
on: push
Matrix: PHP-Linter
Linter
/
ecmascript-linter
Linter
/
json-linter
Linter
/
style-linter
Linter
/
yaml-linter
Quality control
45s
Security checks
22s
Matrix: unit-tests-linux
Matrix: unit-tests-windows
Cleanup artifacts
3s
Annotations
12 errors and 1 warning
|
Quality control
Process completed with exit code 2.
|
|
MissingOverrideAttribute:
tests/src/Auth/Process/FticksTest.php#L66
tests/src/Auth/Process/FticksTest.php:66:5: MissingOverrideAttribute: Method SimpleSAML\Test\Module\fticks\Auth\Process\FticksTest::setup should have the "Override" attribute (see https://psalm.dev/358)
|
|
UnusedClass:
tests/src/Auth/Process/FticksTest.php#L18
tests/src/Auth/Process/FticksTest.php:18:13: UnusedClass: Class SimpleSAML\Test\Module\fticks\Auth\Process\FticksTest is never used (see https://psalm.dev/075)
|
|
Quality control
Process completed with exit code 2.
|
|
MissingOverrideAttribute:
src/Auth/Process/Fticks.php#L350
src/Auth/Process/Fticks.php:350:5: MissingOverrideAttribute: Method SimpleSAML\Module\fticks\Auth\Process\Fticks::process should have the "Override" attribute (see https://psalm.dev/358)
|
|
NullableReturnStatement:
src/Auth/Process/Fticks.php#L205
src/Auth/Process/Fticks.php:205:16: NullableReturnStatement: The declared return type 'string' for SimpleSAML\Module\fticks\Auth\Process\Fticks::escapeFticks is not nullable, but the function returns 'null|string' (see https://psalm.dev/139)
|
|
InvalidNullableReturnType:
src/Auth/Process/Fticks.php#L201
src/Auth/Process/Fticks.php:201:16: InvalidNullableReturnType: The declared return type 'string' for SimpleSAML\Module\fticks\Auth\Process\Fticks::escapeFticks is not nullable, but 'null|string' contains null (see https://psalm.dev/144)
|
|
PossiblyFalseArgument:
src/Auth/Process/Fticks.php#L184
src/Auth/Process/Fticks.php:184:33: PossiblyFalseArgument: Argument 1 of strlen cannot be false, possibly string value expected (see https://psalm.dev/104)
|
|
PossiblyFalseArgument:
src/Auth/Process/Fticks.php#L124
src/Auth/Process/Fticks.php:124:21: PossiblyFalseArgument: Argument 1 of socket_sendto cannot be false, possibly Socket value expected (see https://psalm.dev/104)
|
|
UnusedFunctionCall:
src/Auth/Process/Fticks.php#L123
src/Auth/Process/Fticks.php:123:17: UnusedFunctionCall: The call to socket_sendto is not used (see https://psalm.dev/206)
|
|
PossiblyFalseOperand:
src/Auth/Process/Fticks.php#L115
src/Auth/Process/Fticks.php:115:37: PossiblyFalseOperand: Cannot concatenate with a possibly false false|string (see https://psalm.dev/162)
|
|
ClassMustBeFinal:
src/Auth/Process/Fticks.php#L51
src/Auth/Process/Fticks.php:51:7: ClassMustBeFinal: Class SimpleSAML\Module\fticks\Auth\Process\Fticks is never extended and is not part of the public API, and thus must be made final. (see https://psalm.dev/361)
|
|
UnusedClass:
src/Auth/Process/Fticks.php#L51
src/Auth/Process/Fticks.php:51:7: UnusedClass: Class SimpleSAML\Module\fticks\Auth\Process\Fticks is never used (see https://psalm.dev/075)
|