Skip to content
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

API: Fix null pointer in attempt to Sudo if not logged in (#5872) #5884

Merged
merged 1 commit into from
Jan 28, 2019
Merged

API: Fix null pointer in attempt to Sudo if not logged in (#5872) #5884

merged 1 commit into from
Jan 28, 2019

Conversation

zeripath
Copy link
Contributor

@zeripath zeripath commented Jan 28, 2019 •
edited by techknowlogick
Loading

#5872
Signed-off-by: Andrew Thornton [email protected]

@techknowlogick techknowlogick added this to the 1.7.1 milestone Jan 28, 2019
@techknowlogick techknowlogick added type/bug modifies/api This PR adds API routes or modifies them labels Jan 28, 2019
@techknowlogick techknowlogick changed the title API: Fix null pointer in attempt to Sudo if not logged in (Backport #5872) API: Fix null pointer in attempt to Sudo if not logged in (#5872) Jan 28, 2019
@GiteaBot GiteaBot added the lgtm/need 1 This PR needs approval from one additional maintainer to be merged. label Jan 28, 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 Jan 28, 2019
@zeripath zeripath merged commit 2979953 into go-gitea:release/v1.7 Jan 28, 2019
@zeripath zeripath deleted the fix-null-pointer-on-failed-login-with-sudo branch January 28, 2019 22:03
@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

@techknowlogick techknowlogick techknowlogick approved these changes

@adelowo adelowo adelowo approved these changes

Assignees
No one assigned
Labels
lgtm/done This PR has enough approvals to get merged. There are no important open reservations anymore. modifies/api This PR adds API routes or modifies them type/bug
Projects
None yet
Milestone
1.7.1
Development

Successfully merging this pull request may close these issues.
4 participants
@zeripath @techknowlogick @adelowo @GiteaBot