Change #8627
[0001031] : Password Parameters
| Status: | Work Completed-End life cycle | Start date: | June 22, 2018 | |
|---|---|---|---|---|
| Priority: | Normal | Due date: | July 20, 2018 | |
| Assignee: |  Erni Suhaireen Zulkifli | % Done: | 100% | |
| Category: | - | Spent time: | - | |
| Target version: | - |
Description
Module: System Maintenance> Password Parameters
Issue Description
Field:
Password Complexity Pattern
Issue:
1. No checking on Password Complexity Pattern, manage to key in invalid regex.
2. Based on specs, maximum characters for Password Complexity is 256, there is currently no control on this.
Field:
Activation Link Expiry Day
Issue:
To remove word "Day".
Field:
Password Change Interval
Issue:
Add foot note unit for Password Change Interval
Field:
Dormant period
Issue:
1. To spell period with capital P. period change to Period.
2. Add footer note unit for Dormant Period.
Expected Results
i) As described (refer attachment)
ii) There shall be validation against Password Complexity Pattern
3. Risk / Impact
Medium
History
#1
Updated by Kok Ken Wong over 6 years ago
- Status changed from New - Begin Life Cycle to Finished Development
- Assignee changed from Chun Feng Lim to Nurul Akmal
- % Done changed from 0 to 80
Field:
Password Complexity Pattern
1. No checking on Password Complexity Pattern, manage to key in invalid regex. (Dropped as per discussion with Ming Kit)
2. Based on specs, maximum characters for Password Complexity is 256, there is currently no control on this. (Added 256 max length)
Field:
Activation Link Expiry Day
-To remove word "Day". (Done)
Field:
Password Change Interval
-Add foot note unit for Password Change Interval. (Foot note "minute(s)" has been added)
Field:
Dormant period
1. To spell period with capital P. period change to Period. (Corrected with capital P)
2. Add footer note unit for Dormant Period.(Foot note "Day(s)" has been added)
Please test after deployment. Thank you
#2
Updated by Nurul Akmal over 6 years ago
- File Password Parameters after fixed.PNG added
Hi Ken,
Tested. Found below issue:
Field:
Password Complexity Pattern
2. Based on specs, maximum characters for Password Complexity is 256, there is currently no control on this. (Added 256 max length) - Error displayed: "Error while performing your request" after click Confirm if enter "Password Complexity Pattern" up to 256 chars.
Field:
Activation Link Expiry Day
To remove word "Day". (Done) - Word Day is still there. Expected: Field name --> "Activation Link Expiry" instead of "Activation Link Expiry Day"
Field:
Dormant period
1. To spell period with capital P. period change to Period. (Corrected with capital P) - To change both at field name and error message. Currently, capital P at error message only, at field name still small letter.
#3
Updated by Nurul Akmal over 6 years ago
- Status changed from Finished Development to Development / Work In Progress
- Assignee changed from Nurul Akmal to Erni Suhaireen Zulkifli
#4
Updated by Nurul Akmal over 6 years ago
- Assignee changed from Erni Suhaireen Zulkifli to Kok Ken Wong
#5
Updated by Kok Ken Wong over 6 years ago
- Status changed from Development / Work In Progress to Internal Testing
- Assignee changed from Kok Ken Wong to Nurul Akmal
Missed one file during project pushing. Please retest it after another deployment. Thanks
#6
Updated by Nurul Akmal about 6 years ago
- Assignee changed from Nurul Akmal to Erni Suhaireen Zulkifli
Hi Erni,
Kindly assist to retest the issue.
Thank you.
#7
Updated by Erni Suhaireen Zulkifli about 6 years ago
- Status changed from Internal Testing to User Acceptance Test
- % Done changed from 80 to 100
Issue has been tested and successfully passed.
This issue is currently send for retest.
#8
Updated by Erni Suhaireen Zulkifli about 6 years ago
- Status changed from User Acceptance Test to Development / Work In Progress
- % Done changed from 100 to 90
Hi Ken,
Kindly advice on the issue "Password Complexity Pattern"
#9
Updated by Erni Suhaireen Zulkifli about 6 years ago
- Assignee changed from Erni Suhaireen Zulkifli to Kok Ken Wong
#10
Updated by Ngoh Chee Onn about 6 years ago
- Status changed from Development / Work In Progress to Internal Testing
- Assignee changed from Kok Ken Wong to Erni Suhaireen Zulkifli
Hi Erni,
Password complexity validation had been added, please test after deployment tonight.
Thank you.
#11
Updated by Erni Suhaireen Zulkifli about 6 years ago
- Due date set to July 16, 2018
- Status changed from Internal Testing to Pending Customer Feedback
- % Done changed from 90 to 100
Validation check regex for the password complexity pattern had been implement in SIT, pending user to confirm whether suitable for it.
#12
Updated by Erni Suhaireen Zulkifli about 6 years ago
- Due date changed from July 16, 2018 to July 20, 2018
- Status changed from Pending Customer Feedback to Development / Work In Progress
- Assignee changed from Erni Suhaireen Zulkifli to Ngoh Chee Onn
- % Done changed from 100 to 80
Hi Chee Onn,
Refer below for feedback from user:
Tested failed, hit DB error as below.
018/07/16 19:42:15:634 DEBUG enterprise.services.ss221_settings.RPPUpassMaintenanceResultServices: Entering updateUpass
2018/07/16 19:42:15:674 ERROR rpp.common.utils.RPPLoggerBase: Invalid property name 'UPDATED_BY'
java.lang.IllegalArgumentException: Invalid property name 'UPDATED_BY'
at org.apache.struts.action.DynaActionForm.getDynaProperty(DynaActionForm.java:598)
at org.apache.struts.action.DynaActionForm.get(DynaActionForm.java:241)
at com.rpp.struts.utils.FormDynaBean.get(FormDynaBean.java:104)
at com.rpp.enterprise.services.ss221_settings.RPPUpassMaintenanceResultServices.getSettingField(RPPUpassMaintenanceResultServices.java:177)
at com.rpp.enterprise.services.ss221_settings.RPPUpassMaintenanceResultServices.saveUpassConfigurationToDatabase(RPPUpassMaintenanceResultServices.java:74)
#13
Updated by Ngoh Chee Onn about 6 years ago
- Assignee changed from Ngoh Chee Onn to Chun Feng Lim
Hi CF,
Please fixed this issue, since you updated the column for updated by and updated date.
Thank you
#14
Updated by Chun Feng Lim about 6 years ago
- Status changed from Development / Work In Progress to Internal Testing
- Assignee changed from Chun Feng Lim to Erni Suhaireen Zulkifli
- % Done changed from 80 to 90
Updated, please test
#15
Updated by Erni Suhaireen Zulkifli about 6 years ago
- Status changed from Internal Testing to User Acceptance Test
- % Done changed from 90 to 100
Issue has been send for user retest
#16
Updated by Erni Suhaireen Zulkifli about 6 years ago
- Status changed from User Acceptance Test to Work Completed-End life cycle
Issue has been CLOSED by user Amanda at 19-07-2018