search

Athou / commafeed

Google Reader inspired self-hosted personal RSS reader.
https://www.commafeed.com
Apache License 2.0
2.81k stars 377 forks source link

Max entries age per feed #1547

Closed agios closed 2 months ago

agios commented 2 months ago

Is your feature request related to a problem? Please describe. There are some feeds that have a low volume of entries, but the old entries can still be relevant. Currently if cleanup is configured, it affects all feeds.

Describe the solution you'd like If we could set the maxEntriesAgeDays per feed, we could have it longer or disabled for specific feeds, while others can be cleaned up more frequently.

Athou commented 2 months ago

This is not trivial because feeds are shared with all users that are subscribed to them (see here and here).

You could set commafeed.database.cleanup.entries-max-age to 0 though and let commafeed.database.cleanup.max-feed-capacity cleanup feeds that are over capacity.

agios commented 2 months ago

Thank you, that will work!

Can this also be overriden with an env variable which is easier for my docker deployment? In that case would it be COMMAFEED_DATABASE_CLEANUP_ENTRIES_MAX_AGE?

Athou commented 2 months ago

Can this also be overriden with an env variable which is easier for my docker deployment? In that case would it be COMMAFEED_DATABASE_CLEANUP_ENTRIES_MAX_AGE?

Yep, see https://github.com/Athou/commafeed/blob/master/commafeed-server/doc/commafeed.adoc

agios commented 2 months ago

Thanks again!