haugene / vpn-configs-contrib

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

Port-forwarding for ProtonVPN #254

Closed muggajoe closed 7 months ago

muggajoe commented 7 months ago

Type of change

Additional information

This change depends on natapmpc being installed in the transmission-openvpn container. natpmpc installation has been merged into the dev branch for haugene/docker-transmission-openvpn on 2023-11-14 - https://github.com/haugene/docker-transmission-openvpn/pull/2746

The script will fail gracefully if used with a build of haugene/docker-transmission-openvpn that does not have natpmpc installed.

Confirmed to work with the following ProtonVPN configs:

ch.protonvpn.net.udp es.protonvpn.net.tcp es.protonvpn.net.udp is.protonvpn.net.tcp is.protonvpn.net.udp ng.protonvpn.net.tcp ng.protonvpn.net.udp nl.protonvpn.net.tcp nl.protonvpn.net.udp ro.protonvpn.net.udp ro.protonvpn.net.tcp se.protonvpn.net.tcp se.protonvpn.net.udp uk.protonvpn.net.udp us.protonvpn.net.udp

Checklist

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

pkishino commented 7 months ago

Nice, thanks

ahti commented 7 months ago

Fyi to make this work when using TRANSMISSION_RPC_AUTHENTICATION_REQUIRED=true, I had to modify update-port.sh by inserting the following block after the line with TRANSMISSION_PASSWD_FILE=... (found here, except I didn't need to hardcode my unhashed pw, as the one in transmission-credentials.txt wasn't hashed):

transmission_username=$(head -1 ${TRANSMISSION_PASSWD_FILE})
transmission_passwd=$(tail -1 ${TRANSMISSION_PASSWD_FILE})
transmission_settings_file=${TRANSMISSION_HOME}/settings.json

Without those, $transmission_settings_file wasn't set, so the script would always assume auth wasn't required.

muggajoe commented 7 months ago

The ProtonVPN update-port.sh has been updated with that change. Thanks!