Tweak logout template examples (#321)

* Tweak logout template examples * Add missing `csrf_token` / `csrf_input` * Shorten snippets by removing `html`/`body` tags * Fix Jinja2 example * Don't call is_authenticated in Jinja2 template `is_authenticated` is a property in recent Django versions (since 1.10) and can not be called since the release of Django 2.0
mozilla · Jan 10, 2020 · eb93c1a · eb93c1a
1 parent 0de702e
commit eb93c1a
Showing 1 changed file with 18 additions and 26 deletions.
diff --git a/docs/installation.rst b/docs/installation.rst
@@ -159,37 +159,29 @@ Django templates example:
 
 .. code-block:: html+django
 
-   <html>
-     <body>
-       {% if user.is_authenticated %}
-         <p>Current user: {{ user.email }}</p>
-         <form action="{% url 'oidc_logout' %}" method="post">
-           <input type="submit" value="logout">
-         </form>
-       {% else %}
-         <a href="{% url 'oidc_authentication_init' %}">Login</a>
-       {% endif %}
-     </body>
-   </html>
-
+   {% if user.is_authenticated %}
+     <p>Current user: {{ user.email }}</p>
+     <form action="{% url 'oidc_logout' %}" method="post">
+       {% csrf_token %}
+       <input type="submit" value="logout">
+     </form>
+   {% else %}
+     <a href="{% url 'oidc_authentication_init' %}">Login</a>
+   {% endif %}
 
 Jinja2 templates example:
 
 .. code-block:: html+jinja
 
-   <html>
-     <body>
-       {% if user.is_authenticated() %}
-         <p>Current user: {{ user.email }}</p>
-         <form action="{% url('oidc_logout') %}" method="post">
-           <input type="submit" value="logout">
-         </form>
-       {% else %}
-         <a href="{{ url('oidc_authentication_init') }}">Login</a>
-       {% endif %}
-     </body>
-   </html>
-
+   {% if request.user.is_authenticated %}
+     <p>Current user: {{ request.user.email }}</p>
+     <form action="{{ url('oidc_logout') }}" method="post">
+       {{ csrf_input }}
+       <input type="submit" value="logout">
+     </form>
+   {% else %}
+     <a href="{{ url('oidc_authentication_init') }}">Login</a>
+   {% endif %}
 
 Additional optional configuration
 =================================