Could not open input file: should be sent to stderr
#11141
Labels
Comments
|
Looks like |
|
Switching to STDERR should be fine. U fortunately, the error messages seem to be quite inconsistent with where they are printed. |
nielsdos
added a commit
to nielsdos/php-src
that referenced
this issue
Apr 29, 2023
I grepped for php_printf cases in main/ and sapi/ and converted the cases which clearly indicate errors to fprintf(stderr, ...), like suggested in the linked issue.
nielsdos
added a commit
to nielsdos/php-src
that referenced
this issue
Apr 29, 2023
I grepped for php_printf cases in main/ and sapi/ and converted the cases which clearly indicate errors to fprintf(stderr, ...), like suggested in the linked issue.
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Description
Repro command:
php non-existent.phpCurrently the output (
Could not open input file: non-existent.php) is sent to stdout but it should be sent to stderr instead.I have discovered when using php for interprocess communication where the stdout might be consumed as regular data causing protocol failure hard to understand, as in my case, only the first 4 bytes were consumed (and from
Coulit was hard to identify the problem). Simply said, stdout should never contain any other data then data printed by a valid script.The text was updated successfully, but these errors were encountered: