Skip to content
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

cache served microformats2 responses #15

Closed
snarfed opened this issue Jan 4, 2014 · 2 comments
Closed

cache served microformats2 responses #15

snarfed opened this issue Jan 4, 2014 · 2 comments

Comments

@snarfed
Copy link
Owner

snarfed commented Jan 4, 2014

...with a short expiration. ideally include the cache expiration in the headers and/or content itself.

alternatively, serve them from the stored response JSON in the datastore. drawback is that we don't get updated content.

@snarfed
Copy link
Owner Author

snarfed commented Mar 25, 2014

when we hit the G+ daily API limit, we can't fetch comments or likes in mf2 handlers, which results in ugly half-rendered comments. caching would help a bit; serving from stored responses would fix it entirely.

from discussion in IRC. cc @tantek.

@snarfed
Copy link
Owner Author

snarfed commented May 3, 2016

done, yay. oldest issue we've closed in a long time. :P

oh and for the record, serving from stored responses would be tricky since we always want to serve the current silo response, or 404/410 if it's missing, etc. this should usually still work with stored responses, since we update them during poll if they change, but only every poll period, so they could be stale, and we also may often miss updating them entirely right now since we don't see responses past the first page. that's #657.

@snarfed snarfed closed this as completed May 3, 2016
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

No branches or pull requests

1 participant