search

buildkite / test-collector-python

Python adapter for Buildkite Test Analytics
https://buildkite.com/test-analytics
MIT License
6 stars 5 forks source link

Gracefully handle return characters in api token #12

Closed swebb closed 1 year ago

swebb commented 1 year ago

It's important that the collector doesn't break customer builds. If the buildkite api token includes a return character (i.e., \r or \n) the Python libraries throw an InvalidHeader exception, which would cause the process to error out. This patch gracefully handles the error so the customer's test suite doesn't fail.

jimsynz commented 1 year ago

Thanks for your contribution @swebb