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

etag use gmt_modified #288

Closed
fengmk2 opened this Issue Mar 21, 2014 · 0 comments

Comments

Projects
None yet
1 participant
@fengmk2
Member

fengmk2 commented Mar 21, 2014

Current etag deps on content crc32 etag, it will cause too many memory whatever content not modified.

Use gmt_modified to detect 304 or not.

@fengmk2 fengmk2 added the enhancement label Mar 21, 2014

@fengmk2 fengmk2 added this to the v0.3 koa milestone Mar 21, 2014

@fengmk2 fengmk2 self-assigned this Mar 21, 2014

@fengmk2 fengmk2 closed this in f160d74 Mar 21, 2014

fengmk2 added a commit that referenced this issue Mar 21, 2014

Merge pull request #290 from cnpm/modified-as-etag
use module gmt_modified as etag. fix #288
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment