[Author Prev][Author Next][Thread Prev][Thread Next][Author Index][Thread Index]

Re: [tor-bugs] #9803 [HTTPS Everywhere: Chrome]: uri.host is explosive!



#9803: uri.host is explosive!
------------------------------------------+-----------------
     Reporter:  grgoffe                   |      Owner:  pde
         Type:  defect                    |     Status:  new
     Priority:  normal                    |  Milestone:
    Component:  HTTPS Everywhere: Chrome  |    Version:
   Resolution:                            |   Keywords:
Actual Points:                            |  Parent ID:
       Points:                            |
------------------------------------------+-----------------

Comment (by eisahlee):

 I've seen this for quite a while now, and I'm getting '''really''' annoyed
 by the log-spam. Even if I wouldn't start it from the console, these
 superfluous messages could hide "important" ones.

 It seems that the current behavior is caused by the fix for
 [https://trac.torproject.org/projects/tor/ticket/6388]

 I'm not any good with javascript, but I hope that this should handle the
 trivial cases, where HTTPSEverywhere '''really''' should just keep quiet,
 e.g. about:newtab, about:blank, about:addons, etc. This is why I chose the
 most naive approach: Check whether the uri.spec starts with "about:".

 I have no experience with javascript. I couldn't even test this "fix".
 (But at least, it is in a common format, thanks to {{{git format-
 patch}}}.)

 George, and any other dev: If you build a "testing" version (either of
 this patch, or any other way to fix it; I don't care) and send/link me to
 it, I can check whether it works.

 With regards,
 Eisah

-- 
Ticket URL: <https://trac.torproject.org/projects/tor/ticket/9803#comment:2>
Tor Bug Tracker & Wiki <https://trac.torproject.org/>
The Tor Project: anonymity online
_______________________________________________
tor-bugs mailing list
tor-bugs@xxxxxxxxxxxxxxxxxxxx
https://lists.torproject.org/cgi-bin/mailman/listinfo/tor-bugs