Bug 1841348 Comment 22 Edit History

Note: The actual edited comment in the bug view page will always show the original commenter’s name and original timestamp.

Original comment by
on 
The impact may be less than I had assumed.

For example, SmtpServer.jsm calls promptPassword(), but I tested daily, and the SMTP code still works. So in some places our implementation uses an implementation from elsewhere.

Based on that, can we wait for more broken places to be discovered, and focus on the LDAP code for now?

(The attached patch also has a fix for some RNP code, but that was an easy change.)
Revision 1 by
on 
The impact may be less than I had assumed.

For example, SmtpServer.jsm calls promptPassword(), but I tested daily, and the SMTP code still works. So in some places our mail code uses a different prompt implementation.

Based on that, can we wait for more broken places to be discovered, and focus on the LDAP code for now?

(The attached patch also has a fix for some RNP code, but that was an easy change.)

Back to Bug 1841348 Comment 22