happydpc / goauth2

Automatically exported from code.google.com/p/goauth2
BSD 3-Clause "New" or "Revised" License
0 stars 0 forks source link

Patch for /appengine/serviceaccount/cache.go #28

Closed GoogleCodeExporter closed 9 years ago

GoogleCodeExporter commented 9 years ago
When fetching an item from memcache, Item.Expiration does not accurately 
reflect the actual expiration Duration set for that item so any token retrieved 
from memcache is always being mistakenly flagged as expired.

Original issue reported on code.google.com by prat...@referabuyer.com on 7 Jan 2014 at 4:47

Attachments:

GoogleCodeExporter commented 9 years ago
[deleted comment]
GoogleCodeExporter commented 9 years ago
[deleted comment]
GoogleCodeExporter commented 9 years ago
Added patch to codereview: https://codereview.appspot.com/95560052/

Original comment by prat...@condorcapital.com on 22 May 2014 at 6:05

GoogleCodeExporter commented 9 years ago
I believe that this issue may be hitting us in production code: 
https://groups.google.com/forum/#!topic/google-appengine-go/8V1b6gJ5BR0

Original comment by ianros...@gmail.com on 22 May 2014 at 6:26

GoogleCodeExporter commented 9 years ago
Just ran into this, used up all of my access token quota in about an hour.  The 
provided patch seems to be good, I wonder when they're going to actually apply 
it.  In my version I also added an in-memory cache.

Original comment by ch...@pushbullet.com on 3 Jun 2014 at 11:59

GoogleCodeExporter commented 9 years ago

Original comment by gmle...@google.com on 24 Jun 2014 at 6:31

GoogleCodeExporter commented 9 years ago
This has been fixed in this version:
https://code.google.com/p/goauth2/source/detail?r=bf0f88586e95

Original comment by gmle...@google.com on 7 Jul 2014 at 9:24