Search on Server fails in Newsgroups (NNTP) when there are many hits
Categories
(MailNews Core :: Networking: NNTP, defect)
Tracking
(thunderbird_esr102+ affected, thunderbird108 affected)
People
(Reporter: infofrommozilla, Assigned: rnons)
References
(Regression)
Details
(Keywords: regression)
Attachments
(7 files)
STR:
Subscribe to Newsserver: nntp.aioe.org Newsgroup: de.test
Context menu of group -> "Search Messages..."
Check "Run search on server"
Search for: "From" - "contains" - "Alfred Peters"
Result: Error dialog and the result is incomplete.
Error Console:
10:09:13.768 mailnews.nntp: Got an error id=-304 NntpClient.jsm:849:18
_actionError resource:///modules/NntpClient.jsm:849
_actionXPatResponse resource:///modules/NntpClient.jsm:835
_onData resource:///modules/NntpClient.jsm:215
(Async: EventHandlerNonNull)
_onOpen resource:///modules/NntpClient.jsm:142
(Async: EventHandlerNonNull)
connect resource:///modules/NntpClient.jsm:109
withClient resource:///modules/NntpIncomingServer.jsm:614
getNewNews resource:///modules/NntpService.jsm:104
performExpand resource:///modules/NntpIncomingServer.jsm:331
_toggleRow chrome://messenger/content/folderPane.js:1382
toggleOpenState chrome://messenger/content/folderPane.js:1310
changeOpenState chrome://global/content/elements/tree.js:1246
MozTreeChildren chrome://global/content/elements/tree.js:110
(The quick search finds 229 messages.)
The error occurs with the TB102 with activated nntp.jsmodule and also with the current daily.
Reporter | ||
Comment 1•2 years ago
|
||
The error occurs when the result set is split into 2 or more parts.
This is the first response with the response code (status:) 221, the status text and an empty data field.
Reporter | ||
Updated•2 years ago
|
Reporter | ||
Comment 2•2 years ago
|
||
The second part of the response contains the first data. The status and statusText fields are undefined.
No error is generated here.
Reporter | ||
Comment 3•2 years ago
|
||
The third part of the response again contains data and the undefined statusText field but no status field.
This then generates the error message.
Reporter | ||
Comment 4•2 years ago
|
||
(In reply to Alfred Peters from comment #0)
Error Console:
10:09:13.768 mailnews.nntp: Got an error id=-304 NntpClient.jsm:849:18
This message is from the trunk source code.
https://searchfox.org/comm-central/rev/a1bdcf428c90c504990fede91c86ea8087a1937d/mailnews/news/src/NntpClient.jsm#835
Reporter | ||
Comment 5•2 years ago
|
||
If I search not just a group but the whole server for something that doesn't deliver a hit at all, the TB sometimes seems to choke.
It then starts like this, where the response message is not properly separated into status code and text.
Again, this does not produce an error.
Reporter | ||
Comment 6•2 years ago
|
||
In the response for the next group, the status code is missing again and the error is generated.
Assignee | ||
Comment 7•2 years ago
|
||
I have a patch on local to prevent the error, but it still shows "No matches found".
I also tested 91 and with nntp.jsmodule=false on 102, showed "No matches found" as well.
Is it the same for you? Is this feature long broken?
Reporter | ||
Comment 8•2 years ago
|
||
No, the C module works fine.
I can't say if the JS module ever worked.
Reporter | ||
Comment 9•2 years ago
|
||
JS module: That the search works for few hits obviously depends on the server.
nntp.aioe.org delivers the response line and the data in separate packets as described in (comment #1) and (comment #2). This then fails.
My local news server delivers the first data already in the first data packet together with the response line. As long as the data fits into the first packet, the search with the JS module then works fine.
Assignee | ||
Comment 10•2 years ago
|
||
Updated•2 years ago
|
Assignee | ||
Comment 11•2 years ago
|
||
OK, I see how it works now. Even if XPAT returns 100 results, if 10 of the 100 articles are on local, only 10 will be shown in the search result. That's why I got "No matches found" yesterday.
So "search on server" is only useful to check if a message is still on the server?
Assignee | ||
Updated•2 years ago
|
Comment 12•2 years ago
|
||
Reporter | ||
Updated•2 years ago
|
Reporter | ||
Comment 13•2 years ago
|
||
(In reply to Ping Chen (:rnons) from comment #11)
So "search on server" is only useful to check if a message is still on the server?
You can thus search any header without having to keep it offline.
(Edit: [deleted] - sorry that was nonsense.)
Description
•