Open
Bug 62640
Opened 24 years ago
Updated 2 years ago
Improving the MsgIncomingServer interface for authentication state
Categories
(MailNews Core :: Networking, defect)
MailNews Core
Networking
Tracking
(Not tracked)
ASSIGNED
People
(Reporter: justin, Assigned: lilian.braud)
References
Details
(Keywords: good-first-bug, Whiteboard: [patch-love])
Attachments
(2 files)
|
10.98 KB,
patch
|
Details | Diff | Splinter Review | |
|
48 bytes,
text/x-phabricator-request
|
Details | Review |
Both the POP and IMAP code had separate interfaces for setting and querying the authentication state of the msg server. This patch makes the attribute "authenticated" accessible from the nsIMsgIncomingServer interface, removes the old implementations from the POP and IMAP code, and makes everything use the new attribute on the generic interface. This allows for future extensions (such as X-Sender) to cleanly (and generically) access the authentication state, without requiring special code to deal with each incoming server's implementation.
Sounds good, putterman, sspitzer, mscott?
Comment 4•24 years ago
|
||
Marking NEW so someone can take a look at this patch.
Status: UNCONFIRMED → NEW
Ever confirmed: true
Summary: Improving the MsgIncomingServer interface for authentication state → [PATCH] Improving the MsgIncomingServer interface for authentication state
Updated•20 years ago
|
Product: MailNews → Core
Comment 5•16 years ago
|
||
Seems valid enough to me. Bienvenu?
Assignee: mscott → nobody
QA Contact: esther → mailnews.networking
Summary: [PATCH] Improving the MsgIncomingServer interface for authentication state → Improving the MsgIncomingServer interface for authentication state
Comment 6•16 years ago
|
||
yeah, seems valid. I doubt the patch still applies, but the idea should work.
Updated•16 years ago
|
Product: Core → MailNews Core
Comment 8•5 years ago
|
||
Yes. Just needs someone to dust off the patch.
Updated•3 years ago
|
Assignee: nobody → lilian.braud
Status: NEW → ASSIGNED
Comment 10•2 years ago
|
||
Thanks lilian. I have changed the review request to Magnus
Severity: minor → S3
You need to log in
before you can comment on or make changes to this bug.
Description
•