Skip to content

Commit

Permalink
fix(jans-cli-tui): do not hide ldap servers in authn when database is…
Browse files Browse the repository at this point in the history
… not ldap (#9873)

* fix(jans-cli-tui): do not hide ldap servers in authn when database is not ldap

Signed-off-by: Mustafa Baser <[email protected]>

* docs(jans-cli-tui): ldap servers

Signed-off-by: Mustafa Baser <[email protected]>

---------

Signed-off-by: Mustafa Baser <[email protected]>
  • Loading branch information
devrimyatar authored Oct 22, 2024
1 parent 18a68f4 commit c074f85
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 9 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -168,9 +168,6 @@ and navigate to the new default method with **Up** and **Down** keys.
To choose hit **Space** key.
Navigate to **Save** button and hit **Enter** key

!!! Note
If your backend database is not LDAP, you won't see any LDAP servers
related tab/items.

### Basic Authentication Method

Expand All @@ -184,9 +181,8 @@ navigate to **Save** button and hit **Enter** key

### LDAP Servers As Authentication Method

If your backend database is LDAP, you will have this tab where you can add and
modify LDAP Servers are to be used as the default authentication method.
See below image
In this tab you can add and modify LDAP Servers are to be used as the
default authentication method. See below image

![image](../../../assets/tui-authn-ldap-servers.png)

Expand Down
3 changes: 0 additions & 3 deletions jans-cli-tui/cli_tui/plugins/010_auth_server/authn.py
Original file line number Diff line number Diff line change
Expand Up @@ -154,9 +154,6 @@ def on_cli_object_ready(self):

def on_page_enter(self) -> None:

if common_data.server_persistence_type != 'ldap' and 'LDAP Servers' in self.side_nav_bar.navbar_entries:
self.side_nav_bar.navbar_entries.remove('LDAP Servers')

def populate_acr_list():

acr_values = [(BUILTIN_AUTHN, BUILTIN_AUTHN + ' [builtin]')]
Expand Down

0 comments on commit c074f85

Please sign in to comment.