tradefed: hide auth token and save error to TestJob.failure

Linaro / squadplugins

GNU Affero General Public License v3.0

1 stars 6 forks source link

tradefed: hide auth token and save error to TestJob.failure #34

Closed chaws closed 3 years ago

chaws commented 3 years ago

Error messages contain the auth-token, which is bad practice to leave it in clear text. Also this saves the error message to the TestJob object and only warns in logs.

chaws commented 3 years ago

@mwasilew what do you think of it? My intention is to stop receiving Sentry notifications when Lava is faulty. Mainly because most of the time it seems to be temporary.

mwasilew commented 3 years ago

Looks good.