freedomofpress / securethenews

An automated scanner and web dashboard for tracking TLS deployment across news organizations
https://securethe.news
GNU Affero General Public License v3.0
102 stars 25 forks source link

Onion address override #296

Closed chigby closed 3 years ago

chigby commented 3 years ago

This pull request slightly reorganizes the scan result data to support overriding the onion_available result on a per-site basis. I've done this by:

  1. Renaming the Scan.onion_available to Scan.onion_location_header. This reflects that what the scan is testing for right now is the presence or absence of the header.
  2. Add a onion_address field to the Site model. This is a url field that editors can fill in with the known Onion address for the site.
  3. Replace the Scan.onion_available field with a property of the same name that checks if the scanned site has an onion address. If so, then we always return True, otherwise we return the onion_location_header result.

Fixes #283

chigby commented 3 years ago

Added a panel for onion_address