Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Admin api changes #1974

Merged
merged 14 commits into from
Dec 3, 2024
Merged

Admin api changes #1974

merged 14 commits into from
Dec 3, 2024

Conversation

FreddyDevelop
Copy link
Contributor

Description

Currently not all information for a user is returned in the admin API. This PR adds

  • password
    • get
    • create
    • update
    • delete
  • webauthn
    • list
    • get
    • delete
  • otp
    • get
    • delete
  • sessions
    • list
    • delete

Implementation

Added new admin handlers to the admin router.

Tests

Use the new endpoints in the admin API.

@FreddyDevelop FreddyDevelop marked this pull request as ready for review November 15, 2024 11:13
lfleischmann
lfleischmann reviewed Nov 28, 2024
View reviewed changes
backend/handler/webauthn_credential_admin.go Outdated Show resolved Hide resolved
backend/handler/webauthn_credential_admin_test.go Outdated Show resolved Hide resolved
backend/handler/webauthn_credential_admin_test.go Outdated Show resolved Hide resolved
backend/handler/session_admin_test.go Outdated Show resolved Hide resolved
backend/handler/session_admin_test.go Outdated Show resolved Hide resolved
@FreddyDevelop FreddyDevelop merged commit c264108 into main Dec 3, 2024
8 checks passed
@FreddyDevelop FreddyDevelop deleted the feat-admin-api-changes branch December 3, 2024 10:22
@FreddyDevelop FreddyDevelop mentioned this pull request Dec 12, 2024
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@lfleischmann lfleischmann lfleischmann approved these changes

@bjoern-m bjoern-m Awaiting requested review from bjoern-m

Assignees
No one assigned
Labels
None yet
Projects
Hanko
Archived in project
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@FreddyDevelop @lfleischmann