Skip to content

Commit

Permalink
[fix] Use non-mutable default arguments
Browse files Browse the repository at this point in the history
Fixed signature of netjsonconfig.backends.openvpn.converters.OpenVpn.__intermediate_vpn
to use non-mutable default argument.

Co-authored-by: Aryaman <[email protected]>
  • Loading branch information
pandafy and Aryamanz29 authored Jun 8, 2023
1 parent 2e7d548 commit 96d811f
Showing 1 changed file with 2 additions and 1 deletion.
3 changes: 2 additions & 1 deletion netjsonconfig/backends/openvpn/converters.py
Original file line number Diff line number Diff line change
Expand Up @@ -24,7 +24,8 @@ def to_intermediate_loop(self, block, result, index=None):
result['openvpn'].append(vpn)
return result

def __intermediate_vpn(self, config, remove=[False, 0, '']):
def __intermediate_vpn(self, config, remove=None):
remove = remove or [False, 0, '']
skip_keys = ['script_security', 'remote']
delete_keys = []
# allow server_bridge to be empty and still rendered
Expand Down

0 comments on commit 96d811f

Please sign in to comment.