Since we no longer delete duplicates, expired access tokens are no longer removed. Cron only removed access token based on refresh tokens, and refreshing a access token only deletes the used refresh token.

CommentFileSizeAuthor
#4 2821436--delete-expired-tokens--4.patch4.56 KBe0ipso
#3 simple_oauth-expired_access_token_not_removed-2821436-3.patch1.85 KBseanb
#2 simple_oauth-expired_access_token_not_removed-2821436-2.patch1.76 KBseanb

Comments

seanB created an issue. See original summary.

seanb’s picture

seanb
Primary language Dutch
Location Netherlands
commented
Status: Active » Needs review
StatusFileSize
new simple_oauth-expired_access_token_not_removed-2821436-2.patch1.76 KB

Patch is attached to removed the old access token on refresh.

seanb’s picture

seanb
Primary language Dutch
Location Netherlands
commented
StatusFileSize
new simple_oauth-expired_access_token_not_removed-2821436-3.patch1.85 KB

There was still an issue when the access token was not expired yet (but you're still trying to refresh it). An updated patch is attached.

e0ipso’s picture

e0ipso
Primary language Catalan
Location Can Picafort
commented
StatusFileSize
new 2821436--delete-expired-tokens--4.patch4.56 KB

I made some minor modifications to the patch.

  • e0ipso committed 0875a25 on 8.x-1.x authored by seanB 
    Issue #2821436 by seanB, e0ipso: Expired access tokens not removed on...
e0ipso’s picture

e0ipso
Primary language Catalan
Location Can Picafort
commented
Status: Needs review » Fixed

Thanks!

  • e0ipso committed 0875a25 on 8.x-2.x authored by seanB 
    Issue #2821436 by seanB, e0ipso: Expired access tokens not removed on...

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.