-
Notifications
You must be signed in to change notification settings - Fork 50
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
Following the README results in a "state must be a string" error #15
Comments
This would be resolved properly (without removing the check) by merging #14 |
@kevlened I do not understand , please explain? |
If you follow the steps (listed here) in the README, the sample does not work. When you try to sign in, you will receive a "state must be a string" error message. To avoid the problem, there are two potential solutions:
|
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Zoom isn't sending a state query parameter in the oauth flow. This breaks the check for the state parameter.
My workaround is to comment out the check here, since this is just an example.
The text was updated successfully, but these errors were encountered: