Skip to content
This repository has been archived by the owner on Feb 16, 2023. It is now read-only.

Add PAPERLESS_LOGOUT_REDIRECT_URL #1258

Merged
merged 1 commit into from
Sep 10, 2021

Conversation

tribut
Copy link
Contributor

@tribut tribut commented Aug 26, 2021

Currently with external SSO (PAPERLESS_ENABLE_HTTP_REMOTE_USER), clicking "Logout" will tell the user that it was successful when in reality they are of course still logged in.

The new setting PAPERLESS_LOGOUT_REDIRECT_URL allows the user to be redirected to the real logout page (of the SSO app). The patch is almost trivial, because the functionality already exists in Django.

When set, the user is redirected to this URL after a logout. Especially
useful in conjunction with PAPERLESS_ENABLE_HTTP_REMOTE_USER and SSO.
@jonaswinkler jonaswinkler merged commit d656de8 into jonaswinkler:master Sep 10, 2021
@oeiber
Copy link

oeiber commented Oct 7, 2021

Hi! I tried to set the envirnonment variable but paperless-ng still redirects to /accounts/logout?

@tribut
Copy link
Contributor Author

tribut commented Oct 7, 2021

@oeiber The logout link will still point to /accounts/logout, but the app should redirect you from there to whatever you set as PAPERLESS_LOGOUT_REDIRECT_URL.

@oeiber
Copy link

oeiber commented Oct 7, 2021

Unfortunately not:
image

I still get the default logout page without redirection.

@tribut
Copy link
Contributor Author

tribut commented Oct 7, 2021

@oeiber In this case, please double-check whether you are using a recent-enough version of paperless. This PR has only been merged on the 10th of september. The jonaswinkler/paperless-ng:latest docker image (which currently points to jonaswinkler/paperless-ng:1.5.0) doesn't yet include it.

@oeiber
Copy link

oeiber commented Oct 8, 2021

Ok. thank you! I‘ll wait for the next version ;-)

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants