-
Notifications
You must be signed in to change notification settings - Fork 218
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
Error with level-3 compression on some HTML #84
Comments
Hello. |
No, it's not fixed. Issue can be reproduced with the last git source code. P.S. checked with the latest brotli - no errors with quality=3 |
I'll check it on Monday. Thanks for the report. |
Works for me... Could you create a PR that updates...
|
Sorry, false start. The fix is #83, and it is not yet landed. |
Landed |
Thank you. It seems issue has been resolved. Will check on production in a month, this can be closed right now. |
Hello. Noticed that some HTML (example attached) is broken with brotli_comp_level 3 optimization (nor 2-, nor 4+, exactly 3). With attached HTML under brotli compression level 3 there is no symbols after line 527, position 61. With level 2 or level 4 - the whole document is correctly compressed.
brotli-3.html.txt
The text was updated successfully, but these errors were encountered: