RefreshToken Endpoint #74
Open
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
…g Python, so its impossible to build C extensions afterwards, wcwidth==0.2.6 and cwcwidth==0.1.8 in this case
Aded Refreshtoken rereash route and adjusted some token lifetime values
Collaborator
|
@TaqsBlaze Thank you for taking the time to create this PR. We don't have a GitHub issue for this pull request. We typically develop features based on Github issues. We describe the issue in detail in the GitHub issue and assign someone to work on it. Including a detailed description in the Pull Request explaining what changes your PR is making and the rationale behind the change is helpful. @maunga-et started this auth work in #72, and I haven't merged his PR yet. Is the token refresh missing from that PR? If it is, I prefer that we add it to PR #72 or wait until that PR is reviewed and merged, before adding this change. |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
3 participants
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Added refresh token endpoint
Added refresh token serializer
Adjusted access token lifetime to 1 hour
Adjusted refresh token lifetime to 7 days