get_bearer_token and refresh should return the parsed JSON response.
Nobody has claimed this yet.
Assessment
- Difficulty
- 3/5
- Estimated time
- 1-2 days
- Newbie friendliness
- 35/100
- Issue type
- Feature
- Clarity
- Mostly clear
- Activity status
- Stale
- Tech stack
- python
- Domain
- api, authentication
Research direction
Start by tracing the get_bearer_token and refresh entry points and how they currently copy JSON fields onto AuthClient. Update the behavior so both methods return the parsed JSON response, then verify that callers can use the returned value for attrs-based strict runtime type checking.
Written by the indexing model from the issue text.
Description
Currently these only set the values in the json response as attributes of the AuthClient this makes it difficult to know which attributes will be available when, for instance, get_bearer_token sets realm_id but refresh doesn't.
We want results of talking to AuthClient to be able to participate in our strict runtime type checking which involves converting a primitive python type into an attrs dataclass. Having to construct this dictionary out of the AuthClient before we can do that is annoying and error prone.
- Dominant language
- Python
- Stars
- 86
- Forks
- 59
- PR merge metrics
- No merged PRs in 30d
Contributor guide
No contributing guide indexed for this repository
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
More from intuit/oauth-pythonclient
-
Difficulty 2/5 1-3 hours Newbie friendliness 72/100
intuit/oauth-pythonclient#56 ·
-
Difficulty 2/5 1-3 hours Newbie friendliness 68/100
intuit/oauth-pythonclient#52 ·
-
input Open
Difficulty 5/5 Over a week Newbie friendliness 10/100
intuit/oauth-pythonclient#55 · 2 comments ·
-
Difficulty 4/5 3-5 days Newbie friendliness 25/100
intuit/oauth-pythonclient#47 · 1 comment · 1 reaction ·
-
Difficulty 3/5 1-2 days Newbie friendliness 20/100
intuit/oauth-pythonclient#38 · 3 comments ·
All issues in intuit/oauth-pythonclient
Similar issues
-
enhancement
Difficulty 2/5 1-3 hours Newbie friendliness 70/100
canonical/paas-charm#368 · 1 comment ·
-
Difficulty 2/5 1-3 hours Newbie friendliness 75/100
-
tech debt
Difficulty 2/5 1-3 hours Newbie friendliness 75/100
-
Difficulty 1/5 Under an hour Newbie friendliness 90/100
StevenBlack/hosts#3256 ·
-
Difficulty 1/5 Under an hour Newbie friendliness 90/100
qualcomm/qai-appbuilder#275 ·