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.

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

Allow resetting password for a pg role via mathesar #4119

Open
Tracked by #4092
Anish9901 opened this issue Jan 8, 2025 · 1 comment
Open
Tracked by #4092

Allow resetting password for a pg role via mathesar #4119

Anish9901 opened this issue Jan 8, 2025 · 1 comment
Labels
affects: ux Related to user experience beta: needs discussion Temporary label to mark issues that need discussion needs: ux design restricted: maintainers Only maintainers can resolve this issue work: backend Related to Python, Django, and simple SQL work: frontend Related to frontend code in the mathesar_ui directory

Comments

@Anish9901
Copy link
Member

Anish9901 commented Jan 8, 2025

Description

To create a postgres login role via mathesar or psql, a role name and password are required. In mathesar, users must provide the password when configuring roles to be used for different database servers.

Currently mathesar doesn't validate the password when it is stored, which could result in an incorrect password being saved. Dropping the role entirely and creating a new one is not a viable solution either, as there may be database objects dependent on the role.

Additionally, if a user is connecting their preexisting db to mathesar and has forgotten the password for one of their roles, the only way to reset it is by running a command like ALTER USER user_name WITH PASSWORD 'new_password'; through psql.

Proposed Solution

There should be a way to reset the password for a login role via mathesar.

There are some implications of resetting the password that we'll need to think through:

  • What should happen if the password of a configured role in mathesar is reset? Should the admin need to put in the password for those roles again? Or should we configure them automatically similar to what's proposed in Automatically configure new roles created via mathesar #4118
  • Should we allow mathesar admins to use a different role for resetting the password for a role similar to how we do it for disconnecting db?
@Anish9901 Anish9901 added affects: ux Related to user experience work: backend Related to Python, Django, and simple SQL work: frontend Related to frontend code in the mathesar_ui directory restricted: maintainers Only maintainers can resolve this issue needs: ux design beta: needs discussion Temporary label to mark issues that need discussion labels Jan 8, 2025
@kgodey
Copy link
Contributor

kgodey commented Jan 15, 2025

Can we please add some details to this, @Anish9901

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
affects: ux Related to user experience beta: needs discussion Temporary label to mark issues that need discussion needs: ux design restricted: maintainers Only maintainers can resolve this issue work: backend Related to Python, Django, and simple SQL work: frontend Related to frontend code in the mathesar_ui directory
Projects
None yet
Development

No branches or pull requests

3 participants