Skip to content
This repository has been archived by the owner on May 31, 2022. It is now read-only.

Remove unnecessary ACCESS_TO_REFRESH tuple in RedisTokenStore #1138

Open
Mert-Z opened this issue Aug 16, 2017 · 0 comments
Open

Remove unnecessary ACCESS_TO_REFRESH tuple in RedisTokenStore #1138

Mert-Z opened this issue Aug 16, 2017 · 0 comments

Comments

@Mert-Z
Copy link

Mert-Z commented Aug 16, 2017

Tuples stored with ACCESS_TO_REFRESH key are never queried. Can it be removed or is it left intentionally for future use? If for future use, can it be documented via code comments?

If we remove its references, we can also get rid of the bug on L313 where ACCESS_TO_REFRESH is tried to be cleared by serializing with refresh token as key whereas it should be cleared by serializing related access token as key.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Development

No branches or pull requests

2 participants