Closed Bug 787968 Opened 11 years ago Closed 11 years ago

Replace nsIMsgIncomingServer.useSSL with nsMsgSocketType.SSL (missing from patch for Bug 525238)

Categories

(SeaMonkey :: MailNews: General, defect)

defect
Not set
normal

Tracking

(seamonkey2.12 fixed, seamonkey2.13 fixed, seamonkey2.14 fixed, seamonkey2.15 fixed)

RESOLVED FIXED
seamonkey2.15
Tracking Status
seamonkey2.12 --- fixed
seamonkey2.13 --- fixed
seamonkey2.14 --- fixed
seamonkey2.15 --- fixed

People

(Reporter: philip.chee, Assigned: philip.chee)

References

Details

(Keywords: regression)

Attachments

(1 file)

Bug 525238 removed Components.interfaces.nsIMsgIncomingServer.useSSL and replaced it with Components.interfaces.nsMsgSocketType.SSL but BenB forgot to change SeaMonkey's version of CopyMessageUrl()
[Approval Request Comment]
Regression caused by (bug #): Bug 525238 - Authentication method (plaintext/encrypted password, Kerberos/GSSAPI etc.) should be explicit, not just "Secure authentication"
User impact if declined: Unable to copy message location if server protocol is snews.
Testing completed (on m-c, etc.): baked in Thunderbird since 2010-03-19
Risk to taking this patch (and alternatives if risky): minimal
String changes made by this patch: none.
Attachment #657877 - Flags: superreview?(neil)
Attachment #657877 - Flags: review?(neil)
Attachment #657877 - Flags: approval-comm-release?
Attachment #657877 - Flags: approval-comm-beta?
Attachment #657877 - Flags: approval-comm-aurora?
Attachment #657877 - Flags: superreview?(neil)
Attachment #657877 - Flags: review?(neil)
Attachment #657877 - Flags: review+
Pushed to comm-central:
http://hg.mozilla.org/comm-central/rev/163203274af7
Attachment #657877 - Flags: approval-comm-release?
Attachment #657877 - Flags: approval-comm-release+
Attachment #657877 - Flags: approval-comm-beta?
Attachment #657877 - Flags: approval-comm-beta+
Attachment #657877 - Flags: approval-comm-aurora?
Attachment #657877 - Flags: approval-comm-aurora+
> but BenB forgot to change SeaMonkey's version of CopyMessageUrl()

Sorry. Thanks for fixing it.
You need to log in before you can comment on or make changes to this bug.