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

SimplePie use distint cache for force_feed #2852

Merged
merged 2 commits into from
Mar 28, 2020

Conversation

Alkarex
Copy link
Member

@Alkarex Alkarex commented Mar 28, 2020

#fix #2524
In general, we should avoid using the same cache for feeds that are fetched with distinct parameters / settings.
Will send PR upstream to SimplePie

#fix FreshRSS#2524
In general, we should avoid using the same cache for feeds that are
fetched with distinct parameters / settings..
Will send PR upstream to SimplePie
@Alkarex Alkarex added this to the 1.16.0 milestone Mar 28, 2020
@Alkarex
Copy link
Member Author

Alkarex commented Mar 28, 2020

@nicofrand At last :-)

@Alkarex
Copy link
Member Author

Alkarex commented Mar 28, 2020

#fix #2484

@Alkarex Alkarex mentioned this pull request Mar 28, 2020
@Alkarex Alkarex linked an issue Mar 28, 2020 that may be closed by this pull request
Copy link
Member

@Frenzie Frenzie left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks nice & simple.

@Alkarex
Copy link
Member Author

Alkarex commented Mar 28, 2020

Upstream pull request: simplepie/simplepie#643

@Alkarex Alkarex merged commit e424f45 into FreshRSS:master Mar 28, 2020
@Alkarex Alkarex deleted the simplepie_force_feed_cache branch March 28, 2020 19:24
Alkarex added a commit to Alkarex/FreshRSS that referenced this pull request Apr 1, 2020
@Alkarex Alkarex mentioned this pull request Apr 1, 2020
Alkarex added a commit that referenced this pull request Apr 1, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Cannot force feed addition Cannot add feed
2 participants