Skip to content
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

Special characters e.g. üö§ with Monolog+ Firephp + Chrome extension not working ??? #1521

Closed
wnedoe opened this issue Dec 8, 2020 · 4 comments
Labels
Milestone

Comments

@wnedoe
Copy link

wnedoe commented Dec 8, 2020

Monolog version 2

If i try to log user input including special characters like "Öüħ" to the Firephp Handler, the complete chrome screen of the FIREPHP extension stays blank.
If i remove those characters everythink works perfectly.

Update:
Using ONLY the current class
https://github.com/firephp/firephp-core
together with the firephp chrome extension but WITHOUT MONOLOG

everything works perfectly with all these unicode characters like ÄÜÖ§
they are prperly displayed in the firephp console

So most probably the FirePHPHandler has a bug.

thanks
Alex

@wnedoe wnedoe added the Support label Dec 8, 2020
@Seldaek
Copy link
Owner

Seldaek commented Dec 10, 2020

Can you perhaps show what the exact headers sent look like (both for FirePHPHandler and native firephp-core)? That might hint at why one works and the other not.

@Seldaek
Copy link
Owner

Seldaek commented Dec 10, 2020

Ah never mind, I suspect the issue is because we send unicode unescaped in the JSON. And that goes through headers, which is probably not a great idea.

Can you please try if composer require monolog/monolog:^2.1@dev fixes it for you?

@Seldaek Seldaek added this to the 2.x milestone Dec 10, 2020
@Seldaek Seldaek added Bug and removed Support labels Dec 10, 2020
@wnedoe
Copy link
Author

wnedoe commented Dec 10, 2020 via email

@Seldaek
Copy link
Owner

Seldaek commented Dec 10, 2020

Ok great, I'll try and get a release out soonish.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants