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

Upgrade redis dependency to v9 #17

Closed
wants to merge 1 commit into from
Closed

Conversation

hgokhale
Copy link

We are having some dependency issues so wanted to upgrade redis dependency to V9

All tests are passing after this change

=== RUN   TestTokenStore

  Test redis token store 
    Test authorization code store ✔✔✔✔✔✔
    Test access token store ✔✔✔✔✔✔
    Test refresh token store ✔✔✔✔✔✔


18 total assertions

--- PASS: TestTokenStore (0.02s)
=== RUN   TestTokenStoreWithKeyNamespace

  Test redis token store 
    Test authorization code store ✔✔✔✔✔✔
    Test access token store ✔✔✔✔✔✔
    Test refresh token store ✔✔✔✔✔✔


36 total assertions

--- PASS: TestTokenStoreWithKeyNamespace (0.02s)
PASS
ok  	github.com/go-oauth2/redis/v4	(cached)

@hgokhale
Copy link
Author

Created another PR from separate branch so closing this one - #18

@hgokhale hgokhale closed this Feb 20, 2023
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

Successfully merging this pull request may close these issues.

1 participant