feature/redirect_to_oidc_logout_url_on_logout #1340

Merged
konrad merged 8 commits from viehlieb/api:feature/redirect_to_oidc_logout_url_on_logout into main 2022-12-18 18:26:32 +00:00
Contributor

This Pull Request enables redirect to oidc provider on logout.
Make sure to add logouturl to the config file.
If logouturl is not given or '' - then there is no redirect


auth:
	local:
    	enabled: false
openid:
	enabled: true
	providers:
		logouturl: http://your_oidc_url/if-session.end/vikunja/

The example shows an example LogoutURL for authentik.

Counterpart to:
vikunja/frontend#2805

This Pull Request enables redirect to oidc provider on logout. Make sure to add logouturl to the config file. If logouturl is not given or '' - then there is no redirect ``` auth: local: enabled: false openid: enabled: true providers: logouturl: http://your_oidc_url/if-session.end/vikunja/ ``` The example shows an example LogoutURL for authentik. Counterpart to: https://kolaente.dev/vikunja/frontend/pulls/2805
viehlieb added 3 commits 2022-12-08 14:23:59 +00:00
viehlieb changed title from feature/redirect_to_oidc_logout_url_on_logout to WIP feature/redirect_to_oidc_logout_url_on_logout 2022-12-08 14:42:21 +00:00
Author
Contributor

I just realized, i have to work around the standard openid library of golang.
so i marked as WIP

I just realized, i have to work around the standard openid library of golang. so i marked as WIP
viehlieb added 1 commit 2022-12-08 14:45:42 +00:00
continuous-integration/drone/pr Build is failing Details
b248474a16
remove unneccessary line in openid package
viehlieb changed title from WIP feature/redirect_to_oidc_logout_url_on_logout to feature/redirect_to_oidc_logout_url_on_logout 2022-12-08 14:51:34 +00:00
Author
Contributor

Not WIP anymore.. fixxed the unneccessary line

Not WIP anymore.. fixxed the unneccessary line
konrad requested changes 2022-12-08 15:39:04 +00:00
konrad left a comment
Owner
Please fix the lint errors: https://drone.kolaente.de/vikunja/api/4523/1/11
@ -312,2 +312,4 @@
# The auth url to send users to if they want to authenticate using OpenID Connect.
authurl:
# The oidc logouturl that users will be redirected to on logout
# leave empty or delete key, if you do not want to be redirected
Owner

These two lines will show up in one line in the docs. Can you change them to something like

        # The oidc logouturl that users will be redirected to after logout.
        # Leave empty or delete the entry if you do not want to be redirected.
These two lines will show up in one line in the docs. Can you change them to something like ``` # The oidc logouturl that users will be redirected to after logout. # Leave empty or delete the entry if you do not want to be redirected. ```
viehlieb added 1 commit 2022-12-09 13:30:17 +00:00
continuous-integration/drone/pr Build is failing Details
d419fb86db
did the swag - fixing lint
viehlieb added 2 commits 2022-12-09 13:53:05 +00:00
konrad added 1 commit 2022-12-18 18:17:04 +00:00
konrad scheduled this pull request to auto merge when all checks succeed 2022-12-18 18:18:12 +00:00
konrad merged commit a79b1de2d0 into main 2022-12-18 18:26:32 +00:00
konrad deleted branch feature/redirect_to_oidc_logout_url_on_logout 2023-01-11 21:02:47 +00:00
Sign in to join this conversation.
No reviewers
No Milestone
No Assignees
2 Participants
Notifications
Due Date
The due date is invalid or out of range. Please use the format 'yyyy-mm-dd'.

No due date set.

Dependencies

No dependencies set.

Reference: vikunja/vikunja#1340
No description provided.