Don't suggest the user specify -Dhpss

Bug #241133 reported by Mark Hammond
4
Affects Status Importance Assigned to Milestone
Bazaar
Fix Released
Medium
Martin Pool

Bug Description

While trying to push to launchpad, something went wrong, and part of the message generated by bzr was:

No supported authentication methods left to try!
bzr: ERROR: Connection closed: please check connectivity and permissions (and try -Dhpss if further
diagnosis is required)

Its not clear to a casual user what '-Dhpss' does - and indeed, after specifying it, the behaviour doesn't seem to change at all - you still get the identical output, including a suggestion to add that switch (even though it already is). That flag did cause more output to be written to the log, but that isn't clear (and the additional output generated is no use for a user to diagnose this)

Martin suggested I open a bug on this issue: we should never inform the user to try debugging flags as a matter of course - it has the potential to confuse them, and without telling them what do do *after* they have specified the option they will remain lost.

The message comes from:
  File "D:\src\bazaar\bzr.dev\bzrlib\smart\message.py", line 258, in read_response_tuple
    self._wait_for_response_args()
  File "D:\src\bazaar\bzr.dev\bzrlib\smart\message.py", line 224, in _wait_for_response_args
    self._read_more()
  File "D:\src\bazaar\bzr.dev\bzrlib\smart\message.py", line 247, in _read_more
    "(and try -Dhpss if further diagnosis is required)")
ConnectionReset: Connection closed: please check connectivity and permissions (and try -Dhpss if further diagnosis is required)

Tags: trivial ui
James Westby (james-w)
Changed in bzr:
importance: Undecided → Medium
status: New → Confirmed
Revision history for this message
MaheshK (maheshkulkarni) wrote :

I do get the same kind of error, it seems like something has gone wrong while installation

Revision history for this message
Neil Martinsen-Burrell (nmb) wrote :

This is fixed as of r4416. It should be marked closed.

Revision history for this message
Matt Nordhoff (mnordhoff) wrote :

Yeah, you're right. It was fixed in 1.13.

This is Launchpad; you could've marked it as fixed yourself. :)

Changed in bzr:
assignee: nobody → Martin Pool (mbp)
status: Confirmed → Fix Released
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.