Skip to content

feat: support x509 commit signing - #2844

Merged
extrawurst merged 6 commits into
gitui-org:masterfrom
kaden-l-nelson:feat/x509-signing-method
Jul 1, 2026
Merged

extrawurst merged 6 commits into
gitui-org:masterfrom
kaden-l-nelson:feat/x509-signing-method

Conversation

@kaden-l-nelson

Copy link
Copy Markdown
Contributor

This Pull Request fixes/closes #2514.

It changes the following:

  • Adds support for commit signing with the x509 format

I followed the checklist:

  • I added unittests
  • I ran make check without errors
  • I tested the overall application
  • I added an appropriate item to the changelog

@kaden-l-nelson
kaden-l-nelson force-pushed the feat/x509-signing-method branch 3 times, most recently from 35ef709 to 8e226dd Compare January 23, 2026 16:08
@kaden-l-nelson

Copy link
Copy Markdown
Contributor Author

@extrawurst may you review this PR?

@kaden-l-nelson
kaden-l-nelson force-pushed the feat/x509-signing-method branch from 8e226dd to 35537d5 Compare March 2, 2026 19:55
@kaden-l-nelson
kaden-l-nelson force-pushed the feat/x509-signing-method branch from 35537d5 to 6d14f1a Compare March 31, 2026 21:36
@kaden-l-nelson
kaden-l-nelson force-pushed the feat/x509-signing-method branch from 6d14f1a to 4acae06 Compare June 23, 2026 00:29

@extrawurst extrawurst left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

lgtm just a few comments

Comment thread CHANGELOG.md Outdated
Comment thread asyncgit/src/sync/sign.rs
@extrawurst extrawurst self-assigned this Jul 1, 2026
@extrawurst
extrawurst merged commit 7a397b0 into gitui-org:master Jul 1, 2026
22 checks passed
@kaden-l-nelson
kaden-l-nelson deleted the feat/x509-signing-method branch August 6, 2026 15:15
jianshanbushishan added a commit to jianshanbushishan/gitui that referenced this pull request Sep 7, 2026
Valuable upstream changes taken:
- SSH commit signing via ssh-agent and external sign binaries (gitui-org#2464)
- x509 commit signing (gitui-org#2844)
- GPG signing for non-HEAD reword commits (gitui-org#2959)
- fix: index-out-of-bounds panic in staging apply_selection (gitui-org#2955)
- fix: panic when opening submodule (gitui-org#2896) - send failure now logs
  instead of panicking, adapted to local StatusChanged/Unchanged events
- branch names: spaces auto-convert to dashes (gitui-org#2916)
- ratatui 0.30 / crossterm 0.29 / ratatui-textarea migration
  (ratatui-image bumped 9 -> 11 to match)
- insta snapshot test infra (main.rs split into gitui.rs), snapshots
  regenerated for the local top-bar layout

Kept local versions where they supersede upstream:
- git2 0.21 migration and gix 0.86 (upstream at 0.84)
- editor-at-cursor-line in diff view (superset of upstream gitui-org#2904)
- mailmap cache + commit-hash search in commit_filter
- unchanged-notification redraw skipping ported into Gitui main loop
- Input created once in main and reused across submodule reopens

Extra merge fixes:
- drop .as_ref() in Layout::constraints calls: palette (via
  ratatui-image -> icy_sixel -> quantette) adds AsRef impls for arrays
  that make .as_ref() ambiguous against ratatui 0.30's generic
  IntoIterator parameter
- ratatui pinned to 0.30.0 stack (ratatui-core 0.1.0) to match
  upstream's tested dependency set
- image fallback test relaxed: ratatui-image 11 blends tiny upscaled
  images, exact source pixel color no longer appears in cells
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Signing commits with x509 method

2 participants