Closed
Bug 533361
Opened 15 years ago
Closed 15 years ago
"TLS" should be called "STARTTLS"
Categories
(Webtools :: ISPDB Server, defect)
Webtools
ISPDB Server
Tracking
(Not tracked)
RESOLVED
FIXED
People
(Reporter: BenB, Assigned: bwinton)
Details
Attachments
(2 files)
16.47 KB,
patch
|
BenB
:
review+
davida
:
review+
|
Details | Diff | Splinter Review |
1.05 KB,
patch
|
BenB
:
review+
BenB
:
review+
|
Details | Diff | Splinter Review |
Socket Type has options "SSL", "TLS" and "none". But TLS != STARTTLS. It should be: "No encryption" (XML: plain) "SSL/TLS (special port)" (XML: SSL) "STARTTLS (normal port)" (XML: STARTTLS)
Reporter | ||
Comment 1•15 years ago
|
||
* No transfer encryption No SSL or TLS XML: "plain" * SSL/TLS Connect via SSL. Needs special SSL port. XML: "SSL" * STARTTLS Use TLS, using STARTTLS. Uses normal port. XML: "STARTTLS"
Reporter | ||
Comment 2•15 years ago
|
||
While you're at it, you could also make a help text for the "port" field. Incoming: "Standard ports: IMAP: 143 (if STARTTLS or No transfer encryption) IMAP: 993 (if SSL/TLS) POP3: 143 (if STARTTLS or No transfer encryption) POP3: 995 (if SSL/TLS)" Outgoing: "Standard ports: SMTP: 587 (if STARTTLS or No transfer encryption, and if available) SMTP: 25 (if STARTTLS or No transfer encryption, and if 587 is not available) SMTP: 465 (if SSL/TLS)"
Assignee | ||
Updated•15 years ago
|
Assignee: nobody → bwinton
Comment 3•15 years ago
|
||
Shouldn't we no push STARTTLS as it's something that was there before the standard TLS was implemented ?
Reporter | ||
Comment 4•15 years ago
|
||
I didn't understand what you said, but STARTTLS (if enforced) is as good as SSL/TLS, there's not much of a difference from a high level perspective.
Comment 5•15 years ago
|
||
One is nicely defined in a standard not the other one.
Reporter | ||
Comment 6•15 years ago
|
||
STARTTLS is defined in RFC 2595, RFC 2487 and others, see en.Wikipedia "STARTTLS"
Assignee | ||
Comment 7•15 years ago
|
||
I'm not updating the convert.py script or the serializer, because the code in Thunderbird already expects the new values: http://mxr.mozilla.org/comm-central/source/mailnews/base/prefs/content/accountcreation/readFromXML.js#78 (and line 102). I also didn't add the help text because I feel that these are terms that the person filling out the form should probably know the definitions of. Thanks, Blake.
Attachment #416629 -
Flags: review?(ben.bucksch)
Reporter | ||
Comment 8•15 years ago
|
||
> I'm not updating ... the serializer Yup, the XML values were/are fine as-is, just the UI was wrong. > these are terms that the > person filling out the form should probably know the definitions of. There's a huge confusion of TLS vs. STARTTLS. I present as evidence: This very bug. The programmer here thought that TLS != SSL, TLS == STARTTLS, when TLS is just the new version of SSL, and STARTTLS a certain way to invoke TLS which allows to upgrade plain connections. The descriptions are there to explain exactly that.
Assignee | ||
Comment 9•15 years ago
|
||
And here is some invalid data in autoconfig.momo.com which I found from this bug. Thanks, Blake.
Attachment #416633 -
Flags: review?(gozer)
Reporter | ||
Updated•15 years ago
|
Attachment #416633 -
Flags: review+
Reporter | ||
Comment 10•15 years ago
|
||
Comment on attachment 416633 [details] [diff] [review] [checked-in] A patch to update autoconfig to change TLS to STARTTLS. blake said I should r+ this.
Attachment #416633 -
Flags: review?(gozer) → review+
Reporter | ||
Comment 11•15 years ago
|
||
Comment on attachment 416633 [details] [diff] [review] [checked-in] A patch to update autoconfig to change TLS to STARTTLS. Commited, revision 57570
Reporter | ||
Comment 12•15 years ago
|
||
Comment on attachment 416629 [details] [diff] [review] A patch to change the values allowed. Sorry for the delay! Looks good to me. r=BenB
Attachment #416629 -
Flags: review?(ben.bucksch) → review+
Assignee | ||
Updated•15 years ago
|
Attachment #416633 -
Attachment description: A patch to update autoconfig to change TLS to STARTTLS. → [checked-in] A patch to update autoconfig to change TLS to STARTTLS.
Assignee | ||
Comment 13•15 years ago
|
||
Comment on attachment 416629 [details] [diff] [review] A patch to change the values allowed. Hopefully this will be a quick review. :) Thanks, Blake.
Attachment #416629 -
Flags: review?(david.ascher)
Assignee | ||
Updated•15 years ago
|
Whiteboard: [patch up, needs r davida]
Comment 14•15 years ago
|
||
Comment on attachment 416629 [details] [diff] [review] A patch to change the values allowed. r=davida, although I feel i should read the spec more. but I suspect the original bug is mine and due to my lack of understanding of the details, so I have no reason to think that my code was better than this one, while I have good reason to think the inverse.
Attachment #416629 -
Flags: review?(david.ascher) → review+
Assignee | ||
Updated•15 years ago
|
Whiteboard: [patch up, needs r davida] → [patch up, needs checkin]
Assignee | ||
Comment 15•15 years ago
|
||
Checked in as http://viewvc.svn.mozilla.org/vc?view=revision&revision=58255
Status: NEW → RESOLVED
Closed: 15 years ago
Resolution: --- → FIXED
Whiteboard: [patch up, needs checkin]
Updated•11 years ago
|
Component: ispdb → ISPDB Server
Product: Mozilla Messaging → Webtools
You need to log in
before you can comment on or make changes to this bug.
Description
•