From ff9c4b65787fda704429815e8aaf743e83c14bd1 Mon Sep 17 00:00:00 2001
From: Codimp <codeurimpulsif@users.noreply.github.com>
Date: Sat, 1 Jun 2024 23:57:37 +0200
Subject: [PATCH] Improve titles and Open Graph properties (#619)
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

* template: add specific titles

with a fallback to "εxodus"

* template: add open graph titles

with type, site_name and image

* template: add open graph description

* template: fix extra space
---
 exodus/analysis_query/templates/query_error.html  |  6 ++++++
 exodus/analysis_query/templates/query_submit.html |  6 ++++++
 exodus/analysis_query/templates/query_upload.html |  6 ++++++
 exodus/analysis_query/templates/query_wait.html   |  6 ++++++
 exodus/reports/templates/report_details.html      |  8 ++++++++
 exodus/reports/templates/reports_home.html        |  6 ++++++
 exodus/reports/templates/reports_list.html        | 13 +++++++++++++
 exodus/trackers/templates/stats_details.html      |  6 ++++++
 exodus/trackers/templates/tracker_details.html    |  6 ++++++
 exodus/trackers/templates/trackers_list.html      |  6 ++++++
 exodus/web/templates/base/base.html               |  8 +++++++-
 exodus/web/templates/base/home.html               |  8 +++++++-
 exodus/web/templates/base/next.html               |  6 ++++++
 exodus/web/templates/base/organization.html       |  6 ++++++
 exodus/web/templates/base/permissions.html        |  6 ++++++
 exodus/web/templates/base/trackers.html           |  6 ++++++
 exodus/web/templates/base/understand.html         |  6 ++++++
 17 files changed, 113 insertions(+), 2 deletions(-)

diff --git a/exodus/analysis_query/templates/query_error.html b/exodus/analysis_query/templates/query_error.html
index d7261bcf..f04efb87 100644
--- a/exodus/analysis_query/templates/query_error.html
+++ b/exodus/analysis_query/templates/query_error.html
@@ -1,4 +1,10 @@
 {% extends "base/base.html"%}
+{% block head %}
+  <meta property="og:description" content="{% trans "Error during an analysis" %}">
+  <meta name="description" content="{% trans "Error during an analysis" %}">
+  <meta property="og:title" content="{% trans "New analysis" %}">
+  <title>{% trans "New analysis" %} - εxodus</title>
+{% endblock %}
 {% block content %}
   <div class="row justify-content-sm-center">
     <div class="col-md-8 col-12 col-centered">
diff --git a/exodus/analysis_query/templates/query_submit.html b/exodus/analysis_query/templates/query_submit.html
index 11d835c0..93a94abf 100644
--- a/exodus/analysis_query/templates/query_submit.html
+++ b/exodus/analysis_query/templates/query_submit.html
@@ -2,6 +2,12 @@
 {% load i18n %}
 {% load static %}
 {% get_current_language as LANGUAGE_CODE %}
+{% block head %}
+  <meta property="og:description" content="{% trans "Submit an application handle or URL to run a new analysis" %}">
+  <meta name="description" content="{% trans "Submit an application handle or URL to run a new analysis" %}">
+  <meta property="og:title" content="{% trans "New analysis" %}">
+  <title>{% trans "New analysis" %} - εxodus</title>
+{% endblock %}
 {% block content %}
   <div class="row justify-content-sm-center">
     <div class="col-md-8 col-12 text-center mb-4">
diff --git a/exodus/analysis_query/templates/query_upload.html b/exodus/analysis_query/templates/query_upload.html
index c2a6f849..bc0bb4aa 100644
--- a/exodus/analysis_query/templates/query_upload.html
+++ b/exodus/analysis_query/templates/query_upload.html
@@ -2,6 +2,12 @@
 {% load i18n %}
 {% load static %}
 {% get_current_language as LANGUAGE_CODE %}
+{% block head %}
+  <meta property="og:description" content="{% trans "Upload an APK file to run a new analysis" %}">
+  <meta name="description" content="{% trans "Upload an APK file to run a new analysis" %}">
+  <meta property="og:title" content="{% trans "New analysis" %}">
+  <title>{% trans "New analysis" %} - εxodus</title>
+{% endblock %}
 {% block content %}
   <div class="row justify-content-sm-center">
     <div class="col-md-8 col-12 text-center mb-4">
diff --git a/exodus/analysis_query/templates/query_wait.html b/exodus/analysis_query/templates/query_wait.html
index a565aa2a..80053230 100644
--- a/exodus/analysis_query/templates/query_wait.html
+++ b/exodus/analysis_query/templates/query_wait.html
@@ -2,6 +2,12 @@
 {% load i18n %}
 {% load static %}
 {% get_current_language as LANGUAGE_CODE %}
+{% block head %}
+  <meta property="og:description" content="{% trans "In progress or completed analysis" %}">
+  <meta name="description" content="{% trans "In progress or completed analysis" %}">
+  <meta property="og:title" content="{% trans "New analysis" %}">
+  <title>{% trans "New analysis" %} - εxodus</title>
+{% endblock %}
 {% block content %}
   <div class="row justify-content-sm-center">
     <div class="col-md-8 col-12 text-center mb-4">
diff --git a/exodus/reports/templates/report_details.html b/exodus/reports/templates/report_details.html
index 8e9d2069..d1605626 100644
--- a/exodus/reports/templates/report_details.html
+++ b/exodus/reports/templates/report_details.html
@@ -1,6 +1,14 @@
 {% extends "base/base.html"%}
 {% load i18n %}
 {% get_current_language as LANGUAGE_CODE %}
+{% block head %}
+  {% if report %}
+    <meta property="og:description" content="{% trans "Known trackers, permissions and informations about this specific version of this application" %}">
+    <meta name="description" content="{% trans "Known trackers, permissions and informations about this specific version of this application" %}">
+    <meta property="og:title" content="{% trans "Report for" %} {{ report.application.handle }} {{ report.application.version }}">
+    <title>{% trans "Report for" %} {{ report.application.handle }} {{ report.application.version }} - εxodus</title>
+  {% endif %}
+{% endblock %}
 {% block content %}
   {% load static %}
   {% if report %}
diff --git a/exodus/reports/templates/reports_home.html b/exodus/reports/templates/reports_home.html
index 5c655b51..6a5b12f2 100644
--- a/exodus/reports/templates/reports_home.html
+++ b/exodus/reports/templates/reports_home.html
@@ -1,6 +1,12 @@
 {% extends "base/base.html" %}
 {% load i18n %}
 {% get_current_language as LANGUAGE_CODE %}
+{% block head %}
+  <meta property="og:description" content="{% trans "Search reports by application name, handle or URL" %}">
+  <meta name="description" content="{% trans "Search reports by application name, handle or URL" %}">
+  <meta property="og:title" content="{% trans "Reports" %}">
+  <title>{% trans "Reports" %} - εxodus</title>
+{% endblock %}
 {% block content %}
   {% load static %}
   <div class="row justify-content-md-center">
diff --git a/exodus/reports/templates/reports_list.html b/exodus/reports/templates/reports_list.html
index 2d12cfef..ffd2e3dd 100644
--- a/exodus/reports/templates/reports_list.html
+++ b/exodus/reports/templates/reports_list.html
@@ -1,6 +1,19 @@
 {% extends "base/base.html" %}
 {% load i18n %}
 {% get_current_language as LANGUAGE_CODE %}
+{% block head %}
+  {% if handle %}
+    <meta property="og:description" content="{% trans "Reports of all analyzed versions of this application" %}">
+    <meta name="description" content="{% trans "Reports of all analyzed versions of this application" %}">
+    <meta property="og:title" content="{% trans "Reports list for" %} {{ handle }}">
+    <title>{% trans "Reports list for" %} {{ handle }} - εxodus</title>
+  {% else %}
+    <meta property="og:description" content="{% trans "All applications reports sorted by latest reports, number of trackers or no known trackers" %}">
+    <meta name="description" content="{% trans "All applications reports sorted by latest reports, number of trackers or no known trackers" %}">
+    <meta property="og:title" content="{% trans "Reports list" %}">
+    <title>{% trans "Reports list" %} - εxodus</title>
+  {% endif %}
+{% endblock %}
 {% block content %}
   {% load static %}
   <div class="row justify-content-md-center">
diff --git a/exodus/trackers/templates/stats_details.html b/exodus/trackers/templates/stats_details.html
index 4496e5c3..86a82216 100644
--- a/exodus/trackers/templates/stats_details.html
+++ b/exodus/trackers/templates/stats_details.html
@@ -1,5 +1,11 @@
 {% extends "base/base.html" %}
 {% load i18n %}
+{% block head %}
+  <meta property="og:description" content="{% trans "Most frequent trackers" %}">
+  <meta name="description" content="{% trans "Most frequent trackers" %}">
+  <meta property="og:title" content="{% trans "Statistics" %}">
+  <title>{% trans "Statistics" %} - εxodus</title>
+{% endblock %}
 {% block content %}
   {% load static %}
   <div class="row justify-content-md-center mb-4">
diff --git a/exodus/trackers/templates/tracker_details.html b/exodus/trackers/templates/tracker_details.html
index 9221c72f..6c003dc9 100644
--- a/exodus/trackers/templates/tracker_details.html
+++ b/exodus/trackers/templates/tracker_details.html
@@ -1,6 +1,12 @@
 {% extends "base/base.html" %}
 {% load i18n %}
 {% load markdown %}
+{% block head %}
+  <meta property="og:description" content="{% trans "Informations about this tracker" %}">
+  <meta name="description" content="{% trans "Informations about this tracker" %}">
+  <meta property="og:title" content="{% trans "Tracker profile for" %} {{ tracker.name }}">
+  <title>{% trans "Tracker profile for" %} {{ tracker.name }} - εxodus</title>
+{% endblock %}
 {% block content %}
   <style>
     h2 {
diff --git a/exodus/trackers/templates/trackers_list.html b/exodus/trackers/templates/trackers_list.html
index 611dbee2..d1491858 100644
--- a/exodus/trackers/templates/trackers_list.html
+++ b/exodus/trackers/templates/trackers_list.html
@@ -1,5 +1,11 @@
 {% extends "base/base.html"%}
 {% load i18n %}
+{% block head %}
+  <meta property="og:description" content="{% trans "List of known trackers" %}">
+  <meta name="description" content="{% trans "List of known trackers" %}">
+  <meta property="og:title" content="{% trans "Trackers" %}">
+  <title>{% trans "Trackers" %} - εxodus</title>
+{% endblock %}
 {% block content %}
   {% if trackers %}
     <div class="row justify-content-sm-center">
diff --git a/exodus/web/templates/base/base.html b/exodus/web/templates/base/base.html
index 9aa87a66..b4fd824c 100644
--- a/exodus/web/templates/base/base.html
+++ b/exodus/web/templates/base/base.html
@@ -14,7 +14,13 @@
     <script src="{% static "js/bootstrap.min.js" %}"></script>
     <script src="{% static "js/handlebars.min.js" %}"></script>
     <meta name="viewport" content="width=device-width, initial-scale=1.0">
-    <title>εxodus</title>
+    <meta property="og:type" content="article">
+    <meta property="og:site_name" content="εxodus">
+    <meta  property="og:image" content="{{ request.scheme }}://{{ request.get_host }}{% static "img/logo_purple_without_text.png" %}">
+    {% block head %}
+      <meta  property="og:title" content="εxodus">
+      <title>εxodus</title>
+    {% endblock %}
   </head>
 
   <body>
diff --git a/exodus/web/templates/base/home.html b/exodus/web/templates/base/home.html
index 796e49a0..e29d21b8 100644
--- a/exodus/web/templates/base/home.html
+++ b/exodus/web/templates/base/home.html
@@ -1,6 +1,12 @@
 {% extends "base/base.html"%}
+{% load i18n %}
+{% block head %}
+  <meta property="og:description" content="{% trans "The privacy audit platform for Android applications" %}">
+  <meta name="description" content="{% trans "The privacy audit platform for Android applications" %}">
+  <meta property="og:title" content="{% trans "Home" %}">
+  <title>{% trans "Home" %} - εxodus</title>
+{% endblock %}
 {% block content %}
-  {% load i18n %}
   <div class="row justify-content-center mb-5 mt-5">
     <div class="col-xl-3 col-lg-3 col-md-3 col-sm-4 col-4 my-auto">
       <img src="/static/img/logo_purple_small.svg" class="mx-auto d-block img-fluid" alt="exodus logo">
diff --git a/exodus/web/templates/base/next.html b/exodus/web/templates/base/next.html
index fc81d09a..bab43daf 100644
--- a/exodus/web/templates/base/next.html
+++ b/exodus/web/templates/base/next.html
@@ -1,6 +1,12 @@
 {% extends "base/base.html" %}
 {% load i18n %}
 {% load static %}
+{% block head %}
+  <meta property="og:description" content="{% trans "We know it is not easy to find alternatives in order to escape trackers within our apps. We prepared a list of tools and things you can do to better manage your privacy on your smartphone." %}">
+  <meta name="description" content="{% trans "We know it is not easy to find alternatives in order to escape trackers within our apps. We prepared a list of tools and things you can do to better manage your privacy on your smartphone." %}">
+  <meta property="og:title" content="{% trans "What's next?" %}">
+  <title>{% trans "What's next?" %} - εxodus</title>
+{% endblock %}
 {% block content %}
   <div class="row justify-content-sm-center">
     <div class="col-md-8 col-12 text-center mb-4">
diff --git a/exodus/web/templates/base/organization.html b/exodus/web/templates/base/organization.html
index ca19eac5..5c6fd7e9 100644
--- a/exodus/web/templates/base/organization.html
+++ b/exodus/web/templates/base/organization.html
@@ -1,6 +1,12 @@
 {% extends "base/base.html" %}
 {% load i18n %}
 {% load static %}
+{% block head %}
+  <meta property="og:description" content="{% trans "Exodus Privacy is a non-profit organization led by hacktivists. Its purpose is to help people get a better understanding of the Android applications tracking issues." %}">
+  <meta name="description" content="{% trans "Exodus Privacy is a non-profit organization led by hacktivists. Its purpose is to help people get a better understanding of the Android applications tracking issues." %}">
+  <meta property="og:title" content="{% trans "The organization" %}">
+  <title>{% trans "The organization" %} - εxodus</title>
+{% endblock %}
 {% block content %}
   <div class="row justify-content-sm-center">
     <div class="col-md-8 col-12 text-center mb-4">
diff --git a/exodus/web/templates/base/permissions.html b/exodus/web/templates/base/permissions.html
index 2387f616..b00ee079 100644
--- a/exodus/web/templates/base/permissions.html
+++ b/exodus/web/templates/base/permissions.html
@@ -1,6 +1,12 @@
 {% extends "base/base.html" %}
 {% load i18n %}
 {% load static %}
+{% block head %}
+  <meta property="og:description" content="{% trans "What type of permissions? Why are some permissions indicated as dangerous? Do I have control over these permissions?" %}">
+  <meta name="description" content="{% trans "What type of permissions? Why are some permissions indicated as dangerous? Do I have control over these permissions?" %}">
+  <meta property="og:title" content="{% trans "Permissions" %}">
+  <title>{% trans "Permissions" %} - εxodus</title>
+{% endblock %}
 {% block content %}
   <div class="row justify-content-sm-center">
     <div class="col-md-8 col-12 text-center mb-4">
diff --git a/exodus/web/templates/base/trackers.html b/exodus/web/templates/base/trackers.html
index 74eedbde..3f130361 100644
--- a/exodus/web/templates/base/trackers.html
+++ b/exodus/web/templates/base/trackers.html
@@ -1,6 +1,12 @@
 {% extends "base/base.html" %}
 {% load i18n %}
 {% load static %}
+{% block head %}
+  <meta property="og:description" content="{% trans "What is a tracker? Are all trackers created equal? Who puts these trackers in?" %}">
+  <meta name="description" content="{% trans "What is a tracker? Are all trackers created equal? Who puts these trackers in?" %}">
+  <meta property="og:title" content="{% trans "Trackers" %}">
+  <title>{% trans "Trackers" %} - εxodus</title>
+{% endblock %}
 {% block content %}
   <div class="row justify-content-sm-center">
     <div class="col-md-8 col-12 text-center mb-4">
diff --git a/exodus/web/templates/base/understand.html b/exodus/web/templates/base/understand.html
index 4f934684..7e420261 100644
--- a/exodus/web/templates/base/understand.html
+++ b/exodus/web/templates/base/understand.html
@@ -1,6 +1,12 @@
 {% extends "base/base.html" %}
 {% load i18n %}
 {% load static %}
+{% block head %}
+  <meta property="og:description" content="{% trans "Better understand what are trackers, permissions and what to do next" %}">
+  <meta name="description" content="{% trans "Better understand what are trackers, permissions and what to do next" %}">
+  <meta property="og:title" content="{% trans "Better understand" %}">
+  <title>{% trans "Better understand" %} - εxodus</title>
+{% endblock %}
 {% block content %}
   <div class="row justify-content-sm-center mb-4">
     <div class="col-md-8 col-12 text-center">