1 2 false 2009-02-25T13:16:22+00:00 5639 9903 2070 gzip-deflate-support-for-activeresource 1299 8994 open activeresource patch verified 2009-08-14T07:44:53+01:00 5639 Bart ten Brinke Bart ten Brinke Rick http://rails.lighthouseapp.com/projects/8994/tickets/2070 2.x At the moment ActiveResource does not support gzip/deflate. As it does use XML as a transfer medium, this becomes highly inefficient with large datasets. Gzip is one of the oldest remedies for this, and all major application servers for rails support it, but activeresource doesn't. To enable: have ZLib installed and add this tou your ActiveResource base class: base.headers['Accept-Encoding'] = 'deflate' At the moment ActiveResource does not support gzip/deflate. As it does use XML as a transfer medium, this becomes highly inefficient with large datasets. Gzip is one of the oldest remedies for this, and all major application servers for rails support it, but activeresource doesn't. To enable: have ZLib installed and add this tou your ActiveResource base class: base.headers['Accept-Encoding'] = 'deflate' <div><p>At the moment ActiveResource does not support gzip/deflate. As it does use XML as a transfer medium, this becomes highly inefficient with large datasets.</p> <p>Gzip is one of the oldest remedies for this, and all major application servers for rails support it, but activeresource doesn't.</p> <p>To enable: have ZLib installed and add this tou your ActiveResource base class: base.headers['Accept-Encoding'] = 'deflate'</p></div> 0 At the moment ActiveResource does not support gzip/deflate. As it does use XML as a transfer medium, this becomes highly inefficient with large datasets. Gzip is one of the oldest remedies for this, and all major application servers for rails support it, but activeresource doesn't. To enable: have ZLib installed and add this tou your ActiveResource base class: base.headers['Accept-Encoding'] = 'deflate' <div><p>At the moment ActiveResource does not support gzip/deflate. As it does use XML as a transfer medium, this becomes highly inefficient with large datasets.</p> <p>Gzip is one of the oldest remedies for this, and all major application servers for rails support it, but activeresource doesn't.</p> <p>To enable: have ZLib installed and add this tou your ActiveResource base class: base.headers['Accept-Encoding'] = 'deflate'</p></div> false 2009-02-25T13:16:22+00:00 5639 --- {} 9903 2070 gzip-deflate-support-for-activeresource 0 8994 new activeresource patch 2009-02-25T13:16:25+00:00 5639 Bart ten Brinke Bart ten Brinke http://rails.lighthouseapp.com/projects/8994/tickets/2070 2.x 1 Nice! +1 <div><p>Nice! +1</p></div> false 2009-02-25T15:29:15+00:00 5639 --- {} 9903 2070 gzip-deflate-support-for-activeresource 0 8994 new activeresource patch 2009-02-25T15:29:16+00:00 19965 José Valim Bart ten Brinke http://rails.lighthouseapp.com/projects/8994/tickets/2070 2.x 1 +1 <div><p>+1</p></div> false 2009-02-25T15:58:45+00:00 5639 --- {} 9903 2070 gzip-deflate-support-for-activeresource 0 8994 new activeresource patch 2009-02-25T15:58:47+00:00 3825 Rob Sanheim Bart ten Brinke http://rails.lighthouseapp.com/projects/8994/tickets/2070 2.x 1 1 false 2009-03-02T05:26:38+00:00 5639 --- :assigned_user: 9903 2070 gzip-deflate-support-for-activeresource 0 8994 new activeresource patch 2009-03-02T05:26:41+00:00 141 Michael Koziarski Bart ten Brinke Rick http://rails.lighthouseapp.com/projects/8994/tickets/2070 2.x 1 1 Can this be included in 2.3? Would be a nice combination with the XML parsers. <div><p>Can this be included in 2.3? Would be a nice combination with the XML parsers.</p></div> false 2009-03-10T19:10:14+00:00 5639 --- :tag: activeresource patch 9903 2070 gzip-deflate-support-for-activeresource 0 8994 new activeresource patch verified 2009-03-10T19:10:56+00:00 5639 Bart ten Brinke Bart ten Brinke Rick http://rails.lighthouseapp.com/projects/8994/tickets/2070 2.x 1 1 This is cool, but how are you passing Accept-Encoding in the first place? Too late for 2.3 as we're down to the final release candidate. <div><p>This is cool, but how are you passing Accept-Encoding in the first place?</p> <p>Too late for 2.3 as we're down to the final release candidate.</p></div> false 2009-03-10T20:25:27+00:00 5639 --- :state: new 9903 2070 gzip-deflate-support-for-activeresource 0 8994 open activeresource patch verified 2009-03-10T20:39:54+00:00 85 Jeremy Kemper Bart ten Brinke Rick http://rails.lighthouseapp.com/projects/8994/tickets/2070 2.x 1 1 Last line of the ticket: To enable add this to your ActiveResource base class via a Mixin: base.headers['Accept-Encoding'] = 'deflate' <div><p>Last line of the ticket:</p> <p>To enable add this to your ActiveResource base class via a Mixin: base.headers['Accept-Encoding'] = 'deflate'</p></div> false 2009-03-10T21:44:30+00:00 5639 --- {} 9903 2070 gzip-deflate-support-for-activeresource 0 8994 open activeresource patch verified 2009-03-10T22:09:09+00:00 5639 Bart ten Brinke Bart ten Brinke Rick http://rails.lighthouseapp.com/projects/8994/tickets/2070 2.x 1 1 Is zlib packed with Rails 2.3? Because that will allow for some simplification of the patch. Too bad it's to late for 2.3. <div><p>Is zlib packed with Rails 2.3? Because that will allow for some simplification of the patch. Too bad it's to late for 2.3.</p></div> false 2009-03-10T21:48:57+00:00 5639 --- {} 9903 2070 gzip-deflate-support-for-activeresource 0 8994 open activeresource patch verified 2009-03-10T22:13:42+00:00 5639 Bart ten Brinke Bart ten Brinke Rick http://rails.lighthouseapp.com/projects/8994/tickets/2070 2.x 1 1 zlib is part of the ruby standard library <div><p>zlib is part of the ruby standard library</p></div> false 2009-03-10T22:17:22+00:00 5639 --- {} 9903 2070 gzip-deflate-support-for-activeresource 0 8994 open activeresource patch verified 2009-03-10T22:52:36+00:00 85 Jeremy Kemper Bart ten Brinke Rick http://rails.lighthouseapp.com/projects/8994/tickets/2070 2.x 1 1 Doh :) Repackaged. More tests + added an ActiveResource.enable_delfate option to set the headers for you. <div><p>Doh :) Repackaged. More tests + added an ActiveResource.enable_delfate option to set the headers for you.</p></div> false 2009-03-10T23:33:59+00:00 5639 --- {} 9903 2070 gzip-deflate-support-for-activeresource 0 8994 open activeresource patch verified 2009-03-11T00:07:19+00:00 5639 Bart ten Brinke Bart ten Brinke Rick http://rails.lighthouseapp.com/projects/8994/tickets/2070 2.x 1 2 +1 <div><p>+1</p></div> false 2009-05-19T20:12:32+01:00 5639 --- {} 9903 2070 gzip-deflate-support-for-activeresource 0 8994 open activeresource patch verified 2009-05-19T20:12:35+01:00 57396 Justin Reese Bart ten Brinke Rick http://rails.lighthouseapp.com/projects/8994/tickets/2070 2.x 1 2 Prefer not adding inflate! to net/http since it inflates AS for a special case. Could you use a utility method on the connection instead? <div><p>Prefer not adding inflate! to net/http since it inflates AS for a special case. Could you use a utility method on the connection instead?</p></div> false 2009-05-19T20:26:51+01:00 5639 --- {} 9903 2070 gzip-deflate-support-for-activeresource 0 8994 open activeresource patch verified 2009-05-19T20:26:56+01:00 85 Jeremy Kemper Bart ten Brinke Rick http://rails.lighthouseapp.com/projects/8994/tickets/2070 2.x 1 2 Gzip is something that really belongs in net/http as it is something you do to a packet. <div><p>Gzip is something that really belongs in net/http as it is something you do to a packet.</p></div> false 2009-05-19T21:13:56+01:00 5639 --- {} 9903 2070 gzip-deflate-support-for-activeresource 0 8994 open activeresource patch verified 2009-05-19T21:14:01+01:00 5639 Bart ten Brinke Bart ten Brinke Rick http://rails.lighthouseapp.com/projects/8994/tickets/2070 2.x 1 2 Also I think net http should really support this as every thing that does http transfers will benefit from this. <div><p>Also I think net http should really support this as every thing that does http transfers will benefit from this.</p></div> false 2009-05-19T21:15:55+01:00 5639 --- {} 9903 2070 gzip-deflate-support-for-activeresource 0 8994 open activeresource patch verified 2009-05-19T21:16:01+01:00 5639 Bart ten Brinke Bart ten Brinke Rick http://rails.lighthouseapp.com/projects/8994/tickets/2070 2.x 1 2 It should, but that feature should be a patch to upstream net/http rather than hiding out in Active Support. Then Active Resource can either use net/http's inflate support or fall back to its own utility method. <div><p>It should, but that feature should be a patch to upstream net/http rather than hiding out in Active Support.</p> <p>Then Active Resource can either use net/http's inflate support or fall back to its own utility method.</p></div> false 2009-05-19T22:00:02+01:00 5639 --- {} 9903 2070 gzip-deflate-support-for-activeresource 0 8994 open activeresource patch verified 2009-05-19T22:00:07+01:00 85 Jeremy Kemper Bart ten Brinke Rick http://rails.lighthouseapp.com/projects/8994/tickets/2070 2.x 1 2 Problem is that body is a private attribute of HTTP::NET, so if all the magick is in AR, it would make the patch even more ugly. <div><p>Problem is that body is a private attribute of HTTP::NET, so if all the magick is in AR, it would make the patch even more ugly.</p></div> false 2009-05-19T23:07:19+01:00 5639 --- {} 9903 2070 gzip-deflate-support-for-activeresource 0 8994 open activeresource patch verified 2009-05-19T23:07:22+01:00 5639 Bart ten Brinke Bart ten Brinke Rick http://rails.lighthouseapp.com/projects/8994/tickets/2070 2.x 1 2 Jeremy mentions both: > Prefer not adding inflate! to net/http since it inflates AS for a special case. then.. > that feature should be a patch to upstream net/http I hope this one makes it in somewhere. Theres zero performance penalty of having this option turned off, and the ability to shorten average request duration if turned on. It meets good usability standards as well. D <div><p>Jeremy mentions both:<br></p> <blockquote> <p>Prefer not adding inflate! to net/http since it inflates AS for a special case. then..<br> that feature should be a patch to upstream net/http</p> </blockquote> <p>I hope this one makes it in somewhere. Theres zero performance penalty of having this option turned off, and the ability to shorten average request duration if turned on. It meets good usability standards as well.</p> <p>D</p></div> false 2009-08-14T06:00:44+01:00 5639 --- {} 9903 2070 gzip-deflate-support-for-activeresource 0 8994 open activeresource patch verified 2009-08-14T06:00:50+01:00 27186 Dean Radcliffe Bart ten Brinke Rick http://rails.lighthouseapp.com/projects/8994/tickets/2070 2.x 1 2 I don't really understand the problem with patching HTTP:NET. They will probably never do it upstream because of the zlib requirements. Also rails already has a truckload of mix-ins for base ruby classes (take hash for example). And as stated above. There is no penalty if left disabled. <div><p>I don't really understand the problem with patching HTTP:NET. They will probably never do it upstream because of the zlib requirements. Also rails already has a truckload of mix-ins for base ruby classes (take hash for example).</p> <p>And as stated above. There is no penalty if left disabled.</p></div> false 2009-08-14T07:44:48+01:00 5639 --- {} 9903 2070 gzip-deflate-support-for-activeresource 0 8994 open activeresource patch verified 2009-08-14T07:44:53+01:00 5639 Bart ten Brinke Bart ten Brinke Rick http://rails.lighthouseapp.com/projects/8994/tickets/2070 2.x 99973072d287f268c4cae7c70aada7ded79e8a5f text/plain 2009-02-25T13:16:22+00:00 gzip_support.diff 92753 3135 5639 http://rails.lighthouseapp.com/attachments/92753/gzip_support.diff 4abcec16400f6db4e6fd1f455ccdeeb91851558b text/plain 2009-03-10T23:33:59+00:00 gzip_support2.diff 98290 5066 5639 http://rails.lighthouseapp.com/attachments/98290/gzip_support2.diff