Closed Bug 68094 Opened 25 years ago Closed 25 years ago

SMTP needs to set correct notification callbacks on channel

Categories

(MailNews Core :: Networking: SMTP, defect)

defect
Not set
normal

Tracking

(Not tracked)

VERIFIED FIXED

People

(Reporter: bryner, Assigned: mscott)

References

Details

(Keywords: embed)

Attachments

(2 files)

Currently, SMTP does not call SetNotificationCallbacks on the SocketTransport that it creates. This means that PSM is unable to get the correct nsIPrompt for displaying a cert db password dialog.
Blocks: 67856
Keywords: embed
Attached patch patchSplinter Review
mscott: any estimate on when you'll be able to review the attached patch?
Keywords: review
bryner, can you post -uw diffs? Looks like there are a lot of white space changes in your patch and I can't easily tell what's real and what's not =)
Attached patch diff with -wSplinter Review
thanks bryner. looks good. sr=mscott
jgmyers, can you add r= to go with mscott's sr=?
sspitzer, can you r= this?
looks good to me. r=sspitzer.
checked in
Status: NEW → RESOLVED
Closed: 25 years ago
Resolution: --- → FIXED
QA Contact: esther → stephend
verified fixed.
Status: RESOLVED → VERIFIED
Product: MailNews → Core
Product: Core → MailNews Core
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: