summaryrefslogtreecommitdiff
path: root/src/migration-scripts
diff options
context:
space:
mode:
authorJohn Estabrook <jestabro@vyos.io>2023-06-05 09:45:04 -0500
committerJohn Estabrook <jestabro@vyos.io>2023-06-05 11:29:00 -0500
commit2ce55816aad83d34a0b9c85db03410e7d0512ed1 (patch)
treeac8f86ddf55eb8ab2c9be1e26d64f467e56f83e5 /src/migration-scripts
parent12b7da289665c75c3839b32c77a491d8909f3e73 (diff)
downloadvyos-1x-2ce55816aad83d34a0b9c85db03410e7d0512ed1.tar.gz
vyos-1x-2ce55816aad83d34a0b9c85db03410e7d0512ed1.zip
openconnect: T5259: fix migration logic in delete_value radius|local
Incorrect logic of node deletion now raises error, after T5251.
Diffstat (limited to 'src/migration-scripts')
-rwxr-xr-xsrc/migration-scripts/openconnect/1-to-214
1 files changed, 7 insertions, 7 deletions
diff --git a/src/migration-scripts/openconnect/1-to-2 b/src/migration-scripts/openconnect/1-to-2
index 7031fb252..51cd6bc37 100755
--- a/src/migration-scripts/openconnect/1-to-2
+++ b/src/migration-scripts/openconnect/1-to-2
@@ -39,13 +39,13 @@ if not config.exists(cfg_base):
else:
if config.exists(cfg_base + ['authentication', 'mode']):
if config.return_value(cfg_base + ['authentication', 'mode']) == 'radius':
- # if "mode value radius", change to "tag node mode + valueless node radius"
- config.delete(cfg_base + ['authentication','mode', 'radius'])
- config.set(cfg_base + ['authentication', 'mode', 'radius'], value=None, replace=True)
- elif not config.exists(cfg_base + ['authentication', 'mode', 'local']):
- # if "mode local", change to "tag node mode + node local value password"
- config.delete(cfg_base + ['authentication', 'mode', 'local'])
- config.set(cfg_base + ['authentication', 'mode', 'local'], value='password', replace=True)
+ # if "mode value radius", change to "mode + valueless node radius"
+ config.delete_value(cfg_base + ['authentication','mode'], 'radius')
+ config.set(cfg_base + ['authentication', 'mode', 'radius'], value=None)
+ elif config.return_value(cfg_base + ['authentication', 'mode']) == 'local':
+ # if "mode local", change to "mode + node local value password"
+ config.delete_value(cfg_base + ['authentication', 'mode'], 'local')
+ config.set(cfg_base + ['authentication', 'mode', 'local'], value='password')
try:
with open(file_name, 'w') as f:
f.write(config.to_string())