Skip to content

Conversation

@lunny
Copy link
Member

@lunny lunny commented Dec 12, 2025

Per-repository signing keys have never been officially supported, as they would require users to modify the repository’s config file.

At this point, it is clear that only global signing keys (GPG or SSH) should be allowed. If we want to introduce per-repository signing keys in the future, it will require a complete design proposal.

The endpoint will not be removed for repository special signing key, but it will reference the global signing key.

@GiteaBot GiteaBot added the lgtm/need 2 This PR needs two approvals by maintainers to be considered for merging. label Dec 12, 2025
@github-actions github-actions bot added modifies/api This PR adds API routes or modifies them modifies/go Pull requests that update Go code labels Dec 12, 2025
@lunny lunny marked this pull request as ready for review December 12, 2025 19:03
@lunny lunny added the type/refactoring Existing code has been cleaned up. There should be no new functionality. label Dec 13, 2025
@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 13, 2025
@wxiaoguang wxiaoguang changed the title Remove unreleased support of signing key in the repository git configuration file Remove undocumented support of signing key in the repository git configuration file Dec 14, 2025
@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 14, 2025
lunny and others added 2 commits December 14, 2025 09:01
Co-authored-by: delvh <dev.lh@web.de> Signed-off-by: Lunny Xiao <xiaolunwen@gmail.com>
@lunny lunny added the reviewed/wait-merge This pull request is part of the merge queue. It will be merged soon. label Dec 14, 2025
@lunny lunny enabled auto-merge (squash) December 14, 2025 17:03
@lunny lunny merged commit 26602fd into go-gitea:main Dec 14, 2025
23 checks passed
@GiteaBot GiteaBot added this to the 1.26.0 milestone Dec 14, 2025
@lunny lunny deleted the lunny/remove_unsupported_signing branch December 14, 2025 17:51
@GiteaBot GiteaBot removed the reviewed/wait-merge This pull request is part of the merge queue. It will be merged soon. label Dec 14, 2025
zjjhot added a commit to zjjhot/gitea that referenced this pull request Dec 15, 2025
* giteaofficial/main: Remove undocumented support of signing key in the repository git configuration file (go-gitea#36143) Enable gocheckcompilerdirectives linter (go-gitea#36156) Fix code highlighting on blame page (go-gitea#36157) Check user visibility when redirecting to a renamed user (go-gitea#36148) Fix bug when viewing the commit diff page with non-ANSI files (go-gitea#36149) Refactor `FileTreeItem` type (go-gitea#36137) Fix various bugs (go-gitea#36139) Fix issue close timeline icon (go-gitea#36138) Add permissions to`files-changed` jobs (go-gitea#36142) Add explicit permissions to all actions workflows (go-gitea#36140) Bump `actions/checkout` to v6 (go-gitea#36136) Hide RSS icon when viewing a file not under a branch (go-gitea#36135) Fix SVG size calulation, only use `style` attribute (go-gitea#36133) Add sorting/filtering to admin user search API endpoint (go-gitea#36112)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

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 modifies/go Pull requests that update Go code type/refactoring Existing code has been cleaned up. There should be no new functionality.

4 participants