Page MenuHomePhabricator

geoiplookup should have Cache-Control headers
Closed, ResolvedPublic

Description

Opening bug, since nothing happened since last week.


Version: unspecified
Severity: normal

Details

Reference
bz25910

Event Timeline

bzimport raised the priority of this task from to Medium.Nov 21 2014, 11:19 PM
bzimport set Reference to bz25910.
bzimport added a subscriber: Unknown Object (MLST).

CC Ryan Lane. Talked to him about this on IRC last week, and he thinks we shouldn't use Cache-Control here.

I think a cookie is probably a better idea, specifically a session cookie.

That said, I'm not directly working on this part of the codebase, so if someone comes up with a good solution, and the privacy implications are discussed with the community, any solution is good with me.

From the ops side, this isn't causing any real performance impacts.