<p dir="ltr">Hi Jacob,</p>
<p dir="ltr">I agree completely with the idea, but instead of using simple=true wouldn't it be better to use a separate tag like <simpletarget /> or similar?</p>
<p dir="ltr">It would improve the overall readability of the rulesets and clearly separate targets that need a regex and targets that don't.</p>
<p dir="ltr">True, it will require a bit more tweaking on the code, but it might allow for further expansion of the use we do sqlite db (if implemented in a separate table) or, anyway, be translated in the db as '$target' in the targets table and the word 'simple' in the corresponding rule, while still keeping the code clean and maintain the expandability.</p>
<p dir="ltr">Cheers,</p>
<p dir="ltr">Claudio</p>
<div class="gmail_quote">On Feb 15, 2015 8:11 PM, "Jacob Hoffman-Andrews" <<a href="mailto:jsha@eff.org">jsha@eff.org</a>> wrote:<br type="attribution"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Hi all,<br>
<br>
In the process of updating rulesets to conform with the new ruleset<br>
coverage test requirements, and reviewing your fixes (thank you!), I've<br>
noticed a large number of target hosts just need simple updates. For<br>
instance:<br>
<br>
> <ruleset name="VirtualBox"><br>
><br>
>   <target host="<a href="http://virtualbox.org" target="_blank">virtualbox.org</a>" /><br>
>   <target host="<a href="http://forums.virtualbox.org" target="_blank">forums.virtualbox.org</a>" /><br>
>   <target host="<a href="http://www.virtualbox.org" target="_blank">www.virtualbox.org</a>" /><br>
>   <target host="<a href="http://update.virtualbox.org" target="_blank">update.virtualbox.org</a>" /><br>
><br>
><br>
>   <securecookie host="^\.www\.virtualbox\.org$" name=".+" /><br>
>   <securecookie host="^\.forums\.virtualbox\.org$" name=".+" /><br>
><br>
><br>
>   <!-- Certificate is not valid for "<a href="http://virtualbox.org" target="_blank">virtualbox.org</a>"  --><br>
>   <rule from="^http://(?:www\.)?virtualbox\.org/"<br>
>     to="<a href="https://www.virtualbox.org/" target="_blank">https://www.virtualbox.org/</a>" /><br>
><br>
>   <rule from="^http:"<br>
>       to="https:" /><br>
This ruleset rewrites '<a href="http://virtualbox.org" target="_blank">virtualbox.org</a>' to '<a href="http://www.virtualbox.org" target="_blank">www.virtualbox.org</a>'; For all<br>
other target hosts it simply upgrades to HTTPS. However, one must read<br>
all of the rules in the ruleset carefully to determine that.<br>
<br>
I'd like to propose another update to the ruleset grammar, simple="true":<br>
<br>
<target host="<a href="http://forums.virtualbox.org" target="_blank">forums.virtualbox.org</a>" simple="true" /><br>
<br>
For URLs on that target host, HTTPS Everywhere would always upgrade to<br>
HTTPS, not bothering to evaluate any further regexes.<br>
<br>
If we go this route, I think I can update all the rules from the past<br>
week or so automatically, so no need to pause in fixing rulesets.<br>
<br>
The other change I'm considering is to add an optional 'maintainer'<br>
field for rulesets. This would contain a list of names and email<br>
addresses to be contacted when we have to disable a rule for failing<br>
tests. It would still be fine to submit rulesets with no maintainer field.<br>
<br>
What do you think?<br>
<br>
Thanks,<br>
Jacob<br>
<br>
_______________________________________________<br>
HTTPS-Everywhere mailing list<br>
<a href="mailto:HTTPS-Everywhere@lists.eff.org">HTTPS-Everywhere@lists.eff.org</a><br>
<a href="https://lists.eff.org/mailman/listinfo/https-everywhere" target="_blank">https://lists.eff.org/mailman/listinfo/https-everywhere</a><br>
</blockquote></div>