Closed Bug 450649 Opened 13 years ago Closed 13 years ago

The link "click'here' to get the rest of truncated mail" is not working. (when truncated POP3 mail is moved/copied to other account's folder, including "Global Inbox" case)


(Thunderbird :: Mail Window Front End, defect, P1)


(Not tracked)

Thunderbird 3.0a3


(Reporter: henry.nestler, Assigned: Bienvenu)



(Keywords: regression, Whiteboard: investigating)


(1 file, 1 obsolete file)

User-Agent:       Mozilla/5.0 (X11; U; Linux x86_64; en-US; rv: Gecko/20070730 SUSE/ Firefox/
Build Identifier: 2008073000, 20080813030431

The Link "Click 'here'" for getting the rest of truncated pop3 mail is not working for accounts with "Global inbox".

Reproducible: Always

Steps to Reproduce:
1.Have a pop3 account with Disk Space limit 50KB, enable "Global Inbox"
2.Get a mail that exceeded the limit, for example 150KB
4.Click the Link "click 'here' to get the rest of truncated mail"
Actual Results:  

Expected Results:  
Load the rest of mail.

This is a cleaner description from bug 450486.

The mail header has theses entries:
X-Account-Key: account1
X-UIDL: UID5412-1173099605

File popstate.dat in this account has entries like this:
*pop username
b UID5412-1173099605 1218700081

The non working Link from "Global inbox" in TB3:

For accounts with non "Global Inbox", Link are working in TB3.
Here is such Link:

Same profile directory (same data) used with older Thunderbird Version the Link from "Global inbox" is working as:
Version: unspecified → Trunk
When a truncate mail in Inbox of non-Global-Inbox account is copied to other
account's Inbox, "internal path name of account" field of the link's URI was changed to one for account who owns mail folder. (Tested with tb trunk 2008/8/09 build)

>(In original account's Inbox)
>  mailbox://<user_id_1>@<servernme_1>/Inbox?number=nn&messageid=mm&uidl=uuuu...
>(Copied to Local Folders's Inbox)
>  mailbox://nobody@local%20folders/Inbox?number=nn&messageid=mm&uidl=uuuu...
>(Copied to other non-Global-Inbox account's Inbox)
>  mailbox://<user_id_2>@<servernme_2>/Inbox?number=nn&messageid=mm&uidl=uuuu...

Mail can be moved to other account's mail folder(Global Inbox is one of such cases). X-Account-Key: header should be used if exists, upon generation of link URI for additional download of truncated mail.

Ever confirmed: true
Severity: normal → major
Flags: blocking-thunderbird3.0b1?
I think this should block since global inbox is the default
Flags: blocking-thunderbird3.0b1? → blocking-thunderbird3.0b1+
Assignee: nobody → bienvenu
Summary: The link "click'here' to get the rest of truncated mail" is not working. → The link "click'here' to get the rest of truncated mail" is not working. (when truncated POP3 mail is moved/copied to other account's folder, including "Global Inbox" case)
Muttley, the problem exist also, if pop3 account receives truncated mails directly into the global inbox.

Move/copy such mail is an additional effect for same problem.
putting in b1 milestone, and blocking b1
Flags: blocking-thunderbird3+
Target Milestone: --- → Thunderbird 3.0b1
Switching for b1 flags to target milestones, to avoid flag churn.
Priority: -- → P1
the file uri for the link url is wrong - it's just file:///Inbox, which isn't sufficient. The spec looks OK when you hover over the link, however.
Whiteboard: investigating
It's not the file uri that's the issue - that looks the same in the success case, and the failure case (the failure case being the global inbox case, which is the default). In the failure case, nsPop3Service::NewURI is getting an orphan folder back from the uri (no parent), and in the success case, it's getting a real folder. The URIs look OK in both cases, but it must not be quite right.
the problem is that the partial download url is "local%20folders" and the folder uri is "Local%20Folders" so someone has lower-cased it...
we're probably going through some uri conversion that ensures that host names are lower case. In this case, "Local Folders" is the host name, in our uri scheme.
nsStandardURL::BuildNormalizedSpec is calling NormalizeIDN, which is lower-casing our host name.

We have a lot of uri's in prefs that have "mailbox://Local%20Folders..." so we can't just lower case the host name when creating the uri's for the local folder accounts, since uri's are case-sensitive.  Maybe we can use the original uri...
Keywords: regression
Hardware: PC → All
this just gets more and more complicated - I think the file:///Inbox uri really is the issue, or at least part of it. In 2.0, where this works, the file uri points to the actual mailbox file. This allows us to scan the message, and figure out the account key, which tells us which pop3 server the message is actually on. It would be very helpful to get a regression range for this. A lot of this code changed when we got rid of nsFileSpec but I'm not at all sure that's the cause. When and how gecko creates mailbox urls for displayed messages seems to have changed quite a bit between 2.0 and 3.0 so it's difficult to compare the working code with the code that's failing, but the uri's we're getting from the doc shell are completely different; one is of the account form (nobody@local%20folders//Inbox), the one that works is the actual path to the mail folder on disk.
In 2.0, where this works, mousing over the "get the rest" link correctly shows the url for the pop3 server the message was delivered to in the status bar.
Attached patch possible fix (obsolete) — Splinter Review
this fixes the problem - the change to nsPop3Service to get the msg hdr from the folder instead of from the message url is the thing that got me over the hump. I don't know for sure if the originalspec changes are required. 

I believe GetMessageHdr is only failing for the partial download url, not mailbox urls in general, though I need to verify that. If that's the case, then the change not to use it in the partial download case is OK.
Attached patch proposed fix Splinter Review
turns out this is sufficient.
Attachment #336199 - Attachment is obsolete: true
Attachment #336221 - Flags: superreview?(neil)
Attachment #336221 - Flags: review?(bugzilla)
Attachment #336221 - Flags: superreview?(neil) → superreview+
Comment on attachment 336221 [details] [diff] [review]
proposed fix 

>       nsCOMPtr<nsIMsgMessageUrl> msgUrl = do_QueryInterface(mailboxUrl, &rv);
>       NS_ENSURE_SUCCESS(rv,rv);
>-      msgUrl->GetMessageHeader(getter_AddRefs(msgHdr));
Nit: This was the only reference to msgUrl so you can remove all of this.
Comment on attachment 336221 [details] [diff] [review]
proposed fix 

r=me with the change that Neil commented on.
Attachment #336221 - Flags: review?(bugzilla) → review+
fixed with Neil's nit addressed, changeset 247:8d26696691ae
Closed: 13 years ago
Resolution: --- → FIXED
Duplicate of this bug: 453828
Thanks, is working now in nightly build 20080904030144 (Linux/de).
Verified by comment 19.
You need to log in before you can comment on or make changes to this bug.