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

Use of is_password_changed #3278

Closed
svaabs opened this issue Dec 15, 2017 · 6 comments
Closed

Use of is_password_changed #3278

svaabs opened this issue Dec 15, 2017 · 6 comments
Assignees
Milestone

Comments

@svaabs
Copy link
Contributor

svaabs commented Dec 15, 2017

In cgxp the purpose of the attribute is_password_changed was to show a message and force the user to change his password if the value was set to false. How is this done in ngeo or is there no more use of this feature?

@sbrunner sbrunner added this to the 2.2 milestone Dec 15, 2017
@sbrunner sbrunner added the Ready label Dec 15, 2017
@ger-benjamin
Copy link
Member

It seems that this value exist in ngeo but is not used. So it looks like there is no implemented mechanism to force the user to change his password for now.

@ybolognini
Copy link
Member

To do

@ger-benjamin ger-benjamin added Ongoing and removed Ready labels Jan 8, 2018
ger-benjamin added a commit that referenced this issue Jan 8, 2018
Fix: #3278 - Use of is_password_changed
ger-benjamin added a commit that referenced this issue Jan 8, 2018
Fix: #3278 - Use of is_password_changed
ger-benjamin added a commit that referenced this issue Jan 9, 2018
Fix: #3278 - Use of is_password_changed
@ger-benjamin
Copy link
Member

I've added a gmf-authentication-force-password-change attribute. See #3346

@svaabs
Copy link
Contributor Author

svaabs commented Feb 2, 2018

Thanks for working on this. I had the chance to implement it on our gmf222-branch and unhappily it does not work as expected.
When I login with a user that has is_password_changed set to false, I don't get any message (but I do expect one). As soon as the page is fully refreshed, the message appears.
Can you reproduce this on demo?

@ger-benjamin ger-benjamin reopened this Feb 5, 2018
@ger-benjamin
Copy link
Member

:-/ Sadly I can reproduce the issue, I'll fix it soon

@ger-benjamin
Copy link
Member

Fixed with #3484

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

No branches or pull requests

4 participants