Page MenuHomeVyOS Platform
Create Task
Maniphest T2293

OpenVPN: UnboundLocalError after merging server_network PullRequest
Closed, ResolvedPublicBUG
Actions

Description

With the following test configuration the error can be reproduced:

set interfaces openvpn vtun1 description 'fooo bar'
set interfaces openvpn vtun1 local-address 10.255.1.2
set interfaces openvpn vtun1 local-host '172.18.201.10'
set interfaces openvpn vtun1 local-port '1195'
set interfaces openvpn vtun1 mode 'site-to-site'
set interfaces openvpn vtun1 persistent-tunnel
set interfaces openvpn vtun1 protocol 'udp'
set interfaces openvpn vtun1 remote-address '10.255.1.1'
set interfaces openvpn vtun1 remote-host '172.18.202.10'
set interfaces openvpn vtun1 remote-port '1195'
set interfaces openvpn vtun1 shared-secret-key-file '/config/auth/openvpn-1.key'
Traceback (most recent call last):
  File "/usr/libexec/vyos/conf_mode/interfaces-openvpn.py", line 886, in <module>
    c = get_config()
  File "/usr/libexec/vyos/conf_mode/interfaces-openvpn.py", line 482, in get_config
    default_server = getDefaultServer(server_network, openvpn['server_topology'], openvpn['type'])
UnboundLocalError: local variable 'server_network' referenced before assignment

Details

Version
1.3-rolling-202004140117
Is it a breaking change?
Perfectly compatible
Issue type
Bug (incorrect behavior)

Related Objects

Event Timeline

c-po created this task.Apr 14 2020, 6:29 PM
c-po assigned this task to jjakob.
c-po triaged this task as Urgent! priority.
c-po edited a custom field.
c-po edited a custom field.
c-po changed Is it a breaking change? from Unspecified (possibly destroys the router) to Perfectly compatible.
pasik subscribed.Apr 14 2020, 7:53 PM
jjakob added a comment.Apr 14 2020, 8:08 PM

https://github.com/vyos/vyos-1x/pull/345

Restricted Repository Identity mentioned this in rVYOSONEX079ffb7a850b: Merge pull request #345 from jjakob/openvpn-pool.Apr 15 2020, 5:56 AM
jjakob added a comment.Apr 15 2020, 3:30 PM

The previous commit didn't fix it, this one does: https://github.com/vyos/vyos-1x/pull/348

I verified it with the supplied config.

Restricted Repository Identity mentioned this in rVYOSONEX011bb3423567: Merge pull request #348 from jjakob/openvpn-exception-fix.Apr 15 2020, 4:33 PM
jjakob closed this task as Resolved.Apr 15 2020, 5:17 PM
jjakob moved this task from Need Triage to Finished on the VyOS 1.3 Equuleus board.
erkin set Issue type to Bug (incorrect behavior).Aug 30 2021, 7:04 AM
erkin removed a subscriber: Global Notifications.
syncer edited projects, added VyOS 1.3 Equuleus (1.3.0); removed VyOS 1.3 Equuleus.Aug 30 2022, 2:02 PM
syncer moved this task from Need Triage to Finished on the VyOS 1.3 Equuleus (1.3.0) board.