Error - Unexpected end of stream

[jimmcq]

With a fresh install, I tried to up RetrievePokemonCountExample.php (after updating the auth info) and get the error below:

PHP Fatal error: Uncaught exception 'Exception' with message 'read_bytes(): Unexpected end of stream' in vendor/nicklasw/pogoprotos-php/src/protocolbuffers.inc.php:364

This happens with many of the other example scripts as well.

[NicklasWallgren]

@DrDelay It's due the StreamWrapper. hmm, do you have any thought about it?

[DrDelay]

I can't reproduce it at f70c9ce . I tried PlayerProfile and PokemonCount example, both worked.

[jimmcq]

I'm using PTC auth, if that matters... Other than that, it's unmodified fresh code (git clone, composer install) at f70c9ce

[NicklasWallgren]

Weird, I could reproduce the problem. Reverted the pull request until we find out what's causing the problem.

@jmcquillan Try the latest commit.

[jimmcq]

e8d58cc works for me!

[DrDelay]

What is your guys environment?
I got PHP 7.0.9 NTS

[NicklasWallgren]

Tried both PHP 7.0.3-13 ( NTS ) and PHP 7.0.9-1 ( NTS ). Same issue.

What OS are you using?

[DrDelay]

Windows 7, but I just tried it on a Debian Jessie with a PHP 7.0.9-1~dotdeb+8.1 (cli) ( NTS ) setup, worked aswell 😆

[NicklasWallgren]

I'm running Ubuntu 14.04. I love these sort of problems... :D

[DrDelay]

The condition for the Exception is if (strlen($bytes) !== $len), where $bytes = fread($fp, $len), this text seems a bit misleading. I'm only thinking out loud, can this be character encoding related or something? Like, regional setting of the machine?

[NicklasWallgren]

I was thinking the same, it's probably related to encoding. It usually is.

[DrDelay]

• mbstring is not relevant (tried dis-/enable)
• Windows is German, en_US:en on Debian
• curl 7.49.1 on Windows, curl 7.38.0 on Debian

[jimmcq]

en_US PHP 5.6.24 on Mac OS 10.11.6 with curl 7.43.0

[NicklasWallgren]

The length of the string returned from stream_get_contents differs between the different streams.

94 (ProtobufIO::toStream)
139-141 (Streamwrapper::getResource)

[DrDelay]

Regarding the mbstring that like I hastily said doesn't matter, I found a comment that says one should use mb_strlen($data, '8bit') to get the amount of bytes read (as strlen may be overloaded by the mbstring-ext, using mb_internal_encoding()). I cannot test whether that would fix it, but I can say that it still works with this in my environment. Can someone of you try it out?

[DrDelay]

If so and this means we'd have to force every user to use mbstring-ext, it may be reasonable to just pass it as a string as this works. I'd still like to figure out the reason of this error.

[NicklasWallgren]

Nice find, mb_strlen($data, '8bit') did unfortunately no difference.

[DrDelay]

I'm kinda out of ideas then.

I added this code to your unmarshall-method:

        // Unmarshall the response

        echo mb_internal_encoding(), PHP_EOL;
        echo mb_http_output(), PHP_EOL;
        echo mb_http_input(), PHP_EOL;
        echo mb_regex_encoding(), PHP_EOL;
        $body = $response->getBody();
        echo 'strlen(body->getContents): ', strlen($body->getContents()), PHP_EOL;
        $body->seek(0);
        echo 'body->getSize: ', $body->getSize(), PHP_EOL;
        $body->seek(0);
        $wrap = \GuzzleHttp\Psr7\StreamWrapper::getResource($body);
        echo 'strlen(streamwrapped): ', strlen(stream_get_contents($wrap)), PHP_EOL;
        $body->seek(0);
        echo 'strlen(toStream): ', strlen(stream_get_contents(\ProtobufIO::toStream($response->getBody()->getContents()))), PHP_EOL;
        die();

That results in:

UTF-8
UTF-8

UTF-8
strlen(body->getContents): 141
body->getSize: 141
strlen(streamwrapped): 141
strlen(toStream): 141

I'm assuming you get different sizes. Maybe you could run that and see what encodings you get.

[NicklasWallgren]

Hmm, the problem disappeared after re-installing the dependencies. It could be a bug connected to Guzzle.

Sorry for the late reply, been kind busy today.

[finalpk]

I got the same error with one of my accounts, however it works with my other account

I get this on the working account:

UTF-8
UTF-8
UTF-8 

strlen(body->getContents): 140 
body->getSize: 140
strlen(streamwrapped): 140
strlen(toStream): 140

and this on the one thats not working:

UTF-8
UTF-8
UTF-8

strlen(body->getContents): 141
body->getSize: 141
strlen(streamwrapped): 141
strlen(toStream): 141

[NicklasWallgren]

@finalpk Can you reproduce the problem in the latest version?