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

Update transmission-settings.json #163

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

Conversation

Clickbaitcake
Copy link
Contributor

I think forcing a default password is a good idea specially for folks who run Nefarious on a cloud hosted server. This would need to be reflected in the readme, happy to submit a pull for that as well. This is being discussed in #159

I think forcing a default password is a good idea specially for folks who run Nefarious on a cloud hosted server. This would need to be reflected in the read me, happy to submit a pull for that.
@lardbit
Copy link
Owner

lardbit commented Jan 1, 2022

I am all for this, but I'd like to consider how to piggy-back off the environment variables defined in .env . I'd like to prevent duplication of configuration but I'll have to think about how this would work.

Ideally, we'd add two new variables in .env that would get passed to Transmission:

TRANSMISSION_USER
TRANSMISSION_PASS

@lardbit
Copy link
Owner

lardbit commented Jan 1, 2022

Oh, it looks like the linuxserver/transmission supports USER and PASS environment variables so we can populate them via the .env file.

@lardbit
Copy link
Owner

lardbit commented Jan 1, 2022

Thanks for the help pointing this out and coming up with solutions. I still need to test the VPN setup, but #167 sets a default transmission user/pass without requiring the user to modify the transmission-settings.conf file. I'd appreciate it if you could help me verify it works for you as well.

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