[WebInterface] make forced login message clearer

Drakkar-Software · Jan 14, 2025 · e9fe53f · e9fe53f
1 parent b28b0fc
commit e9fe53f
Show file tree

Hide file tree

Showing 2 changed files with 12 additions and 3 deletions.
diff --git a/Services/Interfaces/web_interface/advanced_templates/advanced_index.html b/Services/Interfaces/web_interface/advanced_templates/advanced_index.html
@@ -4,6 +4,15 @@
 <br>
 <div class="card text-center w-50 mx-auto">
     <h2 class="card-header">Welcome to OctoBot's advanced interface</h2>
+    {% with messages = get_flashed_messages(with_categories=true) %}
+      {% if messages %}
+            {% for category, message in messages %}
+            <div class="alert alert-{{ 'danger' if category == 'error' else 'success' }}">
+                {{ message }}
+            </div>
+            {% endfor %}
+      {% endif %}
+    {% endwith %}
     <div class="card-body py-4">
         This interface is providing insights on some OctoBot advanced concepts and should be used once OctoBot basic
         features are understood.

diff --git a/Services/Interfaces/web_interface/login/web_login_manager.py b/Services/Interfaces/web_interface/login/web_login_manager.py
@@ -105,10 +105,10 @@ def active_login_required(func):
     def decorated_view(*args, **kwargs):
         if is_login_required():
             return _login_required_func(func, *args, **kwargs)
-        flask.flash("For security reasons, please enable password authentication in "
-                    "accounts configuration to use this page.",
+        flask.flash(f"For security reasons, please enable password authentication in "
+                    f"accounts configuration to use the {flask.request.path} page.",
                     category=flask_login.LOGIN_MESSAGE_CATEGORY)
-        return flask.redirect(flask.current_app.login_manager.login_view)
+        return flask.redirect('home')
     return decorated_view