132616 - Cyrus server: Accesing email with embedded images in IMAP sub-folder displays alert and saving attachments fails

Status: VERIFIED FIXED

(MailNews Core :: Networking: IMAP, defect)

Description

[Chris Blackstone]

With Build ID 2002032108 on Mac OS X, Mac OS 9, and Win2k, emails sent from
Novell Groupwise to Groupwise account, forwarded via Groupwise rule to an IMAP
Server "lose" their attachment when moved to an IMAP folder.

I can view emails correctly as long as they are in my INBOX.
As soon as I move the email to a folder on the IMAP server, I can't download PDF,
GIF, or JPG attachments. Each time I try to access an attachment I get an error
message that says 
"Would you like to subscribe to user/cblackst/INBOX/bugs" (the emails are in a
folder called bugs)

This error appears immediately when I try to view emails with GIF and JPG images
attached (I'm guessing there's some relation to Mozilla trying to display the
images inline).
The error appears if I try to save the PDF attachment of an email.
I only get this error after I have moved the emails to a folder on my IMAP
server and then try to access the attachments.

does this make any sense?

Comment 1

[David :Bienvenu]

Changing summary and adding keywords. I suspect this is due to our handling of
the cyrus hierarchy delimiter of ".".

Comment 2

[David :Bienvenu]

Chris, did you set up that other users namespace yourself or did the server do
it? It might help if you attached a protocol log:

http://www.mozilla.org/quality/mailnews/mail-troubleshoot.html#imap

Comment 3

[Chris Blackstone]

The server did it.

I tried to get a protocol log, but there is no seamonkey directory on the Mac
and the instructions don't work in mac OS X, which is my main working OS. 

Comment 4

[David :Bienvenu]

ok, I have a fix for this. I need to iron out some issues with imap mod-utf8
folder names before I attach the fix.

Comment 5

[David :Bienvenu]

Attachment: proposed fix

need to use FindOnlineSubfolder to find sub folder, because it uses the online
name, which is what's in the urls. Also fixed a utf7 issue.

Comment 6

[David :Bienvenu]

Cavin, can you review?

Comment 7

[David :Bienvenu]

we really need to fix this for moz 1.0 - otherwise, we'll get lots of complaints
from cyrus users

Comment 8

[Cavin Song]

Comment on attachment 75789 [details] [diff] [review]
proposed fix

r=cavin.

Comment 9

[(not reading, please use [e-mail] instead)]

Comment on attachment 75789 [details] [diff] [review]
proposed fix

sr=sspitzer

Comment 10

[Asa Dotzler [:asa]]

Comment on attachment 75789 [details] [diff] [review]
proposed fix

a=asa (on behalf of drivers) for checkin to the 1.0 trunk

Comment 11

[David :Bienvenu]

fix checked in.

Comment 12

[Chris Blackstone]

Works correctly in 

Mozilla/5.0 (Macintosh; U; PPC Mac OS X; en-US; rv:0.9.9+) Gecko/20020326
Build ID:  2002032603

Comment 13

[Chris Blackstone]

Works correctly in

Mozilla/5.0 (Windows; U; Windows NT 5.0; en-US; rv:0.9.9+) Gecko/20020326
Build ID: 2002032603

Comment 14

[David :Bienvenu]

Chris, could you mark it verified, then? Thx.

Comment 15

[Gregg Meehan]

Marking as VERIFIED - reporter states this is no longer an issue

Comment 16

[Bill Mason]

*** Bug 132229 has been marked as a duplicate of this bug. ***