Skip to content

Commit

Permalink
Backport PR #112: Bug: logging in after clearing session redirects to…
Browse files Browse the repository at this point in the history
… clearing session (#112)

Prevent user from being logged out when clearing the session.

* Create draft PR for #28

* More session keys to avoid clearing

* Changelog entry

Co-authored-by: Flix6x <Flix6x@users.noreply.github.com>
Co-authored-by: F.N. Claessen <felix@seita.nl>
Co-authored-by: Felix Claessen <30658763+Flix6x@users.noreply.github.com>
  • Loading branch information
4 people committed May 6, 2021
1 parent acd6d48 commit 31b3b5c
Show file tree
Hide file tree
Showing 2 changed files with 4 additions and 1 deletion.
1 change: 1 addition & 0 deletions documentation/changelog.rst
Expand Up @@ -8,6 +8,7 @@ v0.4.1 | May XX, 2021

Bugfixes
-----------
* Prevent logging out user when clearing the session [see `PR #112 <http://www.github.com/SeitaBV/flexmeasures/pull/112>`_]
* Prevent user type data source to be created without setting a user [see `PR #111 <https://github.com/SeitaBV/flexmeasures/pull/111>`_]

v0.4.0 | April 29, 2021
Expand Down
4 changes: 3 additions & 1 deletion flexmeasures/ui/utils/view_utils.py
Expand Up @@ -91,7 +91,9 @@ def render_flexmeasures_template(html_filename: str, **variables):

def clear_session():
for skey in [
k for k in session.keys() if k not in ("_id", "user_id", "csrf_token")
k
for k in session.keys()
if k not in ("_fresh", "_id", "_user_id", "csrf_token", "fs_cc", "fs_paa")
]:
current_app.logger.info(
"Removing %s:%s from session ... " % (skey, session[skey])
Expand Down

0 comments on commit 31b3b5c

Please sign in to comment.