Skip to content

Add git commit/tag ssh signature verification#2077

Merged
stefanprodan merged 1 commit into
fluxcd:mainfrom
bb-Ricardo:rb-feature/adds-git-commit-ssh-signature-verification
Jun 12, 2026
Merged

Add git commit/tag ssh signature verification#2077
stefanprodan merged 1 commit into
fluxcd:mainfrom
bb-Ricardo:rb-feature/adds-git-commit-ssh-signature-verification

Conversation

@bb-Ricardo

Copy link
Copy Markdown
Contributor

This PR adds git commit and tag SSH signature validation.

Ref: #1996

Comment thread config/crd/bases/source.toolkit.fluxcd.io_buckets.yaml Outdated
@bb-Ricardo bb-Ricardo force-pushed the rb-feature/adds-git-commit-ssh-signature-verification branch from c5a36db to 4624aab Compare June 9, 2026 08:07

var keyRings []string
for _, v := range secret.Data {
keyRings = append(keyRings, string(v))

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The current implementation does not implement what is stated in the documentation: https://fluxcd.io/flux/components/source/gitrepositories/#verification-secret-example

Any key (not just .asc) is used for PGP public keys. Therefore a fallback is implemented to not break current setups

@stefanprodan stefanprodan changed the title FR: adds git commit/tag ssh signature verification Add git commit/tag ssh signature verification Jun 9, 2026
@stefanprodan stefanprodan added area/git Git related issues and pull requests area/security Security related issues and pull requests labels Jun 9, 2026
@stefanprodan

Copy link
Copy Markdown
Member

@bb-Ricardo please rebase

Signed-off-by: Ricardo Bartels <ricardo.bartels@telekom.de>
@bb-Ricardo bb-Ricardo force-pushed the rb-feature/adds-git-commit-ssh-signature-verification branch from 4624aab to f565efe Compare June 9, 2026 13:28

@matheuscscp matheuscscp left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM! 🚀

@stefanprodan stefanprodan merged commit 12dba7c into fluxcd:main Jun 12, 2026
6 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area/git Git related issues and pull requests area/security Security related issues and pull requests

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants