Re: [Tails-dev] Please review&merge feature/torbrowser

Delete this message

Reply to this message
Autor: anonym
Data:  
Dla: The Tails public development discussion list
Temat: Re: [Tails-dev] Please review&merge feature/torbrowser
02/05/13 23:48, intrigeri wrote:
>> Bad:
>> * Commit 6629701 breaks the Unsafe Browser. See attached patch for a fix.
>
> Applied and pushed. Nice catch, thanks!


Unfortunately this doesn't seem to be enough. The Unsafe Browser gets
"Unable to find the proxy server" now. I'm kinda sure I tested my patch
successfully yesterday, but I was pretty tired so I wouldn't trust my
memory 100%. I'm currently investigating this.

>> Weird:
>> * In the automated test suite unsafe-browser.feature obviously should
>> fail due to the above, but it fails even earlier then expected; the "(as
>> clearnet)" text in the window title isn't there any more, so sikuli
>> can't even detect the window. This is nothing specific to the Unsafe
>> Browser or even Iceweasel: try `gksu gedit`, or whatever. I couldn't
>> find anything in our git log about us changing this explicitly, but
>> maybe I didn't look deeply enough(?).
>
> I would not be surprised if the hidepid option made it so metacity
> cannot know what user is running this window. See todo/hidepid.


Ah! Thanks!

>> This was not the case in Tails 0.17.2. Any way, we can easily fix
>> this by just cropping away that part of the picture, and that can
>> IMHO even be done after the merge.
>
> Given you can test the result more easily than me, can you please do
> the cropping first thing tomorrow, e.g. before I come up online to do
> the prefs move?


Done. Changes pushed into the offending branch, e.g.
feature/less_kernel_infoleak.

> This kind of late detection of issues motivates me more and more to
> have a more or less automated run of our automated testsuite act as
> a gatekeeper when we merge stuff in.
>
>> intigeri wrote:
>>> I had a quick look, and it seems that some (36) of the prefs we set in
>>> /etc/iceweasel/pref/iceweasel.js are not applied. I haven't checked if
>>> this is a regression against current devel or stable branches.
>>>
>>> I think we should just move all of our iceweasel.js prefs to
>>> user_pref() set in /etc/iceweasel/profile/user.js, and be done with
>>> it. It should work. Only drawback is that it makes it harder for users
>>> to change their config, but IMHO this is rather a feature than a bug.
>>>
>>> If nobody objects, I'll do that, quickly test it, and merge for RC1
>>> early tomorrow.
>
>> I don't object. I guess this means I'm not gonna do the merge then.
>> Well, I'll be around here pretty early tomorrow as well and help out
>> with tests and review.
>
> Awesome! I'll join you around 10AM CEST :)


See you! Hopefully I've found a fix for the proxy issue 'till then.

Cheers!