From 6fe5f1681f47a64070020502a9e862926c799f29 Mon Sep 17 00:00:00 2001
From: Benbb96 <Benbb96@users.noreply.github.com>
Date: Tue, 7 May 2024 14:54:05 +0200
Subject: [PATCH] fix logout in Django 5 with POST action instead of GET

---
 helpdesk/templates/helpdesk/navigation-header.html | 10 +++++++---
 1 file changed, 7 insertions(+), 3 deletions(-)

diff --git a/helpdesk/templates/helpdesk/navigation-header.html b/helpdesk/templates/helpdesk/navigation-header.html
index d63dabe0..5ec166d6 100644
--- a/helpdesk/templates/helpdesk/navigation-header.html
+++ b/helpdesk/templates/helpdesk/navigation-header.html
@@ -58,7 +58,9 @@
             <a class="dropdown-item" href='{% url 'helpdesk:system_settings' %}'><i class="fas fa-fw fa-cogs"></i> {% trans "System Settings" %}</a>
             <div class="dropdown-divider"></div>
             {% endif %}
-            <a class="dropdown-item" href="{% url 'helpdesk:logout' %}"><i class="fas fa-fw fa-sign-out-alt"></i> {% trans "Logout" %}</a>
+            <form action="{% url 'helpdesk:logout' %}" method="post">{% csrf_token %}
+                <button type="submit" class="dropdown-item"><i class="fas fa-fw fa-sign-out-alt"></i> {% trans "Logout" %}</button>
+            </form>
           </div>
         </li>
       {% else %}
@@ -82,8 +84,10 @@
             </div>
             <li class="nav-item dropdown no-arrow">
               {% if user.is_authenticated %}
-                <a class="nav-link dropdown-toggle" href="{% url 'helpdesk:logout' %}" id="userDropdown" role="button" aria-haspopup="true" aria-expanded="false"><i class="fas fa-fw fa-sign-out-alt"></i> {% trans "Logout" %}</a>
-              {% else %}
+                <form action="{% url 'helpdesk:logout' %}" method="post">{% csrf_token %}
+                    <button type="submit" class="nav-link dropdown-toggle" id="userDropdown" role="button" aria-haspopup="true" aria-expanded="false"><i class="fas fa-fw fa-sign-out-alt"></i> {% trans "Logout" %}</button>
+                </form>
+                    {% else %}
                 <a class="nav-link dropdown-toggle" href="{% url 'helpdesk:login' %}?next={% if next %}{{ next|escape }}{% else %}{% url 'helpdesk:home' %}{% endif %}" id="userDropdown" role="button" aria-haspopup="true" aria-expanded="false"><i class="fas fa-fw fa-sign-in-alt"></i> {% trans "Log In" %}</a>
               {% endif %}
             </li>