[Tails-dev] Please review bugfix/do_not_allow_listing_fonts …

Delete this message

Reply to this message
Author: sajolida
Date:  
To: The Tails public development discussion list
Old-Topics: Re: [Tails-dev] Tails 0.14 vs. iceweasel 10.0.9esr-1
Subject: [Tails-dev] Please review bugfix/do_not_allow_listing_fonts [was: Tails 0.14 vs. iceweasel 10.0.9esr-1]
On 06/01/13 18:58, intrigeri wrote:
> Hi,
>
> sajolida@??? wrote (06 Jan 2013 17:48:02 GMT) :
>> While doing that I found out that TBB had two font related preferences
>> that were not in Tails:
>
>>     pref("browser.display.max_font_attempts", 10);
>>     pref("browser.display.max_font_count", 5);

>
>> Those preferences activate the Torbrowser patch called
>> 0011-Limit-the-number-of-fonts-per-document.patch. Adding them to
>> ~/.mozilla/firefox/default/user.js removes the difference in results
>> from those two online fingerprint checks.
>
> Nice catch!
>
>> So, shall I add those to
>> config/chroot_local-includes/etc/iceweasel/profile/user.js, or maybe
>> there is a better place?
>
> In general, it's better to use
> config/chroot_local-includes/etc/iceweasel/pref/iceweasel.js when
> possible, and only put into user.js settings that really need to (as
> in: they're not taken into account or overriden, for various reasons,
> if set in iceweasel.js), so I'd suggest trying to patch iceweasel.js
> (before building an ISO, as our profile is generated at build time
> nowadays) first.


Ok, done, built and tested.

branch: bugfix/do_not_allow_listing_fonts.
commit: a1b143c
ticket: todo/do_not_allow_listing_fonts

Candidate for Tails 0.17