feof(): supplied resource is not a valid stream resource about byte-stream HOT 8 CLOSED

I usually prefer to do an if (!is_resource($res) || feof($res)) check when testing for liveness...

from byte-stream.

I'm curious why it happened, it never happened before. And it's not 100% reproducible.

from byte-stream.

It's really strange. When I put the following code in the line before, it never prints anything, but the warning is still triggered.

if (!\is_resource($stream)) {
    \fwrite(\STDERR, "\n" . $this->trace . "\n\n");
}

from byte-stream.

Question: is $data === '' && !feof($res) ever true? AFAIK, an empty string returned from fread/stream_get_contents should always indicate eof by definiton for layer 4 streams. I'm not sure about tls streams though (as these have overhead not delivered to the reader).

from byte-stream.

By the way, I suspect the issue is related to tls streams and won't ever happen with raw layer 4 streams. Am I correct about this?

from byte-stream.

I'm pretty sure it happens with TLS streams. I'm not sure whether the warning is because of TLS streams. Might also be because of the new process library.

from byte-stream.

Ignore what I just said about the process library. As there's no new version of amphp/file / amphp/parallel yet, that isn't even installed by Artax, yet.

from byte-stream.

This is caused by amphp/parallel when pthreads is enabled.

#0 /home/kelunik/GitHub/amphp/artax/vendor/amphp/parallel/lib/Sync/ChannelledSocket.php:22 Amp\ByteStream\ResourceInputStream->__construct()
#1 /home/kelunik/GitHub/amphp/artax/vendor/amphp/parallel/lib/Threading/Internal/Thread.php:88 Amp\Parallel\Sync\ChannelledSocket->__construct()
#2 /home/kelunik/GitHub/amphp/artax/vendor/amphp/amp/lib/Loop/Driver.php:112 Amp\Parallel\Threading\Internal\Thread->Amp\Parallel\Threading\Internal\{closure}()
#3 /home/kelunik/GitHub/amphp/artax/vendor/amphp/amp/lib/Loop/Driver.php:70 Amp\Loop\Driver->tick()
#4 /home/kelunik/GitHub/amphp/artax/vendor/amphp/amp/lib/Loop.php:76 Amp\Loop\Driver->run()
#5 /home/kelunik/GitHub/amphp/artax/vendor/amphp/parallel/lib/Threading/Internal/Thread.php:99 Amp\Loop::run()
#6 Amp\Parallel\Threading\Internal\Thread->run()

from byte-stream.