unofficial mirror of notmuch@notmuchmail.org
 help / color / mirror / code / Atom feed
From: David Edmondson <dme@dme.org>
To: David Bremner <david@tethera.net>,
	Jameson Graef Rollins <jrollins@finestructure.net>,
	Vagrant Cascadian <vagrant@debian.org>,
	755544@bugs.debian.org
Cc: notmuch@notmuchmail.org
Subject: Re: Bug#755544: notmuch-emacs: doesn't check gpg/pgp signatures by default
Date: Mon, 08 Feb 2016 17:46:42 +0000	[thread overview]
Message-ID: <m2oabr3xj1.fsf@heart-of-gold.hh.sledj.net> (raw)
In-Reply-To: <878unlenhn.fsf@maritornes.cs.unb.ca>

On Tue, Jul 22 2014, David Bremner wrote:
> Jameson Graef Rollins <jrollins@finestructure.net> writes:
>
>> On Mon, Jul 21 2014, David Bremner <david@tethera.net> wrote:
>>> notmuch folks: it seems that in vagrant's message, and several others I
>>> checked, it notmuch-crypto-process-mime==nil, then no signature button
>>> is created at all.
>>
>> Yes, this is true.  The signature button is pretty meaningless if we're
>> not processing the signature.
>>
>> Maybe instead by default we could have a signature button that opens up
>> a notmuch-crypto-process-mime customization buffer?
>>
>> jamie.
>
> looking at the source, there is supposed to be some button:

There is a button for the part (the one that says "[multipart/signed]")
and the help text is associated with that. There is no button
specifically for the signature (because no processing of the signature
took place).

It wouldn't be complicated to add a button in the case where no checking
was done. Pressing the button could (as suggested) offer up
customisation of the variable (or just open the help).

I wonder if I could make the button be "[Danger Will Robinson!!!]" in
flashing red and yellow text...

> ,----
> | (defun notmuch-show-insert-part-multipart/signed (msg part content-type nth depth button)
> |   (button-put button 'face 'notmuch-crypto-part-header)
> |   ;; add signature status button if sigstatus provided
> |   (if (plist-member part :sigstatus)
> |       (let* ((from (notmuch-show-get-header :From msg))
> | 	     (sigstatus (car (plist-get part :sigstatus))))
> | 	(notmuch-crypto-insert-sigstatus-button sigstatus from))
> |     ;; if we're not adding sigstatus, tell the user how they can get it
> |     (button-put button 'help-echo "Set notmuch-crypto-process-mime to process cryptographic MIME parts."))
> `----

  reply	other threads:[~2016-02-08 17:46 UTC|newest]

Thread overview: 7+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
     [not found] <20140721223426.GA5250@siren>
2014-07-21 23:16 ` Bug#755544: notmuch-emacs: doesn't check gpg/pgp signatures by default David Bremner
2014-07-22  1:03   ` Jameson Graef Rollins
2014-07-22  4:30     ` Daniel Kahn Gillmor
2014-07-22  4:39       ` Daniel Kahn Gillmor
2014-07-22  9:44     ` David Bremner
2016-02-08 17:46       ` David Edmondson [this message]
2017-07-16 12:45   ` David Bremner

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

  List information: https://notmuchmail.org/

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

  git send-email \
    --in-reply-to=m2oabr3xj1.fsf@heart-of-gold.hh.sledj.net \
    --to=dme@dme.org \
    --cc=755544@bugs.debian.org \
    --cc=david@tethera.net \
    --cc=jrollins@finestructure.net \
    --cc=notmuch@notmuchmail.org \
    --cc=vagrant@debian.org \
    /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.
Code repositories for project(s) associated with this public inbox

	https://yhetil.org/notmuch.git/

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 read-only IMAP folder(s) and NNTP newsgroup(s).