crazy-max / docker-rtorrent-rutorrent

rTorrent and ruTorrent Docker image
MIT License
460 stars 103 forks source link

Added a couple of options + gave more time to rtorrent before SIGKILL #170

Closed Cryptik-Rick closed 2 years ago

Cryptik-Rick commented 2 years ago

The options added are as follows:

crazy-max commented 2 years ago

As a side note:

Not needed you can already set smth like 45000-50000

Not needed too. See https://github.com/crazy-max/docker-rtorrent-rutorrent#boostrap-config-rtlocalrc

Cryptik-Rick commented 2 years ago

Please create a PR per feature/bug. Also create proper commit messages that reflects your changes and squash when possible.

Sorry I thought you could squash while merging. First commit on Github

Added possibility to set a range of port as opposed to a single one for incoming connections; Not needed you can already set smth like 45000-50000

True.

Added optional configuration file that will be appended to rtorrent.rc if defined Not needed too. See https://github.com/crazy-max/docker-rtorrent-rutorrent#boostrap-config-rtlocalrc

It's arguably way easier to change an outside file than to change your docker image / rebuild. Just thought it'd be a nice feature.