GrahamCampbell/Laravel-GitHub

Allow the max cache lifetime to be set

GrahamCampbell opened this issue · 5 comments

The current default is 30 days. There are cases where people will want to lower this where they are sending millions of requests.

NB ['cache_lifetime' => $time_in_seconds] should be the 2nd parameter passed to addCache when implemented in getBuilder of GitHubFactory

Lowered the default to 48 hours.

Still no option to set as of yet.

Also, changed the caching system to strictly only cache using etags, so always sending requests: c0b74de.

I think this is fine now.