Closed confused-Techie closed 7 months ago
Filling out the template is required.
All new code requires tests to ensure against regressions.
[X] Have you ran tests against this code?
[ ] This PR contains zero code changes.
After merging #199 I've discovered an error watching the logs generated by real requests.
Surprised our testing didn't catch this, but that can be investigated after getting a fix out
Requirements
Filling out the template is required.
All new code requires tests to ensure against regressions.
[X] Have you ran tests against this code?
[ ] This PR contains zero code changes.
Description of the Change
After merging #199 I've discovered an error watching the logs generated by real requests.
Surprised our testing didn't catch this, but that can be investigated after getting a fix out