Skip to content

Commit

Permalink
Fix
Browse files Browse the repository at this point in the history
  • Loading branch information
tvdijen committed Apr 7, 2024
1 parent 83b5198 commit c69ead1
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion src/Auth/Source/X509userCert.php
Original file line number Diff line number Diff line change
Expand Up @@ -214,7 +214,7 @@ public function authenticate(array &$state): void

$merged_ldapcerts = [];
foreach ($this->ldapusercert as $attr) {
$merged_ldapcerts = array_merge($merged_ldapcerts, $ldap_certs[$attr]);
$merged_ldapcerts = array_merge($merged_ldapcerts, $ldap_certs[0][$attr]);

Check failure on line 217 in src/Auth/Source/X509userCert.php

View workflow job for this annotation

GitHub Actions / Quality control

PossiblyNullArrayAccess

src/Auth/Source/X509userCert.php:217:64: PossiblyNullArrayAccess: Cannot access array value on possibly null variable $ldap_certs[0] of type array<array-key, mixed>|null (see https://psalm.dev/079)

Check failure on line 217 in src/Auth/Source/X509userCert.php

View workflow job for this annotation

GitHub Actions / Quality control

PossiblyNullArgument

src/Auth/Source/X509userCert.php:217:64: PossiblyNullArgument: Argument 2 of array_merge cannot be null, possibly null value provided (see https://psalm.dev/078)

Check failure on line 217 in src/Auth/Source/X509userCert.php

View workflow job for this annotation

GitHub Actions / Quality control

PossiblyNullArrayAccess

src/Auth/Source/X509userCert.php:217:64: PossiblyNullArrayAccess: Cannot access array value on possibly null variable $ldap_certs[0] of type array<array-key, mixed>|null (see https://psalm.dev/079)

Check failure on line 217 in src/Auth/Source/X509userCert.php

View workflow job for this annotation

GitHub Actions / Quality control

PossiblyNullArgument

src/Auth/Source/X509userCert.php:217:64: PossiblyNullArgument: Argument 2 of array_merge cannot be null, possibly null value provided (see https://psalm.dev/078)
}
$ldap_certs = $merged_ldapcerts;

Expand Down

0 comments on commit c69ead1

Please sign in to comment.