Bug 1745205 Comment 30 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 
Yes, passwd is always empty with yahoo and probably anything else using oauth2. So the only reason it sometimes works is because ```serverRequiresPasswordForAuthentication``` is false since the connection to Trash is sometimes successfully authenticated so EmptyTrash() gets called and trash gets marked deleted and expunged. See https://searchfox.org/comm-central/rev/91923108c54cd5e574fbaeed2b6af0cbeee37d37/mailnews/imap/src/nsImapIncomingServer.cpp#1858
So not sure check for oauth2 is really helpful since EmptyTrash will still fail if there is no authenticated connection when ```serverRequiresPasswordForAuthentication``` is true.

FYI, there's another bug related to oauth2 where GetPassword() called here: Bug 1714572 comment 36
Revision 1 by
on 
Yes, passwd is always empty with yahoo and probably anything else using oauth2. So the only reason it sometimes works is because ```serverRequiresPasswordForAuthentication``` is false since the connection to Trash is sometimes successfully authenticated so EmptyTrash() gets called and trash gets marked deleted and expunged. See https://searchfox.org/comm-central/rev/91923108c54cd5e574fbaeed2b6af0cbeee37d37/mailnews/imap/src/nsImapIncomingServer.cpp#1858
So not sure check for oauth2 is really helpful since EmptyTrash will still fail if there is no authenticated connection when ```serverRequiresPasswordForAuthentication``` is true.

FYI, there's another bug related to oauth2 where GetPassword() called here: Bug 1714572 comment 36
Edit to FYI...: Actually this describes it better -- bug 1668833 comment 2

Back to Bug 1745205 Comment 30