Add tsk.tr to PSL exceptions

RESOLVED FIXED

Status

()

Core
Networking: Domain Lists
RESOLVED FIXED
7 years ago
6 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)

(Assignee)

Description

7 years ago
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

7 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
(Assignee)

Comment 2

7 years ago
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+
(Assignee)

Comment 4

7 years ago
Checked in on trunk.
http://hg.mozilla.org/mozilla-central/rev/e966ef613c21

Gerv
Status: ASSIGNED → RESOLVED
Last Resolved: 7 years ago
Resolution: --- → FIXED
(Assignee)

Comment 5

7 years ago
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+
(Assignee)

Comment 7

7 years ago
http://hg.mozilla.org/releases/mozilla-1.9.1/rev/772887c7305f
http://hg.mozilla.org/releases/mozilla-1.9.2/rev/c72a7bddca3a

Gerv
status1.9.1: --- → .17-fixed
status1.9.2: --- → .14-fixed

Comment 8

6 years ago
Seems broken. See it here http://www.kkk.tsk.tr/

(https://bugzilla.mozilla.org/show_bug.cgi?id=451833#c77)

Comment 9

6 years ago
Ilhan Y.

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

Comment 10

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

Comment 11

6 years ago
What would Firefox 6 be? :)

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

Comment 12

6 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.
(Assignee)

Comment 13

6 years ago
(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
(Assignee)

Comment 14

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

Gerv

Comment 15

6 years ago
https://bugzilla.mozilla.org/show_bug.cgi?id=654953
Depends on: 654953
You need to log in before you can comment on or make changes to this bug.