Docker: Add DATA_PATH to cron env

[zhaofengli]

Changes proposed in this pull request:

• Adds $DATA_PATH to the cron environment

This fixes automatic feed updating with non-standard data paths.

How to test the feature manually:

1. Set a non-standard DATA_PATH and enable automatic feed updating with CRON_MIN
2. cat /tmp/FreshRSS.log
3. See successful result instead of Failed requirements!

Pull request checklist:

• clear commit messages
• code manually tested
• unit tests written (optional if too hard)
• documentation updated

Additional information can be found in the documentation.

[Alkarex]

Hello,
It probably does not hurt to pass this variable, so let's do it. However, the expected way in Docker is to map the data path to a volume, which can be where you want, so I am a bit unsure of your use-case. Could you provide a bit more info?
https://github.com/FreshRSS/FreshRSS/tree/edge/Docker#docker-compose

Please add a line for you in https://github.com/FreshRSS/FreshRSS/blob/edge/CREDITS.md

[Alkarex]

Related to #5423

[zhaofengli]

However, the expected way in Docker is to map the data path to a volume, which can be where you want, so I am a bit unsure of your use-case.

Sorry for the late reply. My use-case is to deploy FreshRSS on Kubernetes with Litestream¹. Litestream runs as a sidecar container with a shared ephemeral directory mounted at /data. I could have used /var/www/FreshRSS/data, but DATA_PATH exists and I want to use a path consistent with other apps I deploy.

Please add a line for you in edge/CREDITS.md

Thanks, I'll add it the next time I contribute 😄

Footnotes

1. I also patch FreshRSS downstream to use a single SQLite database with table prefixes, but this is obviously too cursed to be upstreamed. ↩

[Alkarex]

Thanks for the explanations.

Thanks, I'll add it the next time I contribute

You can just make a new PR for that