a dot is visible at the end of each (mail)message for servers that don't dot-stuff
Categories
(MailNews Core :: Networking: POP, defect)
Tracking
(thunderbird_esr102 wontfix)
| Tracking | Status | |
|---|---|---|
| thunderbird_esr102 | --- | wontfix |
People
(Reporter: ruurd, Assigned: mkmelin)
References
Details
Attachments
(2 files, 1 obsolete file)
Comment 1•22 years ago
|
||
| Reporter | ||
Comment 2•22 years ago
|
||
Comment 3•22 years ago
|
||
| workaround | ||
| Reporter | ||
Comment 4•22 years ago
|
||
| Reporter | ||
Comment 5•22 years ago
|
||
Comment 6•22 years ago
|
||
Comment 8•21 years ago
|
||
Comment 9•21 years ago
|
||
Comment 10•21 years ago
|
||
Updated•21 years ago
|
Comment 11•21 years ago
|
||
Comment 12•21 years ago
|
||
Comment 13•21 years ago
|
||
Comment 14•20 years ago
|
||
Comment 15•20 years ago
|
||
Comment 16•20 years ago
|
||
Comment 17•18 years ago
|
||
Comment 18•18 years ago
|
||
Comment 19•18 years ago
|
||
Updated•17 years ago
|
Updated•17 years ago
|
Comment 22•5 years ago
|
||
I can confirm that this bug is caused by the combination of an incorrect POP3 LIST response and the Pref: mail.server.default.dot_fix=TRUE
In the program code, mail.server.default.dot_fix=TRUE ensures that TB ignores the end point.
I think this fix was introduced to enable the reception of POP3 servers that did not do dot-doubling in the email body.
Since such servers are rarely used today, I think it makes sense to change the default setting of that pref to FALSE.
What do you think?
Comment 23•5 years ago
|
||
Good question. I don't have expertise in this area, so defer to others.
Comment 24•5 years ago
|
||
Setting dot_fix false definitely fixes the problem for me for emails sent to outlook365 in pop3 mode. With it set at the default true I see a blank line followed by a dot at the end of each text email.
With the same message sent to another server (my isp's server) true or false doesn't matter, don't see the blank line and dot. So seems OK to default it to false.
The problem at the server must be caused by the length given to tb in the in the RETR response since both servers (outlook and isp) return the same final few strings per POP3:5 log. In the log, isp returns a length but outlook doesn't return a length at all.
For the record, I'm attaching the log. Shows first outlook fetch and then isp's.
Updated•5 years ago
|
Comment 25•5 years ago
|
||
(In reply to gene smith from comment #24)
The problem at the server must be caused by the length given to tb in the in the RETR response since both servers (outlook and isp) return the same final few strings per POP3:5 log. In the log, isp returns a length but outlook doesn't return a length at all.
No.
https://searchfox.org/comm-central/source/mailnews/local/src/nsPop3Protocol.cpp#3074
int32_t nsPop3Protocol::RetrResponse(nsIInputStream* inputStream,
[...}
https://searchfox.org/comm-central/source/mailnews/local/src/nsPop3Protocol.cpp#3106
m_pop3ConData->cur_msg_size =
m_pop3ConData->msg_info[m_pop3ConData->last_accessed_msg].size;
If there is no <size> in the RETR response, we use msg_info[].size
And this is set in: nsPop3Protocol::GetList():
https://searchfox.org/comm-central/source/mailnews/local/src/nsPop3Protocol.cpp#2453
m_pop3ConData->msg_info[m_listpos - 1].size = atol(token);
Created attachment 9197599 [details] tb.log.moz_log
2nd email:
| SEND: LIST
| [...]
| RECV: 6 1516
| [...]
| SEND: RETR 6
| [...]
| RECV: +OK 1516 octets
| RECV: [=> 1516 octets]
CHECK - Size fits.
1st email:
| SEND: LIST
| [...]
| RECV: 15 45032
| [...]
| SEND: RETR 15
| [...]
| RECV: (null)
| RECV: [=> 7245 octets]
ERROR: 45032 ≠ 7245
Comment 26•5 years ago
|
||
But the question is: Should we change the default of mail.server.default.dot_fix to FALSE
Comment 27•5 years ago
|
||
If it's only to work around servers that don't dot-stuff, I say just default it to FALSE and avoid the complaints about a final dot appearing in some text emails. If the server doesn't dot-stuff like it should, then it's a server bug (which should now be quite rare but can be worked-around by setting dot_fix back to True).
Comment 28•3 years ago
|
||
Do you agree with comment 27?
| Assignee | ||
Comment 29•3 years ago
|
||
Sound like we should yes.
(I don't see a dot with o365 nor any other servers I use.)
| Assignee | ||
Comment 30•3 years ago
|
||
Updated•3 years ago
|
Comment 31•3 years ago
|
||
Is the Pref still relevant at all?
Or maybe it died with the POP3-JS module?
https://searchfox.org/comm-central/search?q=dot_%3FFix&path=&case=false®exp=true
| Assignee | ||
Comment 32•3 years ago
|
||
Ping noted the same in the review. Indeed it's not implemented, and I don't know of any complaints so far. We should probably just remove the leftovers.
Updated•3 years ago
|
Updated•3 years ago
|
| Assignee | ||
Comment 33•3 years ago
|
||
| Assignee | ||
Updated•3 years ago
|
Comment 34•3 years ago
|
||
Pushed by mkmelin@iki.fi:
https://hg.mozilla.org/comm-central/rev/4b25111bcf1e
remove leftovers of pop3 dot-stuffing (dotFix, dot_fix). r=rnons
Description
•