Skip to content

Add option to disable refresh token invalidation (#6584) #6587

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
Merged
merged 1 commit into from
Apr 12, 2019
Merged

Add option to disable refresh token invalidation (#6584) #6587

merged 1 commit into from
Apr 12, 2019

Conversation

jonasfranz
Copy link
Member

Backport of #6584

@jonasfranz
Add option to disable refresh token invalidation (go-gitea#6584)
bc0aa0e 
* Add option to disable refresh token invalidation

Signed-off-by: Jonas Franz <info@jonasfranz.software>

* Add integration tests and remove wrong todos

Signed-off-by: Jonas Franz <info@jonasfranz.software>

* Fix typo

Signed-off-by: Jonas Franz <info@jonasfranz.software>

* Fix tests and add documentation

Signed-off-by: Jonas Franz <info@jonasfranz.software>

(cherry picked from commit 783cd64)
Signed-off-by: Jonas Franz <info@jonasfranz.software>
@jonasfranz jonasfranz added this to the 1.8.0 milestone Apr 12, 2019
@jonasfranz jonasfranz mentioned this pull request Apr 12, 2019
Merged
@GiteaBot GiteaBot added the lgtm/need 1 This PR needs approval from one additional maintainer to be merged. label Apr 12, 2019
@GiteaBot GiteaBot added lgtm/done This PR has enough approvals to get merged. There are no important open reservations anymore. and removed lgtm/need 1 This PR needs approval from one additional maintainer to be merged. labels Apr 12, 2019
@lafriks lafriks added the type/enhancement An improvement of existing functionality label Apr 12, 2019
@lafriks lafriks merged commit 72f4cdf into go-gitea:release/v1.8 Apr 12, 2019
@jonasfranz jonasfranz deleted the release/v1.8 branch April 12, 2019 10:44
@go-gitea go-gitea locked and limited conversation to collaborators Nov 24, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@lafriks lafriks lafriks approved these changes

+1 more reviewer

@kolaente kolaente kolaente approved these changes

Reviewers whose approvals may not affect merge requirements
Assignees
No one assigned
Labels
lgtm/done This PR has enough approvals to get merged. There are no important open reservations anymore. type/enhancement An improvement of existing functionality
Projects
None yet
Milestone
1.8.0
Development

Successfully merging this pull request may close these issues.
4 participants
@jonasfranz @lafriks @kolaente @GiteaBot