From 48dec7ec766d9996c45e56815a54b8c0149abc28 Mon Sep 17 00:00:00 2001
From: rasmus-kirk <mail@rasmuskirk.com>
Date: Mon, 4 Mar 2024 18:53:07 +0100
Subject: [PATCH] updated readme

---
 README.md                       | 19 ++++++++++++++-----
 nixarr/transmission/default.nix |  4 ++--
 2 files changed, 16 insertions(+), 7 deletions(-)

diff --git a/README.md b/README.md
index 0a55bb4..09bd704 100644
--- a/README.md
+++ b/README.md
@@ -57,11 +57,13 @@ The documentation for the options can be found
 
 ## The Wiki
 
-If you want to know how to setup DDNS with Njalla, or how to manage secrets in nix, check out the [wiki](https://nixarr.rasmuskirk.com/wiki/)
+If you want to know how to setup DDNS with Njalla, or how to manage secrets
+in nix, check out the [wiki](https://nixarr.rasmuskirk.com/wiki/)
 
 ## Importing this module
 
-To use this module, add it to your flake inputs in your nix flake file, like shown in this example flake:
+To use this module, add it to your flake inputs in your nix flake file,
+like shown in this example flake:
 
 ```nix {.numberLines}
   {
@@ -228,11 +230,18 @@ through `localhost:6006`.
 Another solution is to use [tailscale](https://tailscale.com/) or to setup
 your own VPN [manually with wireguard](https://nixos.wiki/wiki/WireGuard).
 
+
 ## VPN Providers
 
-Your VPN-provider must support wg-quick configuration files, this should be most of them. I know that Mullvad and AirVPN has support. It's also _recommended_ that the VPN you're using has support for port forwarding. I
-suggest [AirVpn](https://airvpn.org/), since they accept Monero, but you can
-use whatever you want.
+Your VPN-provider should at the very least support wg-quick configurations,
+this module does not, and will not, support any other setup. Most VPN-providers
+should support this ATM.
+
+Secondly, it's recommended that the VPN you're using has support for _static_
+port forwarding as this module has no builtin support for dynamic port
+forwarding. I suggest [AirVpn](https://airvpn.org/), since they support
+static port forwarding, support wg-quick configurations and accept Monero,
+but you can use whatever you want.
 
 ## Domain Registrars
 
diff --git a/nixarr/transmission/default.nix b/nixarr/transmission/default.nix
index d41b39d..de50af4 100644
--- a/nixarr/transmission/default.nix
+++ b/nixarr/transmission/default.nix
@@ -273,8 +273,8 @@ in {
         then pkgs.flood-for-transmission
         else null;
       package = pkgs.transmission_4;
-      openRPCPort = true;
-      openPeerPorts = true;
+      openRPCPort = false;
+      openPeerPorts = if cfg.vpn.enable then false else true;
       settings =
         {
           download-dir = downloadDir;