intuit / intuit/oauth-pythonclient

Adding support for `x-include-refresh-token-hard-expires-in`

Open Beginner friendly
#56 0 comments 0 reactions 0 assignees View on GitHub
Dominant language
Python
Stars
86
Forks
59
PR merge metrics
No merged PRs in 30d

Description

Hello!

Regarding: https://medium.com/intuitdev/track-your-refresh-tokens-hard-expiry-with-a-new-response-field-7a95ed3ab67f

I have built a wrapper around `intuitlib.client.AuthClient` that subclasses both `refresh` and `get_bearer_token` with adding a new param `include_refresh_token_hard_expires_in=False`. When set to `True` is adds the necessary header field. It also adds `self.x_refresh_token_hard_expires_in = None` to the init.

Would you be open to a pull request to get this into the official version? I see in the blog it's *coming soon*. Not sure if that means it's already in the works or not.

Thanks.

Contributor guide

No contributing guide indexed for this repository

Research direction

Start with intuitlib.client.AuthClient and inspect the refresh and get_bearer_token methods described in the issue. Compare their existing parameters and response handling with the requested include_refresh_token_hard_expires_in option; done means the option adds the required header and exposes the returned value through x_refresh_token_hard_expires_in.

Written by the indexing model from the issue text.

Assessment

Tech stack
python
Domain
authentication
Issue type
Feature
Difficulty
2/5
Estimated time
1-3 hours
Activity status
Quiet
Clarity
Mostly clear
Newbie friendliness
72/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.