diff options
| author | Christian Poessinger <christian@poessinger.com> | 2021-09-02 16:08:57 +0200 |
|---|---|---|
| committer | Christian Poessinger <christian@poessinger.com> | 2021-09-02 16:16:21 +0200 |
| commit | 658de9ea0fbe91e593f9cf0a8c434791282af100 (patch) | |
| tree | b192536d96c45778a2be9261a44f0bc125ac1372 /src/conf_mode | |
| parent | 8032f4bb72db064d2b3f6f0954f6674091822cc4 (diff) | |
| download | vyos-1x-658de9ea0fbe91e593f9cf0a8c434791282af100.tar.gz vyos-1x-658de9ea0fbe91e593f9cf0a8c434791282af100.zip | |
login: T3792: bugfix for usernames containing a hyphen
While migrating to get_config_dict() in commit e8a1c291b1 ("login: radius:
T3192: migrate to get_config_dict()") the user-name was not excluded
from mangling (no_tag_node_value_mangle=True).
This resulted in a username "vyos-user" from CLI to be actually created as
"vyos_user" on the system.
This commit also adds respective Smoketests to prevent this in the future.
Diffstat (limited to 'src/conf_mode')
| -rwxr-xr-x | src/conf_mode/system-login.py | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/src/conf_mode/system-login.py b/src/conf_mode/system-login.py index 93696e653..318ff276d 100755 --- a/src/conf_mode/system-login.py +++ b/src/conf_mode/system-login.py @@ -59,7 +59,7 @@ def get_config(config=None): conf = Config() base = ['system', 'login'] login = conf.get_config_dict(base, key_mangling=('-', '_'), - get_first_key=True) + no_tag_node_value_mangle=True, get_first_key=True) # users no longer existing in the running configuration need to be deleted local_users = get_local_users() |