-
-
Notifications
You must be signed in to change notification settings - Fork 756
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
fix security bug with PHP_AUTH_USER #1252
Conversation
I just sent an email to Milan Holzäpfel, I asked him to make a review of your PR (and I forgot you in cc ..). |
Well, the PR is just his patch. I didn't got the time to test it in all kinds of configurations though, so I open it to everyone. |
It fixes the bug, that's for sure. We just need to be sure if it works on particular installations. |
But what's the bug? Any description somewhere? |
Well, I'm ok with that 👍 |
I'm sending you the details by email. |
@tcitworld Nico already send them to me, thanks :) |
Okay, then I will merge before next version (which I would like to be out tomorrow, I'm away starting from Friday) |
Do you want / can you write a blog post about 1.9.2? |
Yup, I will write it tomorrow. |
fix security bug with PHP_AUTH_USER
Thanks to M. Holzäpfel.