#359 OTP field in the auth form
Closed: Fixed 2 months ago by abompard. Opened a year ago by frostyx.

In Copr, we have at least one person each week who is confused about where to put OTP in the auth form. I think there are two reasons for this (which we found when investigating the issue with the person).

The login form on https://accounts.fedoraproject.org/ has three fields (Username, password, and OTP) while the login form on https://id.fedoraproject.org has only two (Username and password + OTP)
The login form on https://id.fedoraproject.org clearly shows that the field is password + OTP but it is indicated only in the field placeholder and therefore people who auto-fill the form don't see this information.
I don't know what is the preferred solution for this, whether adding the third field for OTP as https://accounts.fedoraproject.org/ does or displaying that the field is password + OTP somewhere (else) outside the field placeholder. Either one is fine with me but the current situation seems to be not sufficient.

Originally reported here
https://github.com/fedora-infra/noggin/issues/715


The split out OTP field was added in #353, but @abompard said it can't work in Fedora right now until python-pam is updated: https://github.com/FirefighterBlu3/python-pam/pull/22

Metadata Update from @ngompa:
- Custom field component adjusted to None
- Custom field type adjusted to None
- Custom field version adjusted to None

a year ago

OK, #353 and #360 contain the necessary changes in Ipsilon and are merged. My patch to python-pam has been merged and we have a RPM in the infra repo that carries the patch.
All we need now is an Ipsilon release so that we can update the infra instances. @ngompa could you cut one when you get the chance please?

Any news here? Perhaps @ngompa could grant @abompard the needed perms to cut a release?

Metadata Update from @abompard:
- Issue close_status updated to: Fixed
- Issue status updated to: Closed (was: Open)

2 months ago

Login to comment on this ticket.

Metadata