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

Short-term admin accounts #22833 #22837

Merged
merged 98 commits into from
Mar 28, 2020

Conversation

lfolco
Copy link
Contributor

@lfolco lfolco commented May 11, 2019

Description (*)

Provides the ability to set an expiration date on admin user accounts, after which the account will be disabled.

Fixed Issues (if relevant)

  1. Short-term admin accounts #22833: Short-term admin accounts

Manual testing scenarios (*)

Manual testing scenario 1

  1. Create an admin user
  2. Set their expiration date to 5 minutes from now
  3. Login as that user
  4. Wait 5 minutes
  5. Navigate to another page
  6. User should be logged out

Manual testing scenario 2

  1. Create an admin user
  2. Set their expiration date to 5 minutes from now
  3. In 5 minutes, try to login as that user
  4. Login should fail with message: "The account sign-in was incorrect or your account is disabled temporarily. Please wait and try again later"

Manual testing scenario 3

  1. Create an admin user
  2. Set their expiration date to 5 minutes from now
  3. In 5 minutes, run the Magento cron manually
  4. Check the admin user: the account should be disabled and the expiration date should now be empty

Contribution checklist (*)

  • Pull request has a meaningful description of its purpose
  • All commits are accompanied by meaningful commit messages
  • All new or changed code is covered with unit/integration tests (if applicable)
  • All automated tests passed successfully (all builds are green)

@m2-assistant
Copy link

m2-assistant bot commented May 11, 2019

Hi @lfolco. Thank you for your contribution
Here is some useful tips how you can test your changes using Magento test environment.
Add the comment under your pull request to deploy test or vanilla Magento instance:

  • @magento-engcom-team give me test instance - deploy test instance based on PR changes
  • @magento-engcom-team give me 2.3-develop instance - deploy vanilla Magento instance

For more details, please, review the Magento Contributor Assistant documentation

@okorshenko okorshenko changed the title Project pepe [WIP] Project pepe May 11, 2019
@okorshenko okorshenko changed the title [WIP] Project pepe [WIP] Short-term admin accounts #22833 May 11, 2019
@sidolov
Copy link
Contributor

sidolov commented May 11, 2019

@magento-engcom-team give me test instance

@magento-engcom-team
Copy link
Contributor

Hi @sidolov. Thank you for your request. I'm working on Magento instance for you

@magento-engcom-team
Copy link
Contributor

Hi @sidolov, here is your new Magento instance.
Admin access: https://pr-22837.instances.magento-community.engineering/admin
Login: admin Password: 123123q

@phoenix128 phoenix128 self-assigned this Jun 10, 2019
@phoenix128
Copy link
Contributor

Hello @lfolco ,
are you still on this PR?

@lfolco
Copy link
Contributor Author

lfolco commented Jun 10, 2019

Yes, I'm running into issues with my tests failing here but not locally. Could use some help!

@lfolco
Copy link
Contributor Author

lfolco commented Jun 10, 2019

I know that this PR has issues with the semantic version checker build, but it's also failing on the static tests build and the functional tests build for stuff that I haven't changed.

@VladimirZaets
Copy link
Contributor

@magento run all tests

1 similar comment
@VladimirZaets
Copy link
Contributor

@magento run all tests

@VladimirZaets
Copy link
Contributor

@magento run all tests

@magento-engcom-team
Copy link
Contributor

Hi @VladimirZaets, thank you for the review.
ENGCOM-6014 has been created to process this Pull Request

@m2-assistant
Copy link

m2-assistant bot commented Mar 28, 2020

Hi @lfolco, thank you for your contribution!
Please, complete Contribution Survey, it will take less than a minute.
Your feedback will help us to improve contribution process.

@lfolco lfolco deleted the project_pepe branch March 14, 2022 21:12
@sdzhepa sdzhepa mentioned this pull request May 9, 2022
This pull request was closed.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.