Skip to content

fix for parameter cache_key#4

Open
cbillen wants to merge 2 commits intoAMar4enko:masterfrom
warmlyyours:master
Open

fix for parameter cache_key#4
cbillen wants to merge 2 commits intoAMar4enko:masterfrom
warmlyyours:master

Conversation

@cbillen
Copy link
Contributor

@cbillen cbillen commented Dec 12, 2016

cache key generation did not take parameter into consideration we always ended up with the same md5

This pull request reverts and fixes this but some of the tests are no longer working and return "Tried to filter for declared parameters but none exist.", this might be due to changes inside grape
https://github.com/ruby-grape/grape/pull/1142/files. I'm not sure how we get those test passing again though.

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.

1 participant