Skip to content
New issue

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

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

Already on GitHub? Sign in to your account

[new feature] Sign commit with GPG #4238

Closed
3 tasks
gpongelli opened this issue Dec 15, 2017 · 6 comments
Closed
3 tasks

[new feature] Sign commit with GPG #4238

gpongelli opened this issue Dec 15, 2017 · 6 comments

Comments

@gpongelli
Copy link
Contributor

gpongelli commented Dec 15, 2017

Do you want to request a feature or report a bug?
feature

What is the current behavior?
GE cannot sign commit.

What is the expected behavior?
GE can sign commit.
Git has to be at least 1.7.9

From Git commit hel page:

-S[<keyid>]
--gpg-sign[=<keyid>]

GPG-sign commits. The keyid argument is optional and defaults to the committer identity; if specified, it must be stuck to the option without a space.

This means that git commit allows to:

  1. Make an unsigned commit (actual behavior)
  2. Make a signed commit with a default key ("-S" option)
  3. Make a signed commit with a key different from default one ("-S <keyid>" option)

List of steps:

  • UI change discussion to handle the three cases above

  • "git commit" command changes

  • "git commit" command tests

@gpongelli
Copy link
Contributor Author

First idea on UI change: change the commit button similar to “fetch/pull” of main dialog.
With automatic “remember last choice”.

What do you think?

@pmiossec
Copy link
Member

First idea on UI change: change the commit button similar to “fetch/pull” of main dialog.
With automatic “remember last choice”.
What do you think?

We could also add the option in the already existing options menu in the commit form that already contains options that are passed to the commit command...

@RussKie RussKie added this to the Next version milestone Dec 17, 2017
@gpongelli
Copy link
Contributor Author

@pmiossec Are you talking about the right menu here, near the commit message field ?

I don’t know if your idea can be user-friendly, if a user want to sign every commit do he have to use the Options menu each time?
How can he switch from the three available state of the commit command?
The main drawback is that those option (no sign, default key, specific key) are mutually exclusive, not like a flag on a menu item... more similar to a radio box choice .

@RussKie have you any idea? Do you see any GUI implication on those proposal?

@RussKie
Copy link
Member

RussKie commented Dec 17, 2017 via email

@pmiossec
Copy link
Member

@gpongelli Yes I'm talking about this one. I don't say it's the good way to do it, I just say it's a possibility to think about since the "Sign-off" feature is already there and works the same way.

I mean:

  • By default, "disabled".
  • Could be enabled (and take the commiter identity by default)
  • there is a textbox to override the default value

@gpongelli
Copy link
Contributor Author

Sorry but I’ve never used “Sign-off” feature, so don’t know how it works and how it was implemented in GE.
It can be a way to sign commit, let’s hear also other proposal ;)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants