Comments (6)
I'd be happy to get into this a bit, if no one is working on it. Need this for something I'm working on at the moment.
from requests-oauthlib.
It seems that my server wasn't returning token_type as part of the response. So this issue should turn into "Add a clearer missing token type error message".
from requests-oauthlib.
@thedrow Thanks for the update. Looking at this, I'm not sure that any work really needs to be done here. It seems to me that raising MissingTokenTypeError is pretty clear about what went wrong. What's more, that is actually the behavior of oauthlib, so it's not particularly suitable for requests-oauthlib to address the issue.
I propose we close this issue.
from requests-oauthlib.
I had to print the response to figure this out. It wasn't easy and it took me about an hour and a half to debug this.
The only thing that should be done is to provide a better error message.
from requests-oauthlib.
Even so, that is an issue for oauthlib, not requests-oauthlib. Look at where the exception is being raised. It's not in requests-oauthlib code. If simply raising MissingTokenTypeError is not enough, I think that is an issue for oauthlib to address, rather than adding code to raise a new exception to say essentially the same thing in requests-oauthlib.
from requests-oauthlib.
@thedrow @nicksloan better error messages are always nice whichever repo, will open a little todo on oauthlib to look into this.
from requests-oauthlib.
Related Issues (20)
- Question: Is this project maintained? HOT 1
- Compliance fix for tokens that never expire (e.g. Mailchimp)
- Redo request after token is updated
- Why is the backend client workflow asking for a code or a response url? HOT 1
- refresh token workflow HOT 1
- rsengun HOT 1
- How to test if LinkedIn token is still valid and re-use? HOT 2
- 1.3.2 release? HOT 3
- (insecure_transport) OAuth 2 MUST utilize https. HOT 4
- LinkedIn issue with multiple scopes HOT 1
- ImportError: No module named parse HOT 3
- Remove 2.7 from setup.py HOT 2
- Access tokens leaked in logs when using DEBUG level
- 1.4.0 claims to support Python 2.7 in PyPI but does not HOT 2
- Changelog suggests Python 3.4-3.6 is unsupported, but project metadata allows these versions
- Release template purposal HOT 2
- Issue refreshing Wix OAuth tokens - cannot send payload as JSON using `refresh_token_request` compliance hook HOT 3
- Can you please add the ability to retrieve and provide the code verifier HOT 1
- BROKEN DOCUMENTATION: Backend Flow
- OAuth 2 web application flow not support PKCE??
Recommend Projects
-
React
A declarative, efficient, and flexible JavaScript library for building user interfaces.
-
Vue.js
🖖 Vue.js is a progressive, incrementally-adoptable JavaScript framework for building UI on the web.
-
Typescript
TypeScript is a superset of JavaScript that compiles to clean JavaScript output.
-
TensorFlow
An Open Source Machine Learning Framework for Everyone
-
Django
The Web framework for perfectionists with deadlines.
-
Laravel
A PHP framework for web artisans
-
D3
Bring data to life with SVG, Canvas and HTML. 📊📈🎉
-
Recommend Topics
-
javascript
JavaScript (JS) is a lightweight interpreted programming language with first-class functions.
-
web
Some thing interesting about web. New door for the world.
-
server
A server is a program made to process requests and deliver data to clients.
-
Machine learning
Machine learning is a way of modeling and interpreting data that allows a piece of software to respond intelligently.
-
Visualization
Some thing interesting about visualization, use data art
-
Game
Some thing interesting about game, make everyone happy.
Recommend Org
-
Facebook
We are working to build community through open source technology. NB: members must have two-factor auth.
-
Microsoft
Open source projects and samples from Microsoft.
-
Google
Google ❤️ Open Source for everyone.
-
Alibaba
Alibaba Open Source for everyone
-
D3
Data-Driven Documents codes.
-
Tencent
China tencent open source team.
from requests-oauthlib.