Ticket #5568 defect new

Opened 14 months ago

Last modified 14 months ago

_cbGetPublicKey method of SSHUserAuthClient heck of swallows exceptions

Reported by: teratorn Owned by:
Priority: normal Milestone:
Component: conch Keywords:
Cc: z3p Branch:
Author: Launchpad Bug:

Description

This code needs to log the Failure.

Change History

1

Changed 14 months ago by DefaultCC Plugin

  • cc z3p added

2

Changed 14 months ago by exarkun

  • keywords easy removed
  • owner set to teratorn

All conch tickets are not easy by default. Also, please fix the summary and description of this ticket.

3

Changed 14 months ago by teratorn

The method, if passed in a failure (it's a target of a Deferred.addBoth call), needs to log it before proceeding to fail the authentication request. I don't know how to make the summary more clear - it swallows at exception it (in my opinion) shouldn't. And I don't know how to change ticket descriptions.

Anyone that looks at the method in question would be able to understand exactly that this ticket is about.

But if you actually have some constructive criticism to give, please do so - "fix it" is completely unhelpful.

4

Changed 14 months ago by teratorn

  • owner teratorn deleted
Note: See TracTickets for help on using tickets.