Closed
Bug 108293
Opened 23 years ago
Closed 3 years ago
support autosubscribe in nntp-js
Categories
(MailNews Core :: Networking: NNTP, defect)
MailNews Core
Networking: NNTP
Tracking
(thunderbird_esr91 unaffected)
RESOLVED
FIXED
97 Branch
Tracking | Status | |
---|---|---|
thunderbird_esr91 | --- | unaffected |
People
(Reporter: sspitzer, Assigned: rnons)
References
Details
Attachments
(1 file)
lazily open the socket, when doing autosubscribe
we should wait until after the user confirms the autosubscribe before opening
the socket.
OS: Windows 2000 → All
Hardware: PC → All
Updated•23 years ago
|
Status: NEW → ASSIGNED
Target Milestone: --- → Future
Updated•20 years ago
|
Product: MailNews → Core
Comment 1•18 years ago
|
||
sorry for the spam. making bugzilla reflect reality as I'm not working on these bugs. filter on FOOBARCHEESE to remove these in bulk.
Assignee: sspitzer → nobody
Status: ASSIGNED → NEW
Updated•16 years ago
|
Product: Core → MailNews Core
Updated•13 years ago
|
Severity: normal → minor
Keywords: perf
Summary: lazily open the socket, when doing autosubscribe → lazily open the news socket, when doing autosubscribe
Target Milestone: Future → ---
Comment 3•3 years ago
|
||
Does this still make sense?
Severity: minor → S4
Flags: needinfo?(remotenonsense)
Assignee | ||
Comment 4•3 years ago
|
||
When clicking a link like news://news.mozilla.org/mozilla.test, if not already subscribed, show a subscribe prompt.
Updated•3 years ago
|
Assignee: nobody → remotenonsense
Status: NEW → ASSIGNED
Assignee | ||
Comment 5•3 years ago
|
||
we should wait until after the user confirms the autosubscribe before opening the socket.
I tested with nsNntpProtocol.cpp, it's already like this. I'm reusing this bug to fix it for nntp-js.
Updated•3 years ago
|
Target Milestone: --- → 97 Branch
Assignee | ||
Updated•3 years ago
|
Keywords: checkin-needed-tb
Pushed by mkmelin@iki.fi:
https://hg.mozilla.org/comm-central/rev/8ccf20c57ff8
Support autosubscribe in nntp-js. r=mkmelin
You need to log in
before you can comment on or make changes to this bug.
Description
•