Skip to content

Add self-service non-admin user RBAC#7593

Merged
gerrod3 merged 1 commit intopulp:mainfrom
gerrod3:user-self-service
Apr 16, 2026
Merged

Add self-service non-admin user RBAC#7593
gerrod3 merged 1 commit intopulp:mainfrom
gerrod3:user-self-service

Conversation

@gerrod3
Copy link
Copy Markdown
Contributor

@gerrod3 gerrod3 commented Apr 14, 2026

This should allow users to update their password without needing admin permission.

📜 Checklist

  • Commits are cleanly separated with meaningful messages (simple features and bug fixes should be squashed to one commit)
  • A changelog entry or entries has been added for any significant changes
  • Follows the Pulp policy on AI Usage
  • (For new features) - User documentation and test coverage has been added

See: Pull Request Walkthrough

@mdellweg
Copy link
Copy Markdown
Member

mdellweg commented Apr 15, 2026

I wonder if we could add this to the login viewset in a useful way.

That would mean the users viewset keeps being admins only, and users only ever interact with the login. We could even use a serializer that asks for the old, the new and a confirmation of the new password (when thinking about ui...), still allowing an admin to just (re-)set a users password.

Copy link
Copy Markdown

@josephtate josephtate left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yes, this looks like "just enough"

@gerrod3
Copy link
Copy Markdown
Contributor Author

gerrod3 commented Apr 16, 2026

@mdellweg I did your suggestion, put it as a patch call to /login. Let me know if you have any concerns

Copy link
Copy Markdown
Member

@mdellweg mdellweg left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yes, I like that much more. Thank you!

@gerrod3 gerrod3 force-pushed the user-self-service branch from aec28ec to b6fd2c8 Compare April 16, 2026 13:43
@gerrod3 gerrod3 merged commit 537e653 into pulp:main Apr 16, 2026
13 of 14 checks passed
@gerrod3 gerrod3 deleted the user-self-service branch April 16, 2026 14:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants