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

FloodUI not persisting config.json #807

Closed
tomato4 opened this issue Dec 24, 2023 · 6 comments
Closed

FloodUI not persisting config.json #807

tomato4 opened this issue Dec 24, 2023 · 6 comments

Comments

@tomato4
Copy link

tomato4 commented Dec 24, 2023

Every time the transmission is updated, the app looses it's configuration (eg. Common Paths). I tried to extract config.json (in /themes/flood-for-transmission) and bind it as external volume, but the mod just rewrites that. I tried to bind it as only RO, but the app just ignores the config (and in log is cp: cannot create regular file '/themes/flood-for-transmission/config.json': Read-only file system) and common paths in settings are empty.

I didn't even find the configuration, where the app copies the config from default to try fix it.

This bug is very annoying, cause I either don't ever update the app or I have to assign common paths again and again.

@LinuxServer-CI
Copy link

This issue has been automatically marked as stale because it has not had recent activity. This might be due to missing feedback from OP. It will be closed if no further activity occurs. Thank you for your contributions.

@aptalca
Copy link
Member

aptalca commented Feb 15, 2024

@tomato4
can you test this PR mod build and let us know if it works?
ghcr.io/linuxserver/mods:pull_request_840
just set -e DOCKER_MODS=ghcr.io/linuxserver/mods:pull_request_840

It should save the config.json in the config folder and make it persistent.

Thanks

@tomato4
Copy link
Author

tomato4 commented Feb 15, 2024

I will look at it no later than at the end of the week and I will let you know, if it is fixed.

@tomato4
Copy link
Author

tomato4 commented Feb 18, 2024

Partially the problem is solved. If I set the variables in config and start container, the config is reflected in the settings (meaning if I set common_path, the common path is set in the settings).

However if I change the settings in the app, the settings is NOT saved into the config file (/themes/flood-for-transmission/config.json). I'm also getting error in the docker log cp: cannot remove '/themes/flood-for-transmission/config.json': Resource busy. So if I want to change the settings permanently, I have to edit the config file directly.

@LinuxServer-CI
Copy link

This issue has been automatically marked as stale because it has not had recent activity. This might be due to missing feedback from OP. It will be closed if no further activity occurs. Thank you for your contributions.

@LinuxServer-CI LinuxServer-CI closed this as not planned Won't fix, can't repro, duplicate, stale May 19, 2024
@LinuxServer-CI LinuxServer-CI moved this from Issues to Done in Issue & PR Tracker May 19, 2024
Copy link

This issue is locked due to inactivity

@github-actions github-actions bot locked as resolved and limited conversation to collaborators Jun 18, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
Archived in project
Development

No branches or pull requests

3 participants