Illegal character encoding specified about php-console HOT 7 CLOSED

Hi Alex

Can you please send me value of your ini_get('mbstring.internal_encoding')?
And it will be great if you attach call backtrace of this error.

Before I fixed this bug, you can try some workaround issue, try to set $phpConsoleConnector->setServerEncoding('utf'-8');.

Thank you!

from php-console.

I've fixed the issue. Please review my pull request #4

from php-console.

For your record the issue was being caused by setPassword() calling on self::CLIENT_ENCODING which by default was set to utf-8 instead of UTF-8

from php-console.

I don't think that this issue must work.

1. mb_convert_encoding() does not care about encoding name case
2. See source code of \PhpConsole\Connector::setServerEncoding() method

Once again:

Can you please send me value of your ini_get('mbstring.internal_encoding')?
And it will be great if you attach call backtrace of this error.

from php-console.

I thought so too, but it seems to have fixed the issue. I no longer get that error.

ini_get('mbstring.internal_encoding') result = ""
ini_get('mbstring') result = false

So it's ignored and instead the value of self::CLIENT_ENCODING is assumed.

Hmm... I can see what your doing with setServerEncoding(), however:

\PhpConsole\Connector::setPassword() line 311 calls This->convertEncoding passing in $toEncoding as self::CLIENT_ENCODING. This ends up passing in the raw uppercased value i manually set. setServerEncoding() on the other side only sets $this->serverEncoding to the lowercased representation. Hence convertEncoding() line 362 calls @mb_convert_encoding("password, 'UTF-8', 'utf-8') and that seems to make it happy. If you pass it a lowercased value as the second parameter it complains about encoding.

I cant find any reference that the encoding type MUST be uppercase so it is indeed odd.

Here is a backtrace from right after that call on line 367

http://pastebin.com/raw.php?i=wwAf2kb3

from php-console.

I have made some fix with trivial refactoring to using UTF-8 everywhere. See 8fe019b

So I think we can close this issue.
Alex, thanks for bug report & thanks for help!

from php-console.

Your welcome! 👍

from php-console.