Package home | Report new bug | New search | Development Roadmap Status: Open | Feedback | All | Closed Since Version 1.5.2

Request #16488 using Net_NNTP_Client::getOverview doesn't seem to compress data
Submitted: 2009-08-03 17:46 UTC
From: fourchette Assigned:
Status: No Feedback Package: Net_NNTP (version 1.4.0)
PHP Version: 5.2.4 OS: Ubuntu 8.04
Roadmaps: (Not assigned)    
Subscription  


 [2009-08-03 17:46 UTC] fourchette (pasde nom)
Description: ------------ i'm not sure if it's feat request or me not using the module correctly using Net_NNTP_Client::getOverview doesn't seem to use compressed data. Net_NNTP_Client::getOverview works just fine. However, since it might require to download a large amount of data, i would like to activate compressed data. I know the server i'm using supports it, but i don't know how nor if Net_NNTP_Client supports it and i've exhausted all the supports channels listed at http://pear.php.net/support/ Test script: --------------- require_once 'Net/NNTP/Client.php'; $nntp = new Net_NNTP_Client(); $nntp->connect('news-europe.giganews.com', 'ssl', 563); $nntp->authenticate('user', 'pass'); $nntp->selectGroup('alt.this.is.a.valid.group'); $size = 500; $start = 1602136576; // any valid message number here $range = $start.'-'.($start+$size-1); $nntp->getOverview($range); // at this point I want data to transit between servers to be compressed and it seems NEt_NNTP_Client offers no way to do it Expected result: ---------------- data transiting between server to be gzipped on the fly Actual result: -------------- data download from server doesn't seem to be compressed.

Comments

 [2009-09-29 23:30 UTC] heino (Heino H. Gehlsen)
Unfortunately Net_NNTP uses Net_Socket, which doesn't support switching between clear and compressed mode. Although it would be of cause be possible to rewrite Net_NNTP to use steams instead, I currently won't invest more time on the package beyond bugfixing and applying patches from other developers, since the package is ancient as in doesn't even use exceptions etc. Also the functionality you ask for is not part of the official NNTP protocol, and finally the protocol itself has recently been updated to version two, which requires the package to be totally revritten anyway...
 [2009-09-29 23:32 UTC] heino (Heino H. Gehlsen)
You are of cause most wellcome to provide a patch!
 [2009-10-04 15:57 UTC] heino (Heino H. Gehlsen)
As a result of looking into bug #16657 i decided to make a few updates to the package anyway. To allow the STARTTLS command tha package had to use socket stream anyway, so on demand compression per command should now be possible to implement. These changes is currently available in SVN. I will however require some patch and solid documentation on how to handle these unofficial extensions to the protocol. It should be possible to make changes to allow easy overriding og some of the involved methods...
 [2010-03-14 21:36 UTC] heino (Heino H. Gehlsen)
-Status: Open +Status: No Feedback
No feedback was provided. The bug is being suspended because we assume that you are no longer experiencing the problem. If this is not the case and you are able to provide the information that was requested earlier, please do so and change the status of the bug back to "Open". Thank you.