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.

Sign up for GitHub

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

Jump to bottom

Stripe.Connect.OAuth.deauthorize bombs with 401 #462

Closed
zgohr opened this issue Feb 25, 2019 · 1 comment
Closed

Stripe.Connect.OAuth.deauthorize bombs with 401 #462

zgohr opened this issue Feb 25, 2019 · 1 comment
Assignees
@zgohr
Labels
2.0 bug

Comments

@zgohr
Copy link
Contributor

zgohr commented Feb 25, 2019

The deauthorize request goes through oauth_request and not perform_request of the API module. As such, the API/secret key isn't sent with the request and will always bomb with 401 from Stripe.

If there's a preferred way to resolve this I can put together a PR, just let me know.
@snewcomer
Copy link
Collaborator

@zgohr Yeah I think you found a bug! A PR would be very welcome. I can get to it this Friday if you don't have time so just lmk.

https://stripe.com/docs/connect/oauth-reference#post-deauthorize

zgohr added a commit to zgohr/stripity_stripe that referenced this issue Feb 27, 2019
@zgohr
Resolves beam-community#462. Add authorization header api key to deau…
9d60113 
…thorization oauth request.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@zgohr zgohr

Labels
2.0 bug
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@zgohr @snewcomer