cacert/cacert-webdb
9
0
Fork 0
Code Issues Pull requests 3 Projects Releases 4 Wiki Activity

Fix client certificate login #16

Merged
jandd merged 4 commits from fix-client-certificate-login into main 2024-05-20 10:35:31 +00:00
Conversation 0 Commits 4 Files changed 3 +1 -1
1 changed files with 1 additions and 1 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files
Showing only changes of commit 9626e7f6fc - Show all commits

2
includes/lib/general.php
View file

@ -30,7 +30,7 @@ function get_email_addresses_from_client_cert() {
// try SAN email addresses first
$envNameBase = "SSL_CLIENT_SAN_Email";
for ($i = 1; $i <= $maxAddresses; $i++) {
for ($i = 0; $i <= $maxAddresses; $i++) {
jandd marked this conversation as resolved
jandd commented 2024-05-05 19:33:29 +00:00
Review
Copy link

SSL_Client_SAN_Email_0 is the first item, that was the reason for 9626e7f

SSL_Client_SAN_Email_0 is the first item, that was the reason for 9626e7f
$envName = sprintf("%s_%d", $envNameBase, $i);
if (!array_key_exists($envName, $_SERVER)) {
break;