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

Bind transmission to tun0 #16

Closed
wants to merge 2 commits into from

Conversation

ianneub
Copy link
Contributor

@ianneub ianneub commented May 23, 2015

I believe this will solve #11

It forces openvpn to use tun0. Then determines the IP of said interface and updates the settings.json config before starting transmission.

This will not work if the user has created a /data/transmission-data folder and is using that.

Maybe we should get rid of that? The user could simply replace the /etc/transmission-daemon/settings.json with their own file as easily as mounting it to /data.

@haugene
Copy link
Owner

haugene commented May 24, 2015

Yes, looks good!

About the /data/transmission-data, I'm using that one. But it's not for the settings.json, that will be rewritten from the template on every startup anyways. It's because Transmission stores torrent state there, like which torrents are added, active, paused, etc.
I have this container in a docker-compose and I have been tearing down and setting the containers up a lot, then it's nice not to have to add the torrents again.
But... I will admit, it's not that big deal. If people want they can mount the /etc/transmission-daemon as well to keep state. So yes, we should maybe remove it...

Can you recreate the pull request against the dev branch? Then I can make some small modifications as well, before pushing new version.

@ianneub ianneub closed this May 24, 2015
@ianneub ianneub mentioned this pull request May 24, 2015
@ianneub ianneub deleted the bind-transmission-to-tun0 branch May 24, 2015 14:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants