Bump actions/checkout from 4 to 5 in the all-actions group #165
php.yml
on: pull_request
Matrix: PHP-Linter
Linter
/
ecmascript-linter
Linter
/
json-linter
Linter
/
style-linter
Linter
/
yaml-linter
Matrix: unit-tests-linux
Matrix: unit-tests-windows
Cleanup artifacts
7s
Annotations
23 errors, 13 warnings, and 4 notices
|
Unit tests, PHP 8.4, windows-latest
Process completed with exit code 1.
|
|
Unit tests, PHP 8.4, windows-latest
Failed to extract simplesamlphp/simplesamlphp: (2) C:\ProgramData\Chocolatey\bin\7z.EXE x -bb0 -y D:\a\simplesamlphp-module-negotiateext\simplesamlphp-module-negotiateext\vendor\composer\tmp-40b361e86b6031cc428378b6cb3592b8.zip -oD:\a\simplesamlphp-module-negotiateext\simplesamlphp-module-negotiateext\vendor\composer\8ba71b02
ERROR: Dangerous link path was ignored : simplesamlphp-simplesamlphp-d791ed7\lib\_autoload.php : ..\src\_autoload.php
|
|
Unit tests, PHP 8.3, windows-latest
Process completed with exit code 1.
|
|
Unit tests, PHP 8.3, windows-latest
Failed to extract simplesamlphp/simplesamlphp: (2) C:\ProgramData\Chocolatey\bin\7z.EXE x -bb0 -y D:\a\simplesamlphp-module-negotiateext\simplesamlphp-module-negotiateext\vendor\composer\tmp-2147784ec5ccc87dd5771e0f9527c7ba.zip -oD:\a\simplesamlphp-module-negotiateext\simplesamlphp-module-negotiateext\vendor\composer\9145edb8
ERROR: Dangerous link path was ignored : simplesamlphp-simplesamlphp-d791ed7\lib\_autoload.php : ..\src\_autoload.php
|
|
Unit tests, PHP 8.3, windows-latest
The strategy configuration was canceled because "unit-tests-windows.windows-latest_8_4" failed
|
|
Unit tests, PHP 8.2, windows-latest
The operation was canceled.
|
|
Unit tests, PHP 8.2, windows-latest
The strategy configuration was canceled because "unit-tests-windows.windows-latest_8_4" failed
|
|
Unit tests, PHP 8.1, windows-latest
The operation was canceled.
|
|
Unit tests, PHP 8.1, windows-latest
The strategy configuration was canceled because "unit-tests-windows.windows-latest_8_4" failed
|
|
Quality control
Process completed with exit code 2.
|
|
UnusedIssueHandlerSuppression:
psalm-dev.xml#L0
/home/runner/work/simplesamlphp-module-negotiateext/simplesamlphp-module-negotiateext/psalm-dev.xml:0:0: UnusedIssueHandlerSuppression: Suppressed issue type "UnresolvableInclude" for /tests/bootstrap.php was not thrown. (see https://psalm.dev/326)
|
|
MissingOverrideAttribute:
tests/src/Controller/NegotiateControllerTest.php#L29
tests/src/Controller/NegotiateControllerTest.php:29:5: MissingOverrideAttribute: Method SimpleSAML\Test\Module\negotiateext\Controller\NegotiateControllerTest::setup should have the "Override" attribute (see https://psalm.dev/358)
|
|
UnusedClass:
tests/src/Controller/NegotiateControllerTest.php#L21
tests/src/Controller/NegotiateControllerTest.php:21:13: UnusedClass: Class SimpleSAML\Test\Module\negotiateext\Controller\NegotiateControllerTest is never used (see https://psalm.dev/075)
|
|
RiskyTruthyFalsyComparison:
src/Auth/Source/Negotiate.php#L230
src/Auth/Source/Negotiate.php:230:13: RiskyTruthyFalsyComparison: Operand of type array<array-key, mixed>|null contains type array<array-key, mixed>, which can be falsy and truthy. This can cause possibly unexpected behavior. Use strict comparison instead. (see https://psalm.dev/356)
|
|
PossiblyNullArgument:
src/Auth/Source/Negotiate.php#L229
src/Auth/Source/Negotiate.php:229:41: PossiblyNullArgument: Argument 1 of SimpleSAML\Module\negotiateext\Auth\Source\Negotiate::lookupUserData cannot be null, possibly null value provided (see https://psalm.dev/078)
|
|
PossiblyUndefinedArrayOffset:
src/Auth/Source/Negotiate.php#L227
src/Auth/Source/Negotiate.php:227:17: PossiblyUndefinedArrayOffset: Possibly undefined array key $_SERVER['REMOTE_USER'] on array{APP_DEBUG?: bool|string, APP_ENV?: string, AUTH_TYPE?: non-empty-string, CONTENT_LENGTH?: string, CONTENT_TYPE?: string, DOCUMENT_ROOT?: non-empty-string, FCGI_ROLE?: non-empty-string, GATEWAY_INTERFACE?: non-empty-string, HOME?: non-empty-string, HTTPS?: string, HTTP_ACCEPT?: non-empty-string, HTTP_ACCEPT_CHARSET?: non-empty-string, HTTP_ACCEPT_ENCODING?: non-empty-string, HTTP_ACCEPT_LANGUAGE?: non-empty-string, HTTP_CACHE_CONTROL?: non-empty-string, HTTP_CDN_LOOP?: non-empty-string, HTTP_CF_CONNECTING_IP?: non-empty-string, HTTP_CF_IPCOUNTRY?: non-empty-string, HTTP_CF_VISITOR?: non-empty-string, HTTP_CLIENT_IP?: non-empty-string, HTTP_CONNECTION?: non-empty-string, HTTP_COOKIE?: non-empty-string, HTTP_DNT?: non-empty-string, HTTP_HOST?: non-empty-string, HTTP_PRIORITY?: non-empty-string, HTTP_REFERER?: non-empty-string, HTTP_SEC_CH_UA?: non-empty-string, HTTP_SEC_CH_UA_MOBILE?: non-empty-string, HTTP_SEC_CH_UA_PLATFORM?: non-empty-string, HTTP_SEC_FETCH_DEST?: non-empty-string, HTTP_SEC_FETCH_MODE?: non-empty-string, HTTP_SEC_FETCH_SITE?: non-empty-string, HTTP_SEC_FETCH_USER?: non-empty-string, HTTP_UPGRADE_INSECURE_REQUESTS?: non-empty-string, HTTP_USER_AGENT?: non-empty-string, HTTP_X_FORWARDED_FOR?: non-empty-string, HTTP_X_FORWARDED_PROTO?: non-empty-string, HTTP_X_REAL_IP?: non-empty-string, ORIG_PATH_INFO?: non-empty-string, PATH?: non-empty-string, PATH_INFO?: non-empty-string, PATH_TRANSLATED?: non-empty-string, PHP_AUTH_DIGEST?: non-empty-string, PHP_AUTH_PW?: non-empty-string, PHP_AUTH_USER?: non-empty-string, PHP_SELF?: non-empty-string, QUERY_STRING?: string, REDIRECT_REMOTE_USER?: non-empty-string, REDIRECT_STATUS?: non-empty-string, REMOTE_ADDR?: non-empty-string, REMOTE_HOST?: non-empty-string, REMOTE_PORT?: string, REMOTE_USER?: non-empty-string, REQUEST_METHOD?: non-empty-string, REQUEST_SCHEME?: non-empty-string, REQUEST_TIME?: int<1755529345, max>, REQUEST_TIME_FLOAT?: float, REQUEST_URI?: non-empty-string, SCRIPT_FILENAME?: non-empty-string, SCRIPT_NAME?: non-empty-string, SERVER_ADDR?: non-empty-string, SERVER_ADMIN?: non-empty-string, SERVER_NAME?: non-empty-string, SERVER_PORT?: non-empty-string, SERVER_PROTOCOL?: non-empty-string, SERVER_SIGNATURE?: non-empty-string, SERVER_SOFTWARE?: non-empty-string, USER?: non-empty-string, argc?: int<1, max>, argv?: non-empty-list<string>, ...<non-empty-string, string>} (see https://psalm.dev/167)
|
|
PossiblyNullArgument:
src/Auth/Source/Negotiate.php#L158
src/Auth/Source/Negotiate.php:158:30: PossiblyNullArgument: Argument 1 of Symfony\Component\HttpFoundation\IpUtils::checkIp cannot be null, possibly null value provided (see https://psalm.dev/078)
|
|
RiskyTruthyFalsyComparison:
src/Auth/Source/Negotiate.php#L93
src/Auth/Source/Negotiate.php:93:15: RiskyTruthyFalsyComparison: Operand of type null|string contains type string, which can be falsy and truthy. This can cause possibly unexpected behavior. Use strict comparison instead. (see https://psalm.dev/356)
|
|
RiskyTruthyFalsyComparison:
src/Auth/Source/Negotiate.php#L91
src/Auth/Source/Negotiate.php:91:15: RiskyTruthyFalsyComparison: Operand of type non-empty-array<int|non-empty-string, array<int|non-empty-string, mixed>|string>|null|string contains type string, which can be falsy and truthy. This can cause possibly unexpected behavior. Use strict comparison instead. (see https://psalm.dev/356)
|
|
MissingOverrideAttribute:
src/Auth/Source/Negotiate.php#L70
src/Auth/Source/Negotiate.php:70:5: MissingOverrideAttribute: Method SimpleSAML\Module\negotiateext\Auth\Source\Negotiate::authenticate should have the "Override" attribute (see https://psalm.dev/358)
|
|
PossiblyUnusedProperty:
src/Auth/Source/Negotiate.php#L34
src/Auth/Source/Negotiate.php:34:22: PossiblyUnusedProperty: Cannot find any references to property SimpleSAML\Module\negotiateext\Auth\Source\Negotiate::$keytab (see https://psalm.dev/149)
|
|
MissingClassConstType:
src/Auth/Source/Negotiate.php#L25
src/Auth/Source/Negotiate.php:25:18: MissingClassConstType: Class constant "SimpleSAML\Module\negotiateext\Auth\Source\Negotiate::STAGEID" should have a declared type. (see https://psalm.dev/359)
|
|
ClassMustBeFinal:
src/Auth/Source/Negotiate.php#L22
src/Auth/Source/Negotiate.php:22:7: ClassMustBeFinal: Class SimpleSAML\Module\negotiateext\Auth\Source\Negotiate is never extended and is not part of the public API, and thus must be made final. (see https://psalm.dev/361)
|
|
PossiblyUndefinedVariable:
tests/src/Controller/NegotiateControllerTest.php#L169
tests/src/Controller/NegotiateControllerTest.php:169:29: PossiblyUndefinedVariable: Possibly undefined variable $cookie, first seen on line 163 (see https://psalm.dev/018)
|
|
PossiblyUndefinedVariable:
tests/src/Controller/NegotiateControllerTest.php#L145
tests/src/Controller/NegotiateControllerTest.php:145:27: PossiblyUndefinedVariable: Possibly undefined variable $cookie, first seen on line 133 (see https://psalm.dev/018)
|
|
PossiblyUndefinedVariable:
tests/src/Controller/NegotiateControllerTest.php#L144
tests/src/Controller/NegotiateControllerTest.php:144:27: PossiblyUndefinedVariable: Possibly undefined variable $cookie, first seen on line 133 (see https://psalm.dev/018)
|
|
PossiblyUndefinedVariable:
tests/src/Controller/NegotiateControllerTest.php#L143
tests/src/Controller/NegotiateControllerTest.php:143:29: PossiblyUndefinedVariable: Possibly undefined variable $cookie, first seen on line 133 (see https://psalm.dev/018)
|
|
PossiblyUndefinedVariable:
tests/src/Controller/NegotiateControllerTest.php#L142
tests/src/Controller/NegotiateControllerTest.php:142:43: PossiblyUndefinedVariable: Possibly undefined variable $cookie, first seen on line 133 (see https://psalm.dev/018)
|
|
PossiblyUndefinedVariable:
tests/src/Controller/NegotiateControllerTest.php#L141
tests/src/Controller/NegotiateControllerTest.php:141:29: PossiblyUndefinedVariable: Possibly undefined variable $cookie, first seen on line 133 (see https://psalm.dev/018)
|
|
PossiblyUndefinedVariable:
tests/src/Controller/NegotiateControllerTest.php#L140
tests/src/Controller/NegotiateControllerTest.php:140:29: PossiblyUndefinedVariable: Possibly undefined variable $cookie, first seen on line 133 (see https://psalm.dev/018)
|
|
PossiblyUndefinedVariable:
tests/src/Controller/NegotiateControllerTest.php#L139
tests/src/Controller/NegotiateControllerTest.php:139:29: PossiblyUndefinedVariable: Possibly undefined variable $cookie, first seen on line 133 (see https://psalm.dev/018)
|
|
PropertyNotSetInConstructor:
tests/src/Controller/NegotiateControllerTest.php#L24
tests/src/Controller/NegotiateControllerTest.php:24:23: PropertyNotSetInConstructor: Property SimpleSAML\Test\Module\negotiateext\Controller\NegotiateControllerTest::$session is not defined in constructor of SimpleSAML\Test\Module\negotiateext\Controller\NegotiateControllerTest or in any methods called in the constructor (see https://psalm.dev/074)
|
|
PropertyNotSetInConstructor:
tests/src/Controller/NegotiateControllerTest.php#L23
tests/src/Controller/NegotiateControllerTest.php:23:29: PropertyNotSetInConstructor: Property SimpleSAML\Test\Module\negotiateext\Controller\NegotiateControllerTest::$config is not defined in constructor of SimpleSAML\Test\Module\negotiateext\Controller\NegotiateControllerTest or in any methods called in the constructor (see https://psalm.dev/074)
|
|
UnusedClass:
src/Controller/NegotiateController.php#L27
src/Controller/NegotiateController.php:27:7: UnusedClass: Class SimpleSAML\Module\negotiateext\Controller\NegotiateController is never used (see https://psalm.dev/075)
|
|
PossiblyNullOperand:
src/Auth/Source/Negotiate.php#L228
src/Auth/Source/Negotiate.php:228:55: PossiblyNullOperand: Cannot concatenate with a possibly null null|string (see https://psalm.dev/080)
|
|
PossiblyUnusedMethod:
src/Auth/Source/Negotiate.php#L46
src/Auth/Source/Negotiate.php:46:21: PossiblyUnusedMethod: Cannot find any calls to method SimpleSAML\Module\negotiateext\Auth\Source\Negotiate::__construct (see https://psalm.dev/087)
|
|
Unit tests, PHP 8.4, windows-latest
The windows-latest label will migrate from Windows Server 2022 to Windows Server 2025 beginning September 2, 2025. For more information see https://github.com/actions/runner-images/issues/12677
|
|
Unit tests, PHP 8.3, windows-latest
The windows-latest label will migrate from Windows Server 2022 to Windows Server 2025 beginning September 2, 2025. For more information see https://github.com/actions/runner-images/issues/12677
|
|
Unit tests, PHP 8.2, windows-latest
The windows-latest label will migrate from Windows Server 2022 to Windows Server 2025 beginning September 2, 2025. For more information see https://github.com/actions/runner-images/issues/12677
|
|
Unit tests, PHP 8.1, windows-latest
The windows-latest label will migrate from Windows Server 2022 to Windows Server 2025 beginning September 2, 2025. For more information see https://github.com/actions/runner-images/issues/12677
|