search

MarkusMcNugen / docker-qBittorrentvpn

Docker container which runs a headless qBittorrent client with WebUI and optional OpenVPN
https://hub.docker.com/r/markusmcnugen/qbittorrentvpn/
GNU General Public License v3.0
174 stars 93 forks source link

VPN_ENABLED requires correct capitalization (small letters only) #138

Open schnillerman opened 2 years ago

schnillerman commented 2 years ago

It took me a while analyzing error messages until I found out that the variables (VPN_ENABLED, at least) require small letters for values.

With VPN_ENABLED=YES, the VPN wouldn't even start up, qBittorrent would start up and be visible with my own IP (tested via MagnetIP).

In my eyes, it should be standard that capitalization of values for variables doesn't matter and is e.g. always converted to lowercase (e.g. lower([string])).