Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

FS#4027 - openvpn-openssl does not support multiple remote entries via UCI #9038

Closed
openwrt-bot opened this issue Sep 10, 2021 · 5 comments
Closed
Labels

Comments

@openwrt-bot
Copy link

openwrt-bot commented Sep 10, 2021

eb3f73:

I might be mistaken, but it looks like .ovpn files afford multiple remote entries, but one cannot configure this via UCI. This fails:

config openvpn 'vpnclient' … option list remote '123.123.123.16 443' option list remote '123.123.123.17 443' option list remote '123.123.123.18 443' option list remote '123.123.123.19 443' …
@openwrt-bot
Copy link
Author

openwrt-bot commented Sep 10, 2021

eb3f73:

Should have read:

I might be mistaken, but it looks like .ovpn files [[https://github.com/OpenVPN/openvpn/blob/9c4e4751a1cbd90611e652a228884ef6953844a9/sample/sample-config-files/client.conf#L39-L41|afford multiple remote entries]], but one cannot configure this via UCI. …

@openwrt-bot
Copy link
Author

openwrt-bot commented Sep 19, 2021

eb3f73:

Should this be moved to https://github.com/openwrt/packages/issues ? (Happy to close this and move it there, if that's more appropriate?)

Alternatively, I'm happy to see if I can craft a PR. I'm assuming what I would need to change lives in https://github.com/openwrt/packages/tree/master/net/openvpn ? Is there a good prior example of migration from an option to a list?

@openwrt-bot
Copy link
Author

openwrt-bot commented Sep 19, 2021

jow-:

The syntax is wrong, it should be:

list remote '123.123.123.16 443'
list remote '123.123.123.17 443'
list remote '123.123.123.18 443'
list remote '123.123.123.19 443'

You need to omit the option keyword.

@openwrt-bot
Copy link
Author

openwrt-bot commented Sep 19, 2021

eb3f73:

Wait, so this is already supported! If so, woohoo! (Apologies for missing this.)

@openwrt-bot
Copy link
Author

openwrt-bot commented Sep 19, 2021

eb3f73:

Yes, I can confirm this already works, and is even part of the sample_client configuration in the default /etc/config/openvpn. Apologies again for missing this!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

1 participant