Closed
Bug 1445853
Opened 6 years ago
Closed 6 years ago
Cannot access emails in local folder which includes Japanese character in the folder name
Categories
(MailNews Core :: Backend, defect)
MailNews Core
Backend
Tracking
(thunderbird60 fixed, thunderbird61 fixed)
RESOLVED
FIXED
Thunderbird 61.0
People
(Reporter: t.matsuu, Assigned: emk)
References
Details
(Keywords: regression)
Attachments
(1 file)
3.24 KB,
patch
|
jorgk-bmo
:
review+
jorgk-bmo
:
approval-comm-beta+
|
Details | Diff | Splinter Review |
In case the folder below exists, Thunderbird can access to the folder and cannot access to the email on Windows. Local Folder +- あ (Folder) +- test mail (Email) When I click the email, Thunderbird returns the URL below is not found. mailbox:///C:/Users/%82%A0/AppData/Roaming/Thunderbird/Profiles/xxxxxxxx.x/Mail/Local%20Folders/%82%A0?number=2 ("%82%A0" means "あ" in Shift_JIS.) Environment: Thunderbird 60.0a (20180312030201) win32 OS: Windows 10, x64 Folder Type: Maildir Additional Info: In IMAP, the folder "あ" is generated as "&MEI-" at local Windows storage. However, in Local Folder, it is generated as "あ" at local Windows storage.
Comment 1•6 years ago
|
||
Interesting problem. IMAP folders are encoded in MUTF-7 which uses pure ASCII, hence the &MEI-. On Windows with default locale "en", I get a local folder of "e85eae05" which is hash encoded, see recent discussion in bug 1440278 comment #14 and bug 1440278 comment #19. That bug should have fixed the issue in TB 60. Now you're saying it doesn't work :-( - Did it work previously in earlier versions? Did we break it? I don't have a Japanese system to investigate. Masatoshi-san, could you please take a look.
Flags: needinfo?(VYV03354)
Assignee | ||
Comment 2•6 years ago
|
||
Assignee | ||
Comment 3•6 years ago
|
||
Hmm, apparently my theory was wrong. Please try this patch (I couldn't build the tree due to nsIURL issues.)
Flags: needinfo?(VYV03354)
Comment 4•6 years ago
|
||
Thanks for the patch. C-C was building last night with M-C at rev c488b8d0e074, looks like Dailies with M-C at 97160a734959 (tip at time of writing) aren't busted either: https://treeherder.mozilla.org/#/jobs?repo=comm-central&revision=b7d028d0299cd9456bb2be2f5702d4138aa5abf3 I can do I try build and get you and Takanori-san to try it out.
Severity: normal → critical
Comment 5•6 years ago
|
||
https://treeherder.mozilla.org/#/jobs?repo=try-comm-central&revision=03ba8e234f1a47e79394472e76419dd088e2829a
Comment 6•6 years ago
|
||
Sorry, I forgot, Win32 on buildbot is busted, here's a TaskCluster build: https://treeherder.mozilla.org/#/jobs?repo=try-comm-central&revision=7f572e667c5eb32318e82db7e4b452f06a908ab8 I'll post links to the binaries when done.
Comment 7•6 years ago
|
||
The executables with this patch are now available: Win 32bit: https://queue.taskcluster.net/v1/task/DMzXma6eS3i6Ffho4zmzqg/runs/0/artifacts/public/build/target.zip https://queue.taskcluster.net/v1/task/DMzXma6eS3i6Ffho4zmzqg/runs/0/artifacts/public/build/setup.exe Win 64bit: https://queue.taskcluster.net/v1/task/VsceEfkrReWAEcjREc30gQ/runs/0/artifacts/public/build/target.zip https://queue.taskcluster.net/v1/task/VsceEfkrReWAEcjREc30gQ/runs/0/artifacts/public/build/setup.exe Takanori-san, can you please try this build. I tried to reproduce the problem. I switched my system locale to Japanese and created a folder called あ. I checked in the file system, that folder is present with that name and also あ.msf. I don't have a problem accessing messages in the folder with TB 61 Daily, if I view the source I see %82%A0 in the message URL. The problem was reported for Maildir, so I'll try this next.
Flags: needinfo?(t.matsuu)
Comment 8•6 years ago
|
||
I created a new profile with Local Folder with maildir storage and can't see the problem there either.
Comment 9•6 years ago
|
||
This is a regression of bug 1440278?
Comment 10•6 years ago
|
||
Yes, it would be. Switching the system locale back to English makes those folder inaccessible, the mbox as well as the maildir one.
Comment 11•6 years ago
|
||
FWIW I ran the try build with an existing profile with no ill effects
Updated•6 years ago
|
Blocks: 1440278
Keywords: regression
Reporter | ||
Comment 12•6 years ago
|
||
I tested win32 binary on win64. This works good for me. (In reply to Jorg K (GMT+1) from comment #7) > The executables with this patch are now available: > Win 32bit: > https://queue.taskcluster.net/v1/task/DMzXma6eS3i6Ffho4zmzqg/runs/0/ > artifacts/public/build/target.zip > https://queue.taskcluster.net/v1/task/DMzXma6eS3i6Ffho4zmzqg/runs/0/ > artifacts/public/build/setup.exe > > Win 64bit: > https://queue.taskcluster.net/v1/task/VsceEfkrReWAEcjREc30gQ/runs/0/ > artifacts/public/build/target.zip > https://queue.taskcluster.net/v1/task/VsceEfkrReWAEcjREc30gQ/runs/0/ > artifacts/public/build/setup.exe > > Takanori-san, can you please try this build. > > I tried to reproduce the problem. I switched my system locale to Japanese > and created a folder called あ. I checked in the file system, that folder is > present with that name and also あ.msf. > > I don't have a problem accessing messages in the folder with TB 61 Daily, if > I view the source I see %82%A0 in the message URL. > > The problem was reported for Maildir, so I'll try this next.
Flags: needinfo?(t.matsuu)
Comment 13•6 years ago
|
||
Thanks for the confirmation. Could anyone tell me how to reproduce the problem locally?
Comment 14•6 years ago
|
||
Comment on attachment 8959783 [details] [diff] [review] Work around of mailbox:// URL non-supporting UTF-8 paths I can't test this, but we have positive confirmation that it works. It also "reverts" these changes from bug 1440278: https://hg.mozilla.org/comm-central/rev/1580fa402264#l26.14 https://hg.mozilla.org/comm-central/rev/1580fa402264#l27.14
Attachment #8959783 -
Flags: review+
Updated•6 years ago
|
Assignee: nobody → VYV03354
Status: NEW → ASSIGNED
Updated•6 years ago
|
Attachment #8959783 -
Flags: approval-comm-beta+
Comment 15•6 years ago
|
||
Pushed by mozilla@jorgk.com: https://hg.mozilla.org/comm-central/rev/a1baa186817e Work-around for mailbox:// URL not supporting UTF-8 paths. r=jorgk
Status: ASSIGNED → RESOLVED
Closed: 6 years ago
Resolution: --- → FIXED
Updated•6 years ago
|
Target Milestone: --- → Thunderbird 61.0
Comment 16•6 years ago
|
||
Beta (TB 60): https://hg.mozilla.org/releases/comm-beta/rev/43daf17883d1e37688af61ed84f0a1a87ad85324
status-thunderbird60:
--- → fixed
status-thunderbird61:
--- → fixed
You need to log in
before you can comment on or make changes to this bug.
Description
•