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

Refresh token after expire #34

Merged
merged 1 commit into from
Aug 14, 2021
Merged

Refresh token after expire #34

merged 1 commit into from
Aug 14, 2021

Conversation

erikn69
Copy link
Contributor

@erikn69 erikn69 commented Aug 10, 2021

Closes #33

@erikn69 erikn69 mentioned this pull request Aug 10, 2021
@masbug masbug merged commit 32549c0 into masbug:1.x Aug 14, 2021
@foremtehan
Copy link

any plan to release new PRs?

@erikn69
Copy link
Contributor Author

erikn69 commented Aug 23, 2021

I can't answer that, @masbug has the super power 🚀

@sr2ds
Copy link

sr2ds commented Feb 15, 2022

Hello everyone, thanks for fix this.

Has some way to force to expire token programmatically?
If yes, how to? Maybe we also can setup one specific test to ensure this situation.

 public function testRefreshTokenAfterExpires()
{
        $adapter = $this->adapter();
        $adapter->logout();
        // assert with error 

        $adapter->refreshToken();
        // assert with success
}

@erikn69
Copy link
Contributor Author

erikn69 commented Feb 15, 2022

Has some way to force to expire token programmatically?

why?

@sr2ds
Copy link

sr2ds commented Feb 15, 2022

Has some way to force to expire token programmatically?

why?

I need make tests in my application and I don't know how to expire token to test, you know?

@erikn69
Copy link
Contributor Author

erikn69 commented Feb 15, 2022

@sr2ds
Copy link

sr2ds commented Feb 15, 2022

https://googleapis.github.io/google-api-php-client/main/Google_Client.html revokeToken

Thanks!

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.

4 participants