Ignore OCSP (Firefox 32+)

pde · pde · commit 83cab7f67598 · 2014-07-24T14:31:35.000-07:00
Currently we have two solutions to this; now let's merge them...
diff --git a/src/components/https-everywhere.js b/src/components/https-everywhere.js
@@ -478,6 +478,11 @@ HTTPSEverywhere.prototype = {
       
       this.log(DBUG,"Got http-on-modify-request: "+channel.URI.spec);
       var lst = this.getApplicableListForChannel(channel); // null if no window is associated (ex: xhr)
+      // Firefox 32+ lets us infer whether this is an OCSP request
+      if ("allowSTS" in channel && !channel.allowSTS) {
+        this.log(INFO, "Channel with HTTPS rewrites forbidden, probably OCSP, for " + channel.URI.spec);
+        return;
+      }
       if (channel.URI.spec in https_everywhere_blacklist) {
         this.log(DBUG, "Avoiding blacklisted " + channel.URI.spec);
         if (lst) lst.breaking_rule(https_everywhere_blacklist[channel.URI.spec]);
