Skip to content

Add self-service password change option. - #3340

Open
meisterT wants to merge 1 commit into
DOMjudge:mainfrom
meisterT:pw_change
Open

Add self-service password change option.#3340
meisterT wants to merge 1 commit into
DOMjudge:mainfrom
meisterT:pw_change

Conversation

@meisterT

Copy link
Copy Markdown
Member

This can be enabled in two ways:

  • by adding roles to the list of roles that are allowed to change their password (e.g. jury)
  • by enabling password change for specific team categories

Fixes #2729

@vmcj

vmcj commented Jan 18, 2026

Copy link
Copy Markdown
Member

I rather not have this implemented but use the REMOTE_USER for this.

@meisterT

Copy link
Copy Markdown
Member Author

REMOTE_USER seems much harder to set up, and the change is relatively self-contained. but if you feel strongly, feel free to close this.

@nickygerritsen

Copy link
Copy Markdown
Member

REMOTE_USER seems much harder to set up, and the change is relatively self-contained. but if you feel strongly, feel free to close this.

Personally i agree with Tobi on all three his remarks

@meisterT

meisterT commented Jul 9, 2026

Copy link
Copy Markdown
Member Author

@vmcj let's make a decision on this - any input from my side required?

@vmcj

vmcj commented Jul 9, 2026

Copy link
Copy Markdown
Member

I'm delaying this one until we have an inperson discussion.

}

#[Route(path: '', name: 'profile_index')]
public function changePasswordAction(Request $request, UserPasswordHasherInterface $passwordHasher): Response

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

If we allow these changes, they might deserve an audit log message.

This can be enabled in two ways:
- by adding roles to the list of roles that are allowed to change their
  password (e.g. `jury`)
- by enabling password change for specific team categories
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Allow user change password

4 participants