Wikkid should be able to disable GPG signing

Bug #661975 reported by Jelmer Vernooij
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Bazaar
Confirmed
Medium
Unassigned
Wikkid Wiki
Triaged
High
Unassigned

Bug Description

I have GPG signing enabled in my bazaar.conf file, causing Bazaar to prompt for my GPG passphrase when doing a commit. This means wikkid hangs while waiting for a passphrase to be entered on the command-line.

Since this is undesirable it would be nice if wikkid could override what I have set in my configuration and do a commit without signing.

Revision history for this message
Robert Collins (lifeless) wrote : Re: [Bug 661975] [NEW] Should disable GPG signing

I'm not really sure that makes sense; various agents can prompt you
via e.g. global windows, and how can wikkid tell (or other repo
checkers tell) that a commit without a signature is permitted?

Revision history for this message
Tim Penhey (thumper) wrote :

There is an environment variable that can be set (BZR_HOME?) that should be
overridden in the start script used to do this.

Having said that, I have GPG signing enabled, and I don't get prompted.
Perhaps I've overridden the setting in my locations.conf.

Revision history for this message
Jelmer Vernooij (jelmer) wrote : Re: Should disable GPG signing

I was running wikkid on a remote machine and it just hung waiting for a passphrase on the remote machine.

I can see that disabling the signing is not necessarily the right thing to do. Perhaps wikkid should unset the signing command so signing will fail when required, instead?

Revision history for this message
Tim Penhey (thumper) wrote : Re: [Bug 661975] Re: Should disable GPG signing

On Mon, 18 Oct 2010 02:11:28 you wrote:
> I was running wikkid on a remote machine and it just hung waiting for a
> passphrase on the remote machine.
>
> I can see that disabling the signing is not necessarily the right thing
> to do. Perhaps wikkid should unset the signing command so signing will
> fail when required, instead?

Is there a way that wikkid could determine if the config is asking for signing
though bzrlib? If so we could bomb out... maybe.

Alternatively force a fail.

I'm not entirely sure which way we should go with this.

Revision history for this message
Martin Pool (mbp) wrote :

It seems plausible (if not very likely) that people would want wikkid
to sign all its commits, and if there was a key with no passphrase
that would be possible. I think the essential bug here is that if
user interaction is not possible, signing should fail (or be skipped)
rather than just hanging. bzr can't tell whether the key is encrypted
or not but it can control what fds and environment variables gpg sees.
Perhaps if bzr is noninteractive it can pass a --batch flag to gpg.

You might be able to work around this with a per-location gpg configuration?

--
Martin

Vincent Ladeuil (vila)
Changed in bzr:
status: New → Confirmed
importance: Undecided → Medium
Tim Penhey (thumper)
Changed in wikkid:
status: New → Triaged
importance: Undecided → High
Jelmer Vernooij (jelmer)
tags: added: signatures
Jonathan Riddell (jr)
summary: - Should disable GPG signing
+ Wikkid should be able to disable GPG signing
Revision history for this message
John A Meinel (jameinel) wrote :

I'd at least argue that having Wikkid disable signing for a branch that a user has configured to require signing seems a little bit wrong. I realize the global-value effect. But I'm pretty sure you can do something like:

[/path/to/wikkid/root]
create_signatures = never

And have that override your bazaar.conf setting. Is that not sufficient for this use case?

Revision history for this message
Tim Penhey (thumper) wrote :

John, I think I do that for my branch. It would be good to be able to determine if a branch requires signing. At least that way we could check on start up.

Revision history for this message
John A Meinel (jameinel) wrote : Re: [Bug 661975] Re: Wikkid should be able to disable GPG signing

-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA1

On 7/7/2011 11:58 PM, Tim Penhey wrote:
> John, I think I do that for my branch. It would be good to be able to
> determine if a branch requires signing. At least that way we could
> check on start up.
>

branch.get_config().signature_needed()

Do you need more than that?

John
=:->
-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1.4.9 (Cygwin)
Comment: Using GnuPG with Mozilla - http://enigmail.mozdev.org/

iEYEARECAAYFAk4WqzIACgkQJdeBCYSNAAMgFACfYOCUGLxhpLCArmtXJFffiNT7
nX4AnjhLQRPxfDfezcVPJmMAlQvHU+Kh
=Ebkw
-----END PGP SIGNATURE-----

Jelmer Vernooij (jelmer)
tags: added: check-for-breezy
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.