Skip to content

Commit

Permalink
[FIX] auth_session_timeout: problem whereby page is refreshed with F5…
Browse files Browse the repository at this point in the history
…, but /web is a public route, so it does not trigger the session check but it does trigger session save, so the file mtime is updated before the second HTTP call makes the check takes place, and session is not expired
  • Loading branch information
thomaspaulb authored and bosd committed Aug 12, 2023
1 parent e306bf2 commit 56e3d2d
Showing 1 changed file with 6 additions and 2 deletions.
8 changes: 6 additions & 2 deletions auth_session_timeout/models/ir_http.py
Original file line number Diff line number Diff line change
Expand Up @@ -10,7 +10,11 @@ class IrHttp(models.AbstractModel):
@classmethod
def _authenticate(cls, endpoint):
res = super(IrHttp, cls)._authenticate(endpoint=endpoint)
auth_method = endpoint.routing["auth"]
if auth_method == "user" and request and request.session and request.session.uid:
if (
request
and request.session
and request.session.uid
and not request.env["res.users"].browse(request.session.uid)._is_public()
):
request.env.user._auth_timeout_check()
return res

0 comments on commit 56e3d2d

Please sign in to comment.