? Success
Pull Request for # 5500

User tests: Successful: Unsuccessful:

avatar nikosdion
nikosdion
23 Dec 2014

Refers to issue gh-5500 "Error Message if we try save & close for a User with YubiKey"

Close gh-5500

Executive summary

There's a misleading error message when saving a user account which has two factor authentication using YubiKey already enabled.

Testing instructions

You need three tests before and after applying the patch, in the precise order specified. When I say create a new user account I mean it. Do NOT try to reuse an existing user account. The first two ensure that there are no regressions. The third test confirms that the bug is fixed.

  1. Create a new user account. Go to the Two Factor Authentication tab. Select the YubiKey authentication method. Leave the secret code blank and save. You get an error. This must be reproduced before AND after the patch.

  2. Create a new user account. Go to the Two Factor Authentication tab. Select the YubiKey authentication method. Click in the Secret Code field and press the YubiKey button. Save. This first save works fine. This must be reproduced before AND after the patch.

  3. Edit the user account from step 2. Do NOT change anything. Click on Save or Save & Close. Before the patch: you get an error about the code being invalid or YubiCloud servers being unreachable but the user saves. After the patch: the user saves without an error.

Backwards compatibility

No impact

Translation impact

No impact

avatar nikosdion nikosdion - open - 23 Dec 2014
avatar jissues-bot jissues-bot - change - 23 Dec 2014
Labels Added: ?
avatar brianteeman
brianteeman - comment - 23 Dec 2014

@test completed all three tests before and after the patch - issue confirmed and patch works


This comment was created with the J!Tracker Application at issues.joomla.org/joomla-cms/5505.
avatar brianteeman brianteeman - test_item - 23 Dec 2014 - Tested successfully
avatar brianteeman brianteeman - change - 23 Dec 2014
Category Authentication Plugins
avatar brianteeman brianteeman - change - 23 Dec 2014
Rel_Number 5500
Relation Type Pull Request for
avatar zero-24
zero-24 - comment - 23 Dec 2014

@test @nikosdion patch works here. Thanks for the quick fix.

Travis fails unrelated: https://travis-ci.org/joomla/joomla-cms/jobs/44981224


This comment was created with the J!Tracker Application at issues.joomla.org/joomla-cms/5505.
avatar zero-24 zero-24 - test_item - 23 Dec 2014 - Tested successfully
avatar zero-24 zero-24 - change - 23 Dec 2014
Status Pending Ready to Commit
avatar zero-24
zero-24 - comment - 23 Dec 2014

moving to RTC


This comment was created with the J!Tracker Application at issues.joomla.org/joomla-cms/5505.
avatar wilsonge wilsonge - close - 24 Dec 2014
avatar wilsonge wilsonge - change - 24 Dec 2014
Status Ready to Commit Closed
Closed_Date 0000-00-00 00:00:00 2014-12-24 01:20:12
avatar wilsonge
wilsonge - comment - 24 Dec 2014

Merged. Thanks guys

Add a Comment

Login with GitHub to post a comment