Skip to content

Add self-service non-admin user RBAC#7593

Open
gerrod3 wants to merge 2 commits intopulp:mainfrom
gerrod3:user-self-service
Open

Add self-service non-admin user RBAC#7593
gerrod3 wants to merge 2 commits 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

@gerrod3 gerrod3 force-pushed the user-self-service branch from 8a31cf1 to 1f7d14e Compare April 14, 2026 20:42
@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

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants