bzr email commit hook

Feature Request: Allow more than one attempt to enter SMTP password

Bug #121146 reported by Stuart Colville
2
Affects Status Importance Assigned to Milestone
Bazaar
Confirmed
Wishlist
Unassigned
bzr email commit hook
Fix Released
Low
Stuart Colville
Ubuntu
Invalid
Undecided
Unassigned

Bug Description

You only get one chance to enter your password for the SMTP server. If the password is incorrect (using the branch from bug #120537 which fixes the missing errors import) the email sending aborts with an error message. It would be great if this behaviour could be changed so you get at least 3 shots at entering the password before the email message is aborted.

Note: This depends on https://code.launchpad.net/~jelmer/bzr-email/120537

See original description
Revision history for this message
Juan Pablo Salazar Bertín (snifer) wrote :

Thanks for your suggestion. However, the changes you are requesting aren't really a bug and require more discussion, which should be done on an appropriate mailing list or forum. <http://www.ubuntu.com/support/community/mailinglists> might be a good start.

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

(Juan's message is saying that this isn't a bug against Ubuntu in general; it is a useful report for Bazaar.)

I agree we should do this. Maybe it should be common to the password prompt logic.

Changed in bzr-email:
importance: Undecided → Low
status: Unconfirmed → Confirmed
Vincent Ladeuil (vila)
Changed in bzr:
assignee: nobody → v-ladeuil
importance: Undecided → Wishlist
status: New → Confirmed
assignee: v-ladeuil → nobody
Revision history for this message
Stuart Colville (muffinresearch) wrote :

Add comment. I've fixed this I just need to register the branch with the fix. Should have a chance to do this in the next few days.

Stuart Colville (muffinresearch)
description: updated
Stuart Colville (muffinresearch)
Changed in bzr-email:
assignee: nobody → muffinresearch
status: Confirmed → Fix Committed
Revision history for this message
James Westby (james-w) wrote :

Hi,

Is this really a bug in bzr? Was it just in bzr-email?

Thanks,

James

Revision history for this message
Stuart Colville (muffinresearch) wrote :

Just bzr-email

Revision history for this message
John A Meinel (jameinel) wrote :

There is also bzrlib/smtp_connection.py, which I thought we wanted bzr-email to use if it was available. (I could have sworn there was a patch for this, did it just get forgotten?)

So I would say it is a bug in both.

Revision history for this message
James Westby (james-w) wrote :

Thanks, closing the bzr part of the bug then.

Thanks,

James

Changed in bzr:
status: Confirmed → Invalid
James Westby (james-w)
Changed in bzr:
status: Invalid → Confirmed
Jelmer Vernooij (jelmer)
Changed in bzr-email:
status: Fix Committed → Fix Released
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.
Subscribing...

Other bug subscribers

Remote bug watches

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