Page 1 of 3
feature: in apply to trusted mode, hide non script domains
Posted: Fri Mar 05, 2010 11:24 pm
by al_9x
In "apply these restrictions to whitelisted" mode, allowing domains affects only script permissions (is that right?). The objects and iframes are allowed or not based only on the global embeddings settings, irrespective of the whitelist. Therefore, optionally at least, it would make sense to not show the domains for non-script resources in the NoScript menu, since allowing them makes no difference (currently allowing such a domain refreshes the page, needlessly, I think). Hiding them makes the menu better reflect the permission needs of the page and less cluttered.
Re: feature: in apply to trusted mode, hide non script domains
Posted: Sat Mar 06, 2010 9:52 pm
by therube
Test page:
http://djeault.blogspot.com/2007/02/iframe-test.html
You can Allow the 'object' *@
http://www.djo.ca which will then show the center page IFRAME contents - without clicking the placemarker.
Now, not sure what that means

?
Re: feature: in apply to trusted mode, hide non script domains
Posted: Sun Mar 07, 2010 12:48 am
by al_9x
In your example, per my proposal, djo.ca would not be shown as a candidate for whitelisting. When "apply to trusted" is checked, the whitelist does not affect embeddings permissions. Allowing djo.ca or even showing it, serves no purpose, and if anything, is misleading, as it suggests that the page might benefit or at least change from allowing it, whereas, in fact, it will make no difference.
Re: feature: in apply to trusted mode, hide non script domains
Posted: Tue Mar 09, 2010 9:27 am
by Giorgio Maone
It does make sense, indeed.
Actually I'd make it a default, controlled by an
about:config preference.
Re: feature: in apply to trusted mode, hide non script domai
Posted: Sat Apr 10, 2010 8:38 pm
by al_9x
I just noticed something, when "no placeholders from untrusted" is checked, the embedding domains should appear in the untrusted menu.
Re: feature: in apply to trusted mode, hide non script domai
Posted: Tue Apr 13, 2010 12:02 pm
by Giorgio Maone
Looking into that, thanks.
Re: feature: in apply to trusted mode, hide non script domai
Posted: Wed Apr 14, 2010 10:22 pm
by Giorgio Maone
al_9x wrote:I just noticed something, when "no placeholders from untrusted" is checked, the embedding domains should appear in the untrusted menu.
Could you make a more detailed test case?
Re: feature: in apply to trusted mode, hide non script domai
Posted: Wed Apr 14, 2010 10:33 pm
by al_9x
Giorgio Maone wrote:Could you make a more detailed test case?
The
page therube posted can serve as an example. djo.ca is no longer a candidate for whitelisting (in apply to trusted mode), which is good, since whitelisting it will not make a difference, but if "no placeholder from untrusted" is set, it should be available for blacklisting, because blacklisting it will alter the page (remove the placeholder)
Re: feature: in apply to trusted mode, hide non script domai
Posted: Thu Apr 15, 2010 11:03 am
by al_9x
in .63
1)
http://www.djo.ca and
http://djo.ca appear in the untrusted menu. It seems to be ignoring the domain level setting. Should be just djo.ca (2nd level is the default)
2) with alwaysShowObjectSources=true, djo.ca is shown in addition to
http://www.djo.ca and
http://djo.ca, and they are in different places in the menu
It looks like there are two different pieces of code putting domains in the untrusted menu. The old code (alwaysShowObjectSources=true) knows how to do it right.
Re: feature: in apply to trusted mode, hide non script domai
Posted: Thu Apr 15, 2010 12:39 pm
by Giorgio Maone
al_9x wrote:in .63
1)
http://www.djo.ca and
http://djo.ca appear in the untrusted menu. It seems to be ignoring the domain level setting. Should be just djo.ca (2nd level is the default)
2) with alwaysShowObjectSources=true, djo.ca is shown in addition to
http://www.djo.ca and
http://djo.ca, and they are in different places in the menu
It looks like there are two different pieces of code putting domains in the untrusted menu. The old code (alwaysShowObjectSources=true) knows how to do it right.
Yes, since the source of the needed information is different than the "normal" flow, I've been forced to slap an extra patch locally, where the untrusted menu is built one site after another.
Making it behave the way you're suggesting requires a major refactoring in several places and likely a performance penalty too.
I'm looking into that, but it definitely couldn't make into this release which needed to be pushed today because Stefano Di Paola will give a public talk tomorrow about a Base64 issue which is covered by it.
Re: feature: in apply to trusted mode, hide non script domai
Posted: Sun Apr 18, 2010 5:54 pm
by al_9x
.64 is not hiding embedding domains.
Re: feature: in apply to trusted mode, hide non script domai
Posted: Sun Apr 18, 2010 8:41 pm
by Giorgio Maone
Please check .65, thanks.
Re: feature: in apply to trusted mode, hide non script domai
Posted: Sun Apr 18, 2010 10:17 pm
by al_9x
in .65 you're hiding (main menu) blogspot.com and showing blogger.com and djo.ca, should be the reverse.
Re: feature: in apply to trusted mode, hide non script domai
Posted: Sun Apr 18, 2010 11:05 pm
by Giorgio Maone
Please check .66, thanks.
Re: feature: in apply to trusted mode, hide non script domai
Posted: Mon Apr 19, 2010 3:56 am
by al_9x
Giorgio Maone wrote:Please check .66, thanks.
Main menu looks ok. If it's not too difficult, consider (when alwaysShowObjectSources=false) showing embedding only domains in the untrusted menu only if "no placeholders from untrusted" is set (otherwise marking such a domain untrusted has no effect). In .66 they are shown regardless, which is better than never, as it was .62. That would be in the spirit of this RFE, of only showing items that make a difference.