Opened 11 years ago

Closed 8 years ago

#2715 defect closed fixed (fixed)

SSHFactory shouldn't disable core dumps

Reported by: Jean-Paul Calderone Owned by:
Priority: normal Milestone:
Component: conch Keywords:
Cc: jesstess Branch:
Author:

Description

If core dumps should be disabled, it is someone else's responsibility to disable them.

Change History (9)

comment:1 Changed 8 years ago by jesstess

Author: jesstess
Branch: branches/sshfactory-coredumps-2715

(In [29299]) Branching to 'sshfactory-coredumps-2715'

comment:1 Changed 8 years ago by jesstess

Owner: changed from z3p to jesstess

comment:3 Changed 8 years ago by jesstess

(In [29300]) SSHFactory no longer disables coredumps.

refs #2715

comment:4 Changed 8 years ago by jesstess

Cc: jesstess added
Keywords: review added
Owner: jesstess deleted

comment:5 Changed 8 years ago by Jean-Paul Calderone

Keywords: review removed
Owner: set to jesstess
  • We should have a ticket for re-adding this functionality, perhaps to twisted.conch.tap (perhaps enabled by default, with a flag to disable it).
  • The news fragment should use the fully-qualified name of SSHFactory.

Aside from that, great. Please merge when the above two points are addressed.

comment:6 Changed 8 years ago by jesstess

(In [29393]) Use fully-qualified name of SSHFactory in the news file.

refs #2715

comment:7 Changed 8 years ago by jesstess

Ticket 4521 has been opened to re-introduce this functionality.

comment:8 Changed 8 years ago by jesstess

Resolution: fixed
Status: newclosed

(In [29394]) Merge sshfactory-coredumps-2715

Author: jesstess Reviewer: exarkun Fixes: #2715

twisted.conch.ssh.factory.SSHFactory no longer disables coredumps. Ticket #4521 was opened to re-introduce this functionality in twisted.conch.tap.

comment:9 Changed 7 years ago by <automation>

Owner: jesstess deleted
Note: See TracTickets for help on using tickets.