Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

#

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Jump to bottom

Remove re-validation information with GreedyCache #86

Merged
merged 1 commit into from
Jul 10, 2017
Merged

Remove re-validation information with GreedyCache #86

merged 1 commit into from
Jul 10, 2017

Conversation

Kevinrob
Copy link
Owner

@Kevinrob Kevinrob commented Jul 9, 2017

Issue #62

@Kevinrob Kevinrob added the bug label Jul 9, 2017
@Kevinrob Kevinrob self-assigned this Jul 9, 2017
@Kevinrob Kevinrob mentioned this pull request Jul 9, 2017
Closed
@Kevinrob Kevinrob merged commit aa9ddb6 into master Jul 10, 2017
@afoeder
Copy link
Contributor

afoeder commented Jul 13, 2017

looks good; though I'm not sure if having added the headers in the first place is not maybe wrong already?

@Kevinrob
Copy link
Owner Author

@afoeder yes and no. It's from the server and the GreedyCache don't have to respect the headers from the server.

@Kevinrob Kevinrob deleted the issue-62 branch November 5, 2023 12:32
# for free to join this conversation on GitHub. Already have an account? # to comment
Reviewers
No reviews
Assignees

@Kevinrob Kevinrob

Labels
bug
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@Kevinrob @afoeder