oslo-incubator

  1. Bug #1343604
  2. Comment #0

Comment 0 for bug 1343604

Revision history for this message
Amrith Kumar (amrith) wrote : Exceptions thrown by execute() return a command that potentially includes passwords

Currently when execute() throws a ProcessExecutionError, it returns the command without masking passwords. In the one place where it logs the command, it correctly masks the password.

It would be prudent to mask the password in the exception as well so that upstream catchers don't have to go through the mask_password() motions.