Hyperfoil / qDup

Lab automation and queuing scripting
Apache License 2.0
12 stars 12 forks source link

unreadable log after using secret states (the ones prefixed by underscore) #182

Closed vitorz closed 1 year ago

vitorz commented 1 year ago

After the introduction of secret states for holding some passwords in a qdup file, most of the log messages are unreadable and huge. It's just a log issue as all the rest looks working to me. The log messages look in this way: ***************e********************************************************************************r**************************************

vitorz commented 1 year ago

this happens if the secret is loaded as empty string.

johnaohara commented 1 year ago

@vitorz thanks for getting to the bottom of this, I will re-open as this is an issue that renders the entire log unreadable

willr3 commented 1 year ago

It sounds like there are two issues:

  1. The SecretFilter should not add an empty string as a secret.
  2. Secret values that contain other secret values need to be checked first to prevent exposing part of the longer secret.

anything we think I missed?

vitorz commented 1 year ago

Hi, thanks for fixing. Nothing is missing for me.

willr3 commented 1 year ago

closing as we merged the fix for the secretsfilter