Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Fix bad method call when deleting user secrets via API #27829

Merged
merged 1 commit into from
Oct 29, 2023

Conversation

jbgomond
Copy link
Contributor

Fixed a little mistake when you deleting user secrets via the API. Found it when working on #27725.
It should be backported to 1.21 I think.

@GiteaBot GiteaBot added the lgtm/need 2 This PR needs two approvals by maintainers to be considered for merging. label Oct 29, 2023
@pull-request-size pull-request-size bot added the size/XS Denotes a PR that changes 0-9 lines, ignoring generated files. label Oct 29, 2023
@github-actions github-actions bot added the modifies/api This PR adds API routes or modifies them label Oct 29, 2023
@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 Oct 29, 2023
@KN4CK3R
Copy link
Member

KN4CK3R commented Oct 29, 2023

Would not have happened if there were tests 😬

@delvh delvh added backport/v1.21 This PR should be backported to Gitea 1.21 type/bug labels Oct 29, 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 Oct 29, 2023
@KN4CK3R KN4CK3R merged commit 641e816 into go-gitea:main Oct 29, 2023
@GiteaBot GiteaBot added this to the 1.22.0 milestone Oct 29, 2023
@KN4CK3R KN4CK3R added backport/v1.21 This PR should be backported to Gitea 1.21 and removed backport/v1.21 This PR should be backported to Gitea 1.21 labels Oct 29, 2023
@KN4CK3R
Copy link
Member

KN4CK3R commented Oct 29, 2023

@GiteaBot why you not backport
Unbenannt

GiteaBot pushed a commit to GiteaBot/gitea that referenced this pull request Oct 29, 2023
Fixed a little mistake when you deleting user secrets via the API. Found
it when working on go-gitea#27725.
It should be backported to 1.21 I think.
@GiteaBot GiteaBot added the backport/done All backports for this PR have been created label Oct 29, 2023
lunny pushed a commit that referenced this pull request Oct 29, 2023
Backport #27829 by @jbgomond

Fixed a little mistake when you deleting user secrets via the API. Found
it when working on #27725.
It should be backported to 1.21 I think.

Co-authored-by: Jean-Baptiste Gomond <dev@jbgomond.com>
zjjhot added a commit to zjjhot/gitea that referenced this pull request Oct 30, 2023
* giteaofficial/main:
  List all Debian package versions in `Packages` (go-gitea#27786)
  Fix merge base commit for fast-forwarded GitLab PRs (go-gitea#27825)
  Fix bad method call when deleting user secrets via API (go-gitea#27829)
  Change katex limits (go-gitea#27823)
  Dockerfile small refactor (go-gitea#27757)
  Use GitLab's squash_commit_sha when available (go-gitea#27824)
  [skip ci] Updated translations via Crowdin
  Upgrade xorm to 1.3.4 (go-gitea#27807)
lunny pushed a commit that referenced this pull request Oct 31, 2023
Adds the missing user secrets API integration tests so #27829 does not
happen again
GiteaBot pushed a commit to GiteaBot/gitea that referenced this pull request Oct 31, 2023
Adds the missing user secrets API integration tests so go-gitea#27829 does not
happen again
techknowlogick pushed a commit that referenced this pull request Oct 31, 2023
Backport #27832 by @jbgomond

Adds the missing user secrets API integration tests so #27829 does not
happen again

Co-authored-by: Jean-Baptiste Gomond <dev@jbgomond.com>
fuxiaohei pushed a commit to fuxiaohei/gitea that referenced this pull request Jan 17, 2024
Fixed a little mistake when you deleting user secrets via the API. Found
it when working on go-gitea#27725.
It should be backported to 1.21 I think.
fuxiaohei pushed a commit to fuxiaohei/gitea that referenced this pull request Jan 17, 2024
Adds the missing user secrets API integration tests so go-gitea#27829 does not
happen again
@go-gitea go-gitea locked as resolved and limited conversation to collaborators Jan 27, 2024
# for free to subscribe to this conversation on GitHub. Already have an account? #.
Labels
backport/done All backports for this PR have been created backport/v1.21 This PR should be backported to Gitea 1.21 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 size/XS Denotes a PR that changes 0-9 lines, ignoring generated files. type/bug
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants