Add tsk.tr to PSL exceptions

RESOLVED FIXED

Status

()

RESOLVED FIXED
8 years ago
8 years ago

People

(Reporter: gerv, Assigned: gerv)

Tracking

unspecified
Points:
---

Firefox Tracking Flags

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

Details

Attachments

(1 attachment)

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

Comment 1

8 years ago
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
Created attachment 485713 [details] [diff] [review]
Patch v.1

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
Last Resolved: 8 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+

Comment 9

8 years ago
Ilhan Y.

What is broken? Two cookies ASPSESSIONIDetcetc are set with domain www.kkk.tsk.tr ?

Comment 10

8 years ago
Firefox 6 highlights "tsk.tr" part of http://www.kkk.tsk.tr/
Shouldn't it be "kkk.tsk.tr"?

Comment 11

8 years ago
What would Firefox 6 be? :)

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

Comment 12

8 years ago
(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
You need to log in before you can comment on or make changes to this bug.