Skip to content
This repository has been archived by the owner on Feb 8, 2018. It is now read-only.

Ignore cookies on static assets #2043

Closed
zwn opened this issue Feb 16, 2014 · 1 comment
Closed

Ignore cookies on static assets #2043

zwn opened this issue Feb 16, 2014 · 1 comment
Labels
TeamX ★

Comments

@zwn
Copy link
Contributor

zwn commented Feb 16, 2014

Right now various hooks (including authentication.inbound) are run very early in the request processing. These hooks needlessly hit the database reading data or even writing (for each request, even for 304). For static assets (=files) nothing like that is necessary. Found at #2041 and #1878 (comment).
@zwn zwn added the DevX ★ label Feb 16, 2014
@seanlinsley seanlinsley mentioned this issue Apr 27, 2014
Merged
@Changaco
Copy link
Contributor

Changaco commented May 2, 2014

This has been fixed in #2332.

@Changaco Changaco closed this as completed May 2, 2014
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees
No one assigned
Labels
TeamX ★
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@zwn @Changaco