Skip to content
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

Config values can contain null characters resulting to panic #692

Closed
manifest opened this issue Jun 2, 2024 · 3 comments
Closed

Config values can contain null characters resulting to panic #692

manifest opened this issue Jun 2, 2024 · 3 comments

Comments

@manifest
Copy link
Contributor

manifest commented Jun 2, 2024

Config values can contain null characters. They need to be trimmed.

This has already been addressed here, but it wasn't completely resolved.

Here is PR with solution.

@manifest
Copy link
Contributor Author

manifest commented Jun 2, 2024

Here is another PR by KowalczykBartek addressing the issue in the same way.

@Guyzeroth
Copy link

@davidblewett Would it be possible to get one of these solutions merged? What should be the process here?

@manifest
Copy link
Contributor Author

manifest commented Aug 9, 2024

Fixed

@manifest manifest closed this as completed Aug 9, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants