PayloadSecurity / VxAPI

A generic interface and CLI for all endpoints of the Falcon Sandbox API
GNU General Public License v3.0
209 stars 58 forks source link

Error messages #36

Closed mcarpenter closed 5 years ago

mcarpenter commented 5 years ago

Two small problems with error messages from vxapi.py:

  1. They are written to stdout.
  2. They contain escape codes even when stdout is not a terminal.

The first can be solved by simply writing to sys.stderr instead. The second can be solved by calling sys.stdout.isatty() and acting appropriately on the result.

kkochanski commented 5 years ago

Hello, thanks for your comment. You're right about those two things - those changes will be implemented soon.

mcarpenter commented 5 years ago

Thanks! Maybe also output from --verbose? Otherwise it gets mixed in with the actual output when debugging.

kkochanski commented 5 years ago

Maybe also output from --verbose? Otherwise it gets mixed in with the actual output when debugging.

Yes, changes were applied also there :)

Resolved with 45a4897bad1af38f62fcf5b8b84226ba539cc006