Homec4science

Don't leave temporary files around when trying to use credentials with…

Authored by epriestley <git@epriestley.com> on Aug 21 2014, 20:26.

Description

Don't leave temporary files around when trying to use credentials with destroyed secrets

Summary: Ref T4284. This fixes at least one problem which can cause the observed behavior.

Test Plan:

  • Before applying patch, used PHABRICATOR_CREDENTIAL=PHID-CDTL-... bin/ssh-connect + debugging prints to verify the keyfile was written and cleaned up normally.
  • Destroyed the credental, verified the temporary file was not cleand up correctly.
  • Applied patch, verified temporary file was not written and command exited with sensible error.

Reviewers: btrahan

Reviewed By: btrahan

Subscribers: epriestley

Maniphest Tasks: T4284

Differential Revision: https://secure.phabricator.com/D10328

Details

Committed
epriestley <git@epriestley.com>Aug 21 2014, 20:26
Pushed
aubortJan 31 2017, 17:16
Parents
rPH7d31ea7c55ea: Minor tweaks to `bin/mail send-test`
Branches
Unknown
Tags
Unknown

Event Timeline

epriestley <git@epriestley.com> committed rPH241cfc2e8312: Don't leave temporary files around when trying to use credentials with… (authored by epriestley <git@epriestley.com>).Aug 21 2014, 20:26