New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
fix(platform/gitlab): Strip unicode null characters from markdown #19664
Conversation
Attempting to post a comment with null characters in them fails with a 400 error. See renovatebot#19532 for more details.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Something is wrong with your solution
Head branch was pushed to by a user without write access
Co-authored-by: Michael Kriese <michael.kriese@visualon.de>
The mocks appear not to be isolated properly between tests, and having a test that mocks this before a test that does not causes the latter to fail.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
conflicts
🎉 This PR is included in version 34.98.2 🎉 The release is available on:
Your semantic-release bot 📦🚀 |
Attempting to post a comment with null characters in them fails with a 400 error. See #19532 for more details.
Changes
Strip unicode null characters from GitLab markdown, as their presence prevents comments from being posted. bundler appears to generate some error messages that include them.
Context
#19532
Documentation (please check one with an [x])
How I've tested my work (please select one)
I have verified these changes via: