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

Make caching work for the Report API #1382

Merged
merged 2 commits into from
Apr 20, 2022
Merged

Conversation

evgeni
Copy link
Member

@evgeni evgeni commented Apr 20, 2022

When implementing the Report API, caching was not properly implemented
and wasn't working for users.

Fixes: #1246

When implementing the Report API, caching was not properly implemented
and wasn't working for users.

Fixes: theforeman#1246
@evgeni
Copy link
Member Author

evgeni commented Apr 20, 2022

It is expected, as we cache http requests and their replies.

@mpl241
Copy link

mpl241 commented Apr 20, 2022

@evgeni, Sorry, I moved my post to the issue.

@evgeni evgeni merged commit a8b0295 into theforeman:develop Apr 20, 2022
@evgeni evgeni deleted the i1246 branch April 20, 2022 17:00
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.

Caching parameters doesn't seem to work
3 participants