Add application setting to disable ETag (again) #1694

Merged
merged 1 commit into from Aug 2, 2013

Conversation

Projects
None yet
2 participants
Contributor

kavu commented Jul 21, 2013

It's updated to upstream duplicate of the old Pull Request #1421.

Old Pull Request description:

Add application setting app.disable('etag'); to disable ETag. Manual ETag addition via res.set('etag', 'somestring);' still works.

Please, check this feature.

Btw, it's my first experience in Express and JS BDD, so, please, if there are any issues or mistakes (for ex., I am not sure about proper spec placement), let me know, maybe I'll can fix 'em.

Thanks!

@visionmedia, sorry for such a long delay, I totally forgot about this pull request and that I need to sync it with upstream. I hope everything is good now.

tj added a commit that referenced this pull request Aug 2, 2013

Merge pull request #1694 from kavu/add_disable_etag
Add application setting to disable ETag (again)

@tj tj merged commit 610e172 into expressjs:master Aug 2, 2013

1 check failed

default The Travis CI build failed
Details

rlidwka pushed a commit to rlidwka/express that referenced this pull request Aug 6, 2014

Merge pull request #1694 from kavu/add_disable_etag
Add application setting to disable ETag (again)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment