General: Ignore decode error of stdout/stderr in run_subprocess

[iLLiCiTiT]

Brief description

Ignore decode errors and replace invalid character (byte) with escaped byte character.

Description

Calling of run_subprocess may cause crashes if output contains some unicode character which (for example Polish name of encoder handler).

Testing notes:

Issue was discovered by a client so testing notes are related to the case.

1. Open TrayPublisher
2. Select family (e.g. Render)
3. Drop video from client to both (source and reveiw) input fields
4. Publish
5. Publishing should went smoothly without issues in ExtractReview plugin

[ynbot]

Task linked: OP-4928 Issue with extract review on Web Publisher for specific MP4s

[64qam]

Now it crashes on Extract burnins.
publish-report-230210-10-14.txt

[Minkiu]

Maybe it should do de decoding on the args and kwargs of the subprocess?

[iLLiCiTiT]

Now it crashes on Extract burnins.

Should be fixed

Maybe it should do de decoding on the args and kwargs of the subprocess?

It's not issue with args but with output from process.

[64qam]

still crashing on burnins
publish-report.txt

[iLLiCiTiT]

still crashing on burnins

Forgot tu push one commit