Version: 0.9.0 (using KDE 3.4.0 Level "b" , SUSE 9.3) Compiler: gcc version 3.3.5 20050117 (prerelease) (SUSE Linux) OS: Linux (x86_64) release 2.6.11.4-20a-smp When fetching the article list knode assumes that the server understands "list overview.fmt" and handles errors badly: C: GROUP something S: 211 1139 2362 3500 something group selected C: LIST OVERVIEW.FMT S: 501 command syntax error C: <stops, disconnects after some time> If the server does not understand LIST OVERVIEW.FMT then Knode should assume the standard format and go ahead with XOVER or OVER.
SVN commit 419141 by vkrause: Check the response code after sending LIST OVERVIEW.FMT command. KNode now works again with servers that don't support this command. CCBUG: 104422, 106390 M +1 -1 knnntpclient.cpp --- trunk/KDE/kdepim/knode/knnntpclient.cpp #419140:419141 @@ -398,7 +398,7 @@ // see RFC 2980 section 2.1.7 QStrList headerformat; cmd = "LIST OVERVIEW.FMT"; - if (sendCommand( cmd, rep )) { + if ( sendCommand( cmd, rep ) && rep == 215 ) { QStrList tmp; if (getMsg(tmp)) { for(QCString s = tmp.first(); s; s = tmp.next()) {
SVN commit 419143 by vkrause: Backport from trunk for KDE 3.4.2: Check the response code after sending LIST OVERVIEW.FMT command. KNode now works again with servers that don't support this command. BUG: 104422, 106390 M +1 -1 knnntpclient.cpp --- branches/KDE/3.4/kdepim/knode/knnntpclient.cpp #419142:419143 @@ -398,7 +398,7 @@ // see RFC 2980 section 2.1.7 QStrList headerformat; cmd = "LIST OVERVIEW.FMT"; - if (sendCommand( cmd, rep )) { + if ( sendCommand( cmd, rep ) && rep == 215 ) { QStrList tmp; if (getMsg(tmp)) { for(QCString s = tmp.first(); s; s = tmp.next()) {
You need to log in before you can comment on or make changes to this bug.