Fix first_party_host to use uri.hostname insteads (EFForg#18760)

Chan Chak Shing · zoracon · commit 6d50b914b931 · 2020-01-22T12:39:31.000-08:00
diff --git a/chromium/background-scripts/background.js b/chromium/background-scripts/background.js
@@ -335,7 +335,7 @@ function onBeforeRequest(details) {
 
     // Check if an user has disabled HTTPS Everywhere on this site.  We should
     // ensure that all subresources are not run through HTTPS Everywhere as well.
-    browserSession.putTab(details.tabId, 'first_party_host', uri.host, true);
+    browserSession.putTab(details.tabId, 'first_party_host', uri.hostname, true);
   }
 
   if (disabledList.has(browserSession.getTab(details.tabId, 'first_party_host', null)) ||

Original file line number	Diff line number	Diff line change
`@@ -335,7 +335,7 @@ function onBeforeRequest(details) {`
`335`	`335`
`336`	`336`	`// Check if an user has disabled HTTPS Everywhere on this site. We should`
`337`	`337`	`// ensure that all subresources are not run through HTTPS Everywhere as well.`
`338`		`- browserSession.putTab(details.tabId, 'first_party_host', uri.host, true);`
	`338`	`+ browserSession.putTab(details.tabId, 'first_party_host', uri.hostname, true);`
`339`	`339`	`}`
`340`	`340`
`341`	`341`	`if (disabledList.has(browserSession.getTab(details.tabId, 'first_party_host', null)) \|\|`