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

Kapua-sys Account Can Have Expiration Date #2149

Closed
LeoNerdoG opened this issue Oct 26, 2018 · 2 comments · Fixed by #2278
Closed

Kapua-sys Account Can Have Expiration Date #2149

LeoNerdoG opened this issue Oct 26, 2018 · 2 comments · Fixed by #2278
Assignees
Labels
Bug This is a bug or an unexpected behaviour. Fix it!

Comments

@LeoNerdoG
Copy link
Contributor

For now kapua-sys user cannot have an expiration Date as this is root user. On the other hand, kapua-sys account can have an Expiration Date, which is (in my opinion) dangerous as user can accidentally set Expiration date in the past and then everything gets locked.
I suggest we disabled the Expiration Date field for kapua-sys user or at least show error if anything is entered in the field.

Testflow:

  1. Login as kapua-sys
  2. Go to Settings view and try to set an Expiration date
    kapua does not return any kind of error.

Expected behavior
I suggest we disabled the Expiration Date field for kapua-sys user or at least show error if anything is entered in the field.

Screenshot:
screenshot 2018-10-26 at 11 56 44

Version of Kapua
1.1.0-SNAPSHOT

Type of deployment
[ ] Local Vagrant deployment
[x] Docker
[ ] Openshift (in its variants)
[ ] Others

Main component affected
[x] Console (in case of console please report info on which browser you encountered the problem)
[ ] REST API
[ ] Message Broker
[ ] - Others

Browsers used:

  • Google Chrome (Version 69.0.3497.100 (Official Build) (64-bit))
  • Safari (Version 12.0 (13604.3.5))
  • Mozilla Firefox (63.0 (64-bit))
  • Opera (Version 55.0.2994.44)
@muros-ct muros-ct self-assigned this Oct 26, 2018
@Coduz Coduz added the Bug This is a bug or an unexpected behaviour. Fix it! label Oct 29, 2018
@lorthirk
Copy link

lorthirk commented Oct 29, 2018

Does this happen in REST API as well?

EDIT: I checked in the code. There's nothing in the backend service that prevents the expiration date to be set for the administration account. Should we decide to proceed, this has to be implemented in the backend service and not just in the console.

@LeoNerdoG
Copy link
Contributor Author

I have verified that this issue has been resolved.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Bug This is a bug or an unexpected behaviour. Fix it!
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants