Closed Bug 606922 Opened 14 years ago Closed 14 years ago

Add tsk.tr to PSL exceptions

Categories

(Core Graveyard :: Networking: Domain Lists, defect)

Product:
Core Graveyard
Component:
Networking: Domain Lists
Type:
defect
Priority:
Not set
Severity:
normal

Tracking

(status1.9.2 .14-fixed, status1.9.1 .17-fixed)

Status:
RESOLVED FIXED
Tracking Flags:
Tracking Status
status1.9.2 --- .14-fixed
status1.9.1 --- .17-fixed

People

(Reporter: gerv, Assigned: gerv)

References

Details

Attachments

(1 file)

Patch v.1
478 bytes, patch
beltzner
: approval2.0+
dveditz
: approval1.9.2.14+
dveditz
: approval1.9.1.17+
Details | Diff | Splinter Review 
Follow-up to bug 571433.

.tr has a "*.tr" rule, but we need to add an exception for the www.tsk.tr website.

Volkan: we would still appreciate it if you have a definitive list of .tr TLDs, or some contact with the registrar so we can find it out.

Gerv
Hi Gerv,

Sorry, I contacted the official registrar of Turkey through official means but I forgot to write it here. They informed me that tsk.tr and nic.tr are the only examples of .tr tlds. They told me they hadn't changed their policy and they won't be allowing other organizational addresses yet.

Best,
Volkan 

(In reply to comment #0)
> Follow-up to bug 571433.
> 
> .tr has a "*.tr" rule, but we need to add an exception for the www.tsk.tr
> website.
> 
> Volkan: we would still appreciate it if you have a definitive list of .tr TLDs,
> or some contact with the registrar so we can find it out.
> 
> Gerv
Attached patch Patch v.1 — — Splinter Review 
Here is a patch to implement this policy update. Requesting approval.

Gerv
Assignee: nobody → gerv
Status: NEW → ASSIGNED
Attachment #485713 - Flags: approval2.0?
Comment on attachment 485713 [details] [diff] [review]
Patch v.1

a=beltzner
Attachment #485713 - Flags: approval2.0? → approval2.0+
Checked in on trunk.
http://hg.mozilla.org/mozilla-central/rev/e966ef613c21

Gerv
Status: ASSIGNED → RESOLVED
Closed: 14 years ago
Resolution: --- → FIXED
Comment on attachment 485713 [details] [diff] [review]
Patch v.1

Requesting branch approval.

Gerv
Attachment #485713 - Flags: approval1.9.2.14?
Attachment #485713 - Flags: approval1.9.1.17?
Comment on attachment 485713 [details] [diff] [review]
Patch v.1

Approved for 1.9.2.14 and 1.9.1.17, a=dveditz for release-drivers
Attachment #485713 - Flags: approval1.9.2.14?
Attachment #485713 - Flags: approval1.9.2.14+
Attachment #485713 - Flags: approval1.9.1.17?
Attachment #485713 - Flags: approval1.9.1.17+
Ilhan Y.

What is broken? Two cookies ASPSESSIONIDetcetc are set with domain www.kkk.tsk.tr ?
Firefox 6 highlights "tsk.tr" part of http://www.kkk.tsk.tr/
Shouldn't it be "kkk.tsk.tr"?
What would Firefox 6 be? :)

http://www.tsk.tr/ is the primary domain.
(In reply to comment #11)
> What would Firefox 6 be? :)

What do you mean? http://nightly.mozilla.org/

> http://www.tsk.tr/ is the primary domain.

It doesn't seem so; https://www.nic.tr/index.php?USRACTN=YEARSTAT#tsk.tr says that there are 35 domain names that end with tsk.tr
They are separate domainnames, not subdomain. That is what I see.
(In reply to comment #10)
> Firefox 6 highlights "tsk.tr" part of http://www.kkk.tsk.tr/
> Shouldn't it be "kkk.tsk.tr"?

No. That's precisely what the exception changes. *.tr means tsk.tr would be considered a public suffix and so it would highlight kkk.tsk.tr. Adding !tsk.tr means that tsk.tr is not considered a public suffix, so it highlights tsk.tr.

Gerv
...so Firefox is behaving as it should. If you think the PSL is wrong, please open another bug.

Gerv
Depends on: 654953
Product: Core → Core Graveyard
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: