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

Show Modality accordign to requirement #815

Closed
yurem opened this Issue Jan 12, 2018 · 9 comments

Comments

Projects
None yet
3 participants
@yurem
Contributor

yurem commented Jan 12, 2018

We need to find way to get data to display Modality according to this requirement:
#814

@yurem yurem added this to the 3.2.0 milestone Jan 12, 2018

shekhar16 added a commit that referenced this issue May 16, 2018

@shekhar16

This comment has been minimized.

Contributor

shekhar16 commented May 16, 2018

image

@willow9886

This comment has been minimized.

Contributor

willow9886 commented May 16, 2018

yurem added a commit that referenced this issue May 17, 2018

@shekhar16

This comment has been minimized.

Contributor

shekhar16 commented May 18, 2018

@willow9886 the 2 fields which were displayed in above screen shot is for other authentication methods which are stored in oxexternaluid, I think this related to passport #814.
For these method we only had auth_method_id and auth_method_details in format i.e. oxExternalUid: auth_method_id:auth_method_details

@willow9886

This comment has been minimized.

Contributor

willow9886 commented May 21, 2018

@shekhar16 i see, I still think we need human readable descriptions for modality. If it's authentication at an external IDP via Passport, can we list the external provider, e.g. Facebook, Google, etc. ? If not, then we should discuss our options. Please ping me on Rocket ASAP to discuss.

@shekhar16

This comment has been minimized.

Contributor

shekhar16 commented May 28, 2018

image

@shekhar16

This comment has been minimized.

Contributor

shekhar16 commented May 28, 2018

@willow9886 we added all devices there available values.
super gluu ,u2f devices ,otp devices and mobile devices.

@willow9886

This comment has been minimized.

Contributor

willow9886 commented May 28, 2018

@shekhar16 where did the nickname values come from for the two Passport entries?

@shekhar16

This comment has been minimized.

Contributor

shekhar16 commented May 29, 2018

@willow9886 its the initial part of oxexternalUID attribute for example :
for Oxexternaluid = passport-saml:alex.samuseu@yandex.ru, the nickname value is passport-saml

@shekhar16

This comment has been minimized.

Contributor

shekhar16 commented May 29, 2018

image

shekhar16 added a commit that referenced this issue Jun 19, 2018

yurem added a commit that referenced this issue Jun 20, 2018

@shekhar16 shekhar16 closed this Jul 3, 2018

yurem added a commit that referenced this issue Aug 9, 2018

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment