After an evaluation, GNOME has moved from Bugzilla to GitLab. Learn more about GitLab.
No new issues can be reported in GNOME Bugzilla anymore.
To report an issue in a GNOME project, go to GNOME GitLab.
Do not go to GNOME Gitlab for: Bluefish, Doxygen, GnuCash, GStreamer, java-gnome, LDTP, NetworkManager, Tomboy.
Bug 676478 - Broken gzip decoding
Broken gzip decoding
Status: RESOLVED FIXED
Product: glib
Classification: Platform
Component: gio
unspecified
Other Linux
: Normal major
: ---
Assigned To: gtkdev
gtkdev
: 676613 (view as bug list)
Depends on:
Blocks:
 
 
Reported: 2012-05-21 09:47 UTC by Sergio Villar
Modified: 2012-06-14 10:31 UTC
See Also:
GNOME target: ---
GNOME version: ---

Attachments
Refactor coding-test.c results (8.01 KB, patch)
2012-05-25 09:25 UTC, Sergio Villar 		none Details | Review
coding-test: test also SoupRequest API (13.26 KB, patch)
2012-05-25 09:26 UTC, Sergio Villar 		none Details | Review

Description Sergio Villar 2012-05-21 09:47:26 UTC 
I have noticed that many b.g.o urls are not properly rendered with libsoup master. I researched a bit, and found that by not sending the "Accept-Endoding: gzip, deflate" header all issues are gone. So we should be doing something wrong in the decoding streams.

It's easy to see the issue for example in this URL: https://bugzilla.gnome.org/buglist.cgi?query_format=specific&order=relevance+desc&bug_status=__open__&product=libsoup&content=
Comment 1 Dan Winship 2012-05-23 11:29:54 UTC 
Hm... how does that URL fail for you? It works fine for me with libsoup master
Comment 2 Sergio Villar 2012-05-23 11:35:08 UTC 
Now that you mention it it's actually not 100% reproducible (it was for me the other day), I get the same behaviour in the new bug b.g.o page as well...
Comment 3 Sergio Villar 2012-05-23 11:41:44 UTC 
(In reply to comment #1)
> Hm... how does that URL fail for you? It works fine for me with libsoup master

It fails in the sense that several resources are not retrieved (like CSS) and the page layout is a mess. I could have attached a screenshot but it'd be useless.
Comment 4 Sergio Villar 2012-05-25 09:25:38 UTC 
Created attachment 214924 [details] [review]
Refactor coding-test.c results

Refactors coding-test results. This will help us adding tests for the new API
Comment 5 Sergio Villar 2012-05-25 09:26:36 UTC 
Created attachment 214925 [details] [review]
coding-test: test also SoupRequest API

According to the tests everything should be working fine...
Comment 6 Sergio Villar 2012-05-25 09:45:12 UTC 
Arg! haven't seen your commit from yesterday before....
Comment 7 Dan Winship 2012-05-30 13:02:50 UTC 
Turns out this is GConverterInputStream's fault
Comment 8 Dan Winship 2012-05-30 13:03:20 UTC 
fixed in glib master
Comment 9 Xan Lopez 2012-06-14 10:31:40 UTC 
*** Bug 676613 has been marked as a duplicate of this bug. ***