git.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: Andrey Utkin <andrey.od.utkin@gmail.com>
To: Jeff King <peff@peff.net>
Cc: linux-kernel@vger.kernel.org, git@vger.kernel.org
Subject: Re: Don't use PGP/GPG signatures in mail that contains patches
Date: Tue, 19 Jan 2016 13:52:56 +0200	[thread overview]
Message-ID: <569E2398.2050206@gmail.com> (raw)
In-Reply-To: <20160118214857.GA24136@sigill.intra.peff.net>

[-- Attachment #1: Type: text/plain, Size: 2790 bytes --]

On 18.01.2016 23:48, Jeff King wrote:
> I don't know about other receiving scripts, but "git am" will handle
> signed PGP-MIME out of the box (I didn't try it with inline signatures,
> but I imagine it would stick the "BEGIN PGP MESSAGE" cruft into the
> commit message).
> 
> However, there's an open question of what to _do_ with such a signature.
> The email signature does not function as a valid git commit signature.
> So you are left with one of:
> 
>   1. The receiver can verify the origin of the email before applying the
>      patch.
> 
>   2. The receiver can keep a copy of the email "somewhere", so people
>      can later re-verify it, and then hand-verify that it matches what
>      got applied.
> 
>      That "somewhere" may just be a mailing list archive, but you could
>      get fancy with scripts and associate it with the applied commit
>      (e.g., using "git notes").
> 
> But those are really questions for the project. If you are mailing your
> patches to Linus, does he actually care about (1)? My general impression
> of his past opinion is that it's more important to read the patch text
> than the "From" line. Of course subsystem maintainers and other projects
> may have different opinions.
> 
> I think (2) is more compelling, if only to create a better record in the
> mailing list archive. Assuming the receivers of your patches don't mind
> (and I know some people really _don't_ like things like PGP-MIME,
> because their mail readers are not good at replying in-line to the
> patches then), I don't it would be a bad thing to teach git-send-email
> an option to send it.

Thank you for thoughtful reply!
Surely email submission signature cannot be used as git commit
signature. And surely there are issues of usability. And surely people
are used not to care.
But still, if we encourage signing maillist correspondence, we would
avoid impersonation attacks. Imagine that somebody sends stupid
submissions from your name, maintainers shout at you, and your
reputataion is... changed. Of course, you will be able to sort things
out after you read the replies and reply that it's not you. But, given
to openness of maillists, the attacker is able to follow your replies
and insert his ones. Or to reply to your valid submissions that they are
not from you.
Still it seems that making fun of that is not much harder than
masquerading on GitHub
(https://github.com/amoffat/masquerade/commit/9b0562595cc479ac8696110cb0a2d33f8f2b7d29)
Sure there are anti-spoofing measures like DKIM. Honestly I am not aware
if vger.kernel.org is so restrictive that it accepts only letters from
super-safe email servers, but I guess it is not, because not everybody
has this stuff configured on their email servers.


[-- Attachment #2: OpenPGP digital signature --]
[-- Type: application/pgp-signature, Size: 819 bytes --]

  reply	other threads:[~2016-01-19 11:53 UTC|newest]

Thread overview: 4+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2016-01-18  1:27 Don't use PGP/GPG signatures in mail that contains patches Andrey Utkin
2016-01-18 21:48 ` Jeff King
2016-01-19 11:52   ` Andrey Utkin [this message]
2016-01-19 21:05     ` Eric Wong

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=569E2398.2050206@gmail.com \
    --to=andrey.od.utkin@gmail.com \
    --cc=git@vger.kernel.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=peff@peff.net \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).