Project

General

Profile

Actions

Feature #1384

closed

Support "Password never expires"

Added by Alena Peterová about 6 years ago. Updated over 5 years ago.

Status:
Closed
Priority:
High
Assignee:
Ondřej Kopr
Category:
Password
Target version:
Start date:
11/19/2018
Due date:
% Done:

100%

Estimated time:
16.00 h
Owner:

Description

Please add a new checkbox "Password never expires" to the user detail. If this checkbox is checked, then the password of that user doesn't expire, even if password policy specifies maximum password age.

If the checkbox is unchecked, then maximum password age applies for the user.

This checkbox can be changed only by administrators with special authorization, e.g. APP_ADMIN. Changes of this property are audited in the audit log for identities.

When "admin" identity is created during the first initialization of CzechIdM, the value of "Password never expires" for this identity will be "true".
All other identities have it by default "false".

Use case:
We have "Maximum password age = 90". After 90 days, the password for "admin" expired, so we can't login to IdM anymore.
We want to set "Password never expires" for admin and other super users, so we don't lose access to IdM anymore.


Files

PasswordNeverExpires.png (49.1 KB) PasswordNeverExpires.png Alena Peterová, 11/20/2018 08:11 AM

Related issues

Related to IdStory Identity Manager - Task #1185: Create endpoint to read metadata about identity passwordClosedOndřej Kopr07/19/2018

Actions
Actions #1

Updated by Ondřej Kopr about 6 years ago

It's security lacks :( are you sure that you wanted this? If I will implemented this you want never expiring password for all users that has permission APP_ADMIN (super admin). Because check user by username is very dangerous (rename or etc). On some project we just set null to expiring column in database and that is all, it is not good way, but it is more security friendly than checkbox "never expire password for all APP_ADMIN".

Actions #2

Updated by Peter Štrunc about 6 years ago

I dont think that it was meant for all admin users, but just for one user (or small subset) which will be used as service access in case all users lose their access to IdM.

Actions #3

Updated by Alena Peterová about 6 years ago

I changed the description to be more specific. I definitely don't want to use this widely on all super admins and also not for specific username.
A similar feature is e.g. in Active Directory, where it's standardly used for service accounts, see screenshot.
Changing anything in database is not desired.

Actions #4

Updated by Vladimír Kotýnek about 6 years ago

I guess this should be some kind of privilege assigned to a specific role so e.g. Security Manager can approve it. We definitely don't want all superAdmins without password expiration. Also, not all users we want to exclude from password expiration are superAdmins.

Actions #5

Updated by Ondřej Kopr about 6 years ago

  • Related to Task #1185: Create endpoint to read metadata about identity password added
Actions #6

Updated by Vít Švanda over 5 years ago

  • Target version set to Quartz (9.6.0)
Actions #7

Updated by Vít Švanda over 5 years ago

  • Estimated time set to 16.00 h
Actions #8

Updated by Ondřej Kopr over 5 years ago

Updated request:

  • new agenda/detail on identity detail (information about password),
  • new rest enpoint,
  • new permissions for the password metadata,
  • new attribute passwordNeverExpires,
  • new behavior with password never expires.
Actions #9

Updated by Ondřej Kopr over 5 years ago

  • Status changed from New to In Progress
Actions #10

Updated by Ondřej Kopr over 5 years ago

  • % Done changed from 0 to 90

Feature, documentation and tests was done in ticket #1185.

Actions #11

Updated by Ondřej Kopr over 5 years ago

  • Status changed from In Progress to Needs feedback
  • Assignee changed from Ondřej Kopr to Radek Tomiška

Please Radek could you made a feedback? Commit and documentaion can be found in ticket #1185. Thank you

Actions #12

Updated by Radek Tomiška over 5 years ago

  • Status changed from Needs feedback to Resolved
  • Assignee changed from Radek Tomiška to Ondřej Kopr
  • % Done changed from 90 to 100

I did test and review, it works, all password features are available on one place, and code looks nice, thx!

Note: I did only minor changes about rest, show loading, locales, init admin identity with password never expires, added author, since ... :):
https://github.com/bcvsolutions/CzechIdMng/commit/34ccb8c1473db4367ef4c85c64d3a23db76fedde

Actions #13

Updated by Ondřej Kopr over 5 years ago

  • Status changed from Resolved to Closed
Actions

Also available in: Atom PDF