"route" has to be the first option in ufw command (#31756)

For supporting deletion of ufw routes, route option has to be placed before delete option
pull/40017/merge
lu1as 7 years ago committed by John R Barker
parent 48e5791860
commit 21004d86f9

@ -314,11 +314,11 @@ def main():
elif command == 'rule': elif command == 'rule':
# Rules are constructed according to the long format # Rules are constructed according to the long format
# #
# ufw [--dry-run] [delete] [insert NUM] [route] allow|deny|reject|limit [in|out on INTERFACE] [log|log-all] \ # ufw [--dry-run] [route] [delete] [insert NUM] allow|deny|reject|limit [in|out on INTERFACE] [log|log-all] \
# [from ADDRESS [port PORT]] [to ADDRESS [port PORT]] \ # [from ADDRESS [port PORT]] [to ADDRESS [port PORT]] \
# [proto protocol] [app application] [comment COMMENT] # [proto protocol] [app application] [comment COMMENT]
cmd.append([module.boolean(params['delete']), 'delete'])
cmd.append([module.boolean(params['route']), 'route']) cmd.append([module.boolean(params['route']), 'route'])
cmd.append([module.boolean(params['delete']), 'delete'])
cmd.append([params['insert'], "insert %s" % params['insert']]) cmd.append([params['insert'], "insert %s" % params['insert']])
cmd.append([value]) cmd.append([value])
cmd.append([params['direction'], "%s" % params['direction']]) cmd.append([params['direction'], "%s" % params['direction']])

Loading…
Cancel
Save