Skip to content

Conversation

@piercefreeman
Copy link
Owner

@piercefreeman piercefreeman commented Jun 30, 2025

While testing some more complex projects we've seen this error locally:

[2025-06-30T22:06:28Z ERROR firehot::layer] Error reading from child process stdout: stream did not contain valid UTF-8

monitor_stream was using std::io::Lines to read from child process stdout/stderr streams. The Lines iterator expects all input to be valid UTF-8, but child processes can output invalid UTF-8 bytes, causing the iterator to fail with a UTF-8 decoding error.

@piercefreeman piercefreeman merged commit 573fa48 into main Jun 30, 2025
12 checks passed
@piercefreeman piercefreeman deleted the feature/fix-utf8-support branch June 30, 2025 22:58
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

Successfully merging this pull request may close these issues.

1 participant