Skip to content

store and retrieve response status code#65

Merged
krazyjakee merged 3 commits intorv-kip:masterfrom
artursudnik:status-code
Sep 11, 2018
Merged

store and retrieve response status code#65
krazyjakee merged 3 commits intorv-kip:masterfrom
artursudnik:status-code

Conversation

@artursudnik
Copy link
Contributor

Correction that stores and serves from cache http response status codes. This is important when for example part of application crashes after expensive computing and returns "500" to web browser. When responding with 200 status code and error message, response can be for example cached client side.

@rv-kip
Copy link
Owner

rv-kip commented Jun 30, 2017

If this pull request is still desired, please fix the conflicts and we'll get it merged. Otherwise, please close.

Conflicts:
	lib/ExpressRedisCache/route.js
@artursudnik
Copy link
Contributor Author

fixed

@Spasfonx
Copy link

This PR will be merged some day ? I'm still facing this bug which return a 200 status code when a 500 is cached.

@artursudnik
Copy link
Contributor Author

I wish it was.

@krazyjakee krazyjakee merged commit df4ed8e into rv-kip:master Sep 11, 2018
@mattcasey
Copy link

Hi, the PR was merged but the latest version does not include it. Can we get a version bump?

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.

5 participants