Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Ability to pass through arguments on UDP and TCP ports. #41

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

omerinero
Copy link

Since I need to run aMule on random ports generated by my VPN I had to provide those ports in the aMule preferences before aMule started. Maybe this feature will be useful to someone else.
Sometimes I need to do manual configuration, like when my VPN container restarts and the random port is re-generated, I'm thinking about integrating VPN and aMule all in one container ( something like this https://github.com/haugene/docker-transmission-openvpn )

@Marcosaurios
Copy link

Hey, maybe this is not the best place to ask but I haven't found so many peeps using vpn on aMule. May I ask which vpn provider are you using which gets high id when connecting to a server? Tried with PrivateInternetAccess vpn but no luck, and I was considering AirVPN. Thanks!

@omerinero
Copy link
Author

I utilize ProtonVPN Plus, consistently choosing servers that support P2P. To maintain a secure and anonymous connection, I launch an additional container (Gluetun) that establishes the VPN connection, enabling aMule to utilize this network configuration.

@ngosang
Copy link
Owner

ngosang commented Dec 14, 2023

I do not oppose but I don't think this will be useful for most users. If I understand you well:

  1. Your VPN changes the open ports frequently (this could be good for security but bad for aMule network since most peers in DHT will have to discover you again and so on...)
  2. When that happens you edit the docker-compose.yml to change the environment variables you included in this PR and the "ports" section.

aMule has 3 ports than must be open to the Internet, not 2. The second is related to the first and it's not configured in aMule.conf. You have to change that port in the "ports" section too and that is hard to explain if other users mess with the ports...

| `-p 4662`               | ED2K TCP port (must be open to Internet). 
| `-p 4665/udp`           | ED2K global search UDP port (tcp port +3) (must be open to Internet).  
| `-p 4672/udp`           | ED2K UDP port (must be open to Internet).       

Since you are already updating the docker-compose.yml and you have the aMule.conf file mounted in the container (you can access to it from the host). Why don't you create a personal script to replace the docker-compose.yml, the aMule.conf and restart the docker-compose? You can also extend this Docker image to add VPN support.
I prefer to keep things simple, I don't have a lot of time to answer configuration questions.

@ngosang ngosang added the wontfix This will not be worked on label Jan 20, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
wontfix This will not be worked on
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants