YunoHost-Apps / vpnclient_ynh

VPN Client app for YunoHost
GNU Affero General Public License v3.0
41 stars 25 forks source link

Use default DNS resolvers from Yunohost when DNS settings are empty #105

Closed hidrarga closed 1 year ago

hidrarga commented 1 year ago

Problem

The DNS method was hardcoded as "custom" in the cube file importer. Therefore, when we import a cube file with empty DNS resolvers, it produces an error instead of using the default DNS resolvers from Yunohost.

Solution

I'm checking if the DNS resolvers from the cube file are empty, then I'm using the default DNS resolvers from Yunohost (dns_method = yunohost).

PR Status

Automatic tests

Automatic tests can be triggered on https://ci-apps-dev.yunohost.org/ after creating the PR, by commenting "!testme", "!gogogadgetoci" or "By the power of systemd, I invoke The Great App CI to test this Pull Request!". (N.B. : for this to work you need to be a member of the Yunohost-Apps organization)