vikunja/vikunja
8
54
Fork 94
Code Issues 130 Pull Requests 35 Packages Releases 37 Activity

Add Issuer and Subject to user list command #3063

Merged
konrad merged 1 commits from jyte/vikunja:main into main 2025-01-30 17:22:04 +00:00
Conversation 2 Commits 1 Files Changed 1 +4
jyte commented 2025-01-30 14:51:48 +00:00
Contributor
Copy Link

Hi,
i believe that it is possible to endup in the following situation :

  • A user logs in using an authorized OIDC provider
  • A vikunja user is created with the issuer & subject from the OIDC provider
  • The same user logs in using another OIDC provider
  • A 2nd vikunja user is created with a different issuer (possibly all other fields beside created, updated and id are equals)

I think it is important to be able to distinguish them in the user list command

Hi, i believe that it is possible to endup in the following situation : - A user logs in using an authorized OIDC provider - A vikunja user is created with the issuer & subject from the OIDC provider - The same user logs in using another OIDC provider - A 2nd vikunja user is created with a different issuer (possibly all other fields beside `created`, `updated` and `id` are equals) I think it is important to be able to distinguish them in the `user list` command
jyte added 1 commit 2025-01-30 14:51:50 +00:00
Add Issuer and Subject to user list command
All checks were successful
continuous-integration/drone/pr Build is passing
Details
fc05cc384d
frederick commented 2025-01-30 14:54:45 +00:00
Member
Copy Link

Hi jyte!

Thank you for creating a PR!

I've deployed the frontend changes of this PR on a preview environment under this URL: https://3063-main--vikunja-frontend-preview.netlify.app

You can use this url to view the changes live and test them out.
You will need to manually connect this to an api running somewhere. The easiest to use is https://try.vikunja.io/.

This preview does not contain any changes made to the api, only the frontend.

Have a nice day!

Beep boop, I'm a bot.

Hi jyte! Thank you for creating a PR! I've deployed the frontend changes of this PR on a preview environment under this URL: https://3063-main--vikunja-frontend-preview.netlify.app You can use this url to view the changes live and test them out. You will need to manually connect this to an api running somewhere. The easiest to use is https://try.vikunja.io/. This preview does not contain any changes made to the api, only the frontend. Have a nice day! > Beep boop, I'm a bot.
jyte changed title from WIP: Add Issuer and Subject to user list command to Add Issuer and Subject to user list command 2025-01-30 15:30:41 +00:00
konrad merged commit e9444159c5 into main 2025-01-30 17:22:04 +00:00
konrad commented 2025-01-30 17:22:11 +00:00
Owner
Copy Link

Thanks!

Thanks!
Sign in to join this conversation.
Reviewers
No Reviewers
Labels
Clear labels
dependencies
duplicate
This issue or pull request already exists
help wanted
Need some help
invalid
Something is wrong
kind/bug
Something is not working
kind/feature
New feature
needs reproduction
question
More information is needed
security
wontfix
This won't be fixed
No Label
Milestone
No items
No Milestone
Assignees
Clear assignees
dpschen frederick konrad renovate shilch
No Assignees
3 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: vikunja/vikunja#3063
No description provided.
Delete Branch "jyte/vikunja:main"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?