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

Following the README results in a "state must be a string" error #15

Open
kevlened opened this issue Aug 17, 2023 · 4 comments
Open

Following the README results in a "state must be a string" error #15

kevlened opened this issue Aug 17, 2023 · 4 comments

Comments

@kevlened
Copy link

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.

@kevlened
Copy link
Author

This would be resolved properly (without removing the check) by merging #14

@g4uravbahugun4
Copy link

g4uravbahugun4 commented Sep 9, 2023

@kevlened I do not understand , please explain?

@kevlened
Copy link
Author

kevlened commented Sep 9, 2023

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:

  1. Comment out the state validation in your local version. This is insecure, but allows the sample to run.
  2. The more secure option is to change the line shown in this PR: Changes GET to POST request when getting an access token #14

@RealmX1
Copy link

RealmX1 commented Oct 30, 2024

The merge is still not done.

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

No branches or pull requests

3 participants