1794185 - "Get Next 500 Messages" does not work reliably in NNTP-JS

Status: RESOLVED FIXED

(MailNews Core :: Networking: NNTP, defect)

Description

[b4]

User Agent: Mozilla/5.0 (Windows NT 10.0; Win64; x64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/105.0.0.0 Safari/537.36

Steps to reproduce:

Subscribe to alt.comp.software.thunderbird and retrieve the first 500 headers. Then use "Get Next 500 Messages" to get more headers.

Actual results:

Usually another 500 or up to 1000 headers are fetched, but then fetching stops. Switching to NNTP C++ reliably fetches all 6300+ headers eventually.

It's not reproducible 100%, the amount of messages fetched before the malfunction varies.

Expected results:

500 messages should have been fetched all the time.

Comment 1

[Ping Chen [:rnons] (away)]

Attachment: Bug 1794185 - Fix "Get Next n Messages" for nntp-js. r=mkmelin

Update nsIDBFolderInfo.knownArtsSet based on XOVER range, doesn't matter if artciles in the range don't exist on the server.

Comment 2

[Ping Chen [:rnons] (away)]

The problem is getLastMissingRange keeps returning the same missing range https://searchfox.org/comm-central/rev/0b0768b7a4c022316bb6d09e84677d5f9ffe6535/mailnews/news/src/NntpNewsGroup.jsm#85`

C: XOVER 5576-5577 
S: 224 No articles in 5576-5577

Comment 3

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/comm-central/rev/dfc327877efc
Fix "Get Next n Messages" for nntp-js. r=mkmelin

Comment 4

[Ping Chen [:rnons] (away)]

Attachment: 1794185-esr102.patch

[Approval Request Comment]
Regression caused by (bug #): nntp-js
User impact if declined: "Get Next N Messages" does not work in some cases
Testing completed (on c-c, etc.): beta
Risk to taking this patch (and alternatives if risky): low

Comment 5

[Wayne Mery (:wsmwk)]

Comment on attachment 9300057 [details] [diff] [review]
1794185-esr102.patch

[Triage Comment]
Approved for esr102

Comment 6

[Rob Lemley [:rjl]]

Thunderbird 102.4.2:
https://hg.mozilla.org/releases/comm-esr102/rev/8e31084a0ed7