JanssenProject / jans

An open source enterprise digital identity platform for CIAM or workforce... Janssen is a distribution of standards-based, developer friendly, components that are engineered to work together in any cloud. #OAuth #OpenID #FIDO
https://docs.jans.io
Apache License 2.0
457 stars 74 forks source link

feat(jans-cli-tui): create ACR screen on TUI #8217

Closed yuriyz closed 1 month ago

yuriyz commented 6 months ago

Description

TUI "ACR" screen should have clear sections: 1) Built-in For now it's just: simple_password_auth 2) Script acrs all script names listed here 3) Aliases Mappings between alias and technical name acr_values=myOTP -> acr_values=org.gluu.agama.otp.main 4) Agama flows acr_values=agama_org.gluu.agama.otp.main

https://chat.gluu.org/group/service_auth_server?msg=Dca2335PvjKNsFN8b

Relevant documents that might need updates

Note: The developer needs to add to this list based on the complete understanding of the feature and its implementation

Test cases and code coverage


Document the changes

yuriyz commented 6 months ago

Wait until this ticket is solved https://github.com/JanssenProject/jans/issues/8216

ossdhaval commented 6 months ago

updated the description for documentation tasks

devrimyatar commented 1 month ago

@yuriyz Should I extend the following screen for Aliases and Agama Flow

authn

nynymike commented 1 month ago

It would be nice if simple_password_auth and auth_ldap_server were configurable by the admin.

devrimyatar commented 1 month ago

It would be nice if simple_password_auth and auth_ldap_server were configurable by the admin.

Yes they are

image

image

yuriyz commented 1 month ago

@yuriyz Should I extend the following screen for Aliases and Agama Flow

@devrimyatar yes

duttarnab commented 1 month ago

Hi @yuriyz @devrimyatar @pujavs

yuriyz commented 1 month ago

Aliases are part of AS config. So this should already work.

{
  ...
  "acrMappings": {
    "acr_alias1": "basic",
    "acr_alias2": "otp" 
  }
  ...
}

@duttarnab thanks for sharing info for Agama flows. I guess it was missing piece for @devrimyatar

devrimyatar commented 1 month ago

Aliases are part of AS config. So this should already work.

{
  ...
  "acrMappings": {
    "acr_alias1": "basic",
    "acr_alias2": "otp" 
  }
  ...
}

@duttarnab thanks for sharing info for Agama flows. I guess it was missing piece for @devrimyatar

@yuriyz I imeplemented Aliases section

image

image

duttarnab commented 1 month ago

Hi @yuriyz my understanding is, for Agama Flows we need to show all the added Agama projects on the auth server. We already have a screen for this, we just need to shift this screen under a separate tab under Authn menu. Kindly let us know if this understanding is incorrect.

image

yuriyz commented 1 month ago

Right, idea is to show it on TUI as well.