haugene / vpn-configs-contrib

A collection of configs for various VPN providers
GNU General Public License v3.0
174 stars 739 forks source link

re-remove tun-ipv6 from mullvad configs #244

Closed rozab closed 8 months ago

rozab commented 9 months ago

Breaking change

I don't think this should be a breaking change because ipv6 didn't seem to be working with mullvad anyway? I am unsure because the author of #200 didn't seem to have problems, perhaps he had it disabled elsewhere.

Proposed change

Re-removed tun-ipv6 from all openvpn configs. This was previously removed in PR #7 but re-added in #200, breaking the configurations for docker containers with ipv6 enabled. Details in issue #1355.

I am experiencing the same errors people had in that issue, and making this change locally has fixed things for me.

Type of change

Additional information

Checklist

If user exposed functionality or configuration variables are added/changed:

pkishino commented 8 months ago

I had a quick look at #200 and yeah, seeing now it was just left in because that was in default.. it would be nice to get more people who actually use mullvad to comment on this before merging it in