chromium / suspicious-site-reporter

Extension for reporting suspicious sites to Safe Browsing.
Apache License 2.0
96 stars 22 forks source link

Fetch redirect URLs from referrer chain as setup for redirect-related signals #44

Closed livvielin closed 5 years ago

livvielin commented 5 years ago

Fetch redirect URLs from referrer chain and alert if there are many redirects

Add a new signal examining the redirect chain to provide more information that's helpful in determining if a site is suspicious. This patch also provides the setup for other signals related to the redirect chain, such as whether the site redirected through a TLD associated with a high rate of spam/abuse.

ericlaw1979 commented 5 years ago

Cool! This is, I believe, the first check that relies on data from the specific navigation, rather than the from just the FQDN.

This means that any cache we create can't simply use the url's domain name as a key.

(The current cache isn't key'd at all, which causes #8)

livvielin commented 5 years ago

Cool! This is, I believe, the first check that relies on data from the specific navigation, rather than the from just the FQDN.

This means that any cache we create can't simply use the url's domain name as a key.

(The current cache isn't key'd at all, which causes #8)

Yes, good point! We probably need something like a combination of url's domain name with tab id in this case.