Skip to content

Conversation

@GiteaBot
Copy link
Collaborator

@GiteaBot GiteaBot commented Dec 5, 2023

Backport #28352 by @KN4CK3R

Fixes #28324

The name parameter can't contain some characters (https://github.com/keybase/go-crypto/blob/master/openpgp/keys.go#L680) but is optional. Therefore just use an empty string.

Fixes go-gitea#28324

The name parameter can't contain some characters
(https://github.com/keybase/go-crypto/blob/master/openpgp/keys.go#L680)
but is optional. Therefore just use an empty string.
@GiteaBot GiteaBot added the lgtm/need 2 This PR needs two approvals by maintainers to be considered for merging. label Dec 5, 2023
@GiteaBot GiteaBot added this to the 1.21.2 milestone Dec 5, 2023
@GiteaBot GiteaBot requested review from delvh, jolheiser and lunny December 5, 2023 00:48
@GiteaBot GiteaBot added lgtm/need 1 This PR needs approval from one additional maintainer to be merged. and removed lgtm/need 2 This PR needs two approvals by maintainers to be considered for merging. labels Dec 5, 2023
@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 Dec 5, 2023
@lunny lunny merged commit b71d4c3 into go-gitea:release/v1.21 Dec 5, 2023
@go-gitea go-gitea locked as resolved and limited conversation to collaborators Mar 4, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

lgtm/done This PR has enough approvals to get merged. There are no important open reservations anymore. topic/packages type/bug

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants