Closed Bug 11727 Opened 25 years ago Closed 25 years ago

[DOGFOOD] IMAP: Need password dialog prompt

Categories

(MailNews Core :: Backend, defect, P3)

x86
Windows NT
defect

Tracking

(Not tracked)

VERIFIED FIXED

People

(Reporter: jefft, Assigned: Bienvenu)

References

Details

Need password prompt.
I believe mscott may be working on this, and there may be issues so you should probably wait until he gets back on Monday...
Blocks: 11091
Assignee: jefft → mscott
Reassign to mscott as David metioned that Scott is working on it. A few implementation notes: o The password dialog needs to be brought up under the mozilla thread instead of imap thread. o nsIImapMiscellaneous::GetPasswordForUser() is the method we want to implement for the password dialog. o Make sure the proxy event SetNotifyCompletion(PR_TRUE) when creating GetPassworForUserProxyEvent() o And call WaitForFEEventCompletion() after calling m_imapMiscellaneousSink->GetPasswordForUser(). o The nsIImapMiscellaneousSink::GetPasswordForUser() interface may need to be changed to take the returning password.
Status: NEW → ASSIGNED
Target Milestone: M10
I'll take a look at this. Although, the dialog guys have written a proxy prompt dialog implementation which is designed for imap where you want to bring up a dialog from the imap thread and not the ui thread? Do we still need to bring it up from the UI thread if we are going to use this helper dialog implementation instead?
No, in theory we can just bring up the dialog from the imap thread. It would be nice to get rid of a proxy event...but it's nice to have a fallback in case the dialog guys don't succeed 100% on all platforms.
Scott, if you haven't done any work on this, I can take it. Lemme know.
Assignee: mscott → bienvenu
Status: ASSIGNED → NEW
I'll take this so Scott can work on local progress.
alright David!
Status: NEW → RESOLVED
Closed: 25 years ago
Resolution: --- → FIXED
fixed, mostly in nsImapProtocol.cpp and nsImapMailFolder.cpp
Status: RESOLVED → VERIFIED
Using 1999100617M10 commercial build on win98 and 1999100615M10 commercial build on mac and 1999100118M10 commercial build on linux this is fixed. Verified
Product: MailNews → Core
Product: Core → MailNews Core
You need to log in before you can comment on or make changes to this bug.