DBALException bin2hex() expects parameter 1 to be string, resource given #5796

New Issue

Closed

opened 2026-01-22 15:18:11 +01:00 by admin · 7 comments

admin commented 2026-01-22 15:18:11 +01:00

Owner

Copy Link

Originally created by @holantomas on GitHub (Dec 5, 2017).

Originally assigned to: @Ocramius on GitHub.

Hi,
I have entity FileContent with few properities id,bytes. bytes is blob. When I'm saving file stream from upload to DB it try to throw exeption bellow but it fall on param formating when it try to call bin2hex on stream resource (uploaded file).

Trying to throw exception
SQLSTATE[08S01]: Communication link failure: 1153 Got a packet bigger than 'max_allowed_packet' bytes"

File: .../doctrine/dbal/lib/Doctrine/DBAL/DBALException.php:197

187:         *
188:         * @return string
189:         */
190:        private static function formatParameters(array $params)
191:        {
192:            return '[' . implode(', ', array_map(function ($param) {
193:                $json = @json_encode($param);
194:    
195:                if (! is_string($json) || $json == 'null' && is_string($param)) {
196:                    // JSON encoding failed, this is not a UTF-8 string.
197: // THIS LINE -->   return '"\x' . implode('\x', str_split(bin2hex($param), 2)) . '"';
198:                }
199:    
200:                return $json;
201:            }, $params)) . ']';

Don't know what else to report, so you can ask what you need.
And yes I know how to solve first exception.

Originally created by @holantomas on GitHub (Dec 5, 2017). Originally assigned to: @Ocramius on GitHub. Hi, I have entity `FileContent` with few properities `id`,`bytes`. `bytes` is `blob`. When I'm saving file stream from upload to DB it try to throw exeption bellow but it fall on param formating when it try to call bin2hex on stream resource (uploaded file). Trying to throw exception `SQLSTATE[08S01]: Communication link failure: 1153 Got a packet bigger than 'max_allowed_packet' bytes"` ``` File: .../doctrine/dbal/lib/Doctrine/DBAL/DBALException.php:197 187: * 188: * @return string 189: */ 190: private static function formatParameters(array $params) 191: { 192: return '[' . implode(', ', array_map(function ($param) { 193: $json = @json_encode($param); 194: 195: if (! is_string($json) || $json == 'null' && is_string($param)) { 196: // JSON encoding failed, this is not a UTF-8 string. 197: // THIS LINE --> return '"\x' . implode('\x', str_split(bin2hex($param), 2)) . '"'; 198: } 199: 200: return $json; 201: }, $params)) . ']'; ``` Don't know what else to report, so you can ask what you need. And yes I know how to solve first exception.

admin added the BugDuplicate labels 2026-01-22 15:18:11 +01:00

admin closed this issue 2026-01-22 15:18:12 +01:00

admin commented 2026-01-22 15:18:12 +01:00

Author

Owner

Copy Link

@Ocramius commented on GitHub (Dec 5, 2017):

Could you maybe add a test case that reproduces the issue, so we can prevent regressions on a fix?

@Ocramius commented on GitHub (Dec 5, 2017): Could you maybe add a test case that reproduces the issue, so we can prevent regressions on a fix?

admin commented 2026-01-22 15:18:12 +01:00

Author

Owner

Copy Link

@holantomas commented on GitHub (Dec 5, 2017):

Error is depend on your mysql settings of max_allowed_packet. By default it's set to 16M. I tried to upload PHP stream resource with 20MB file.

Database definition ofr example.

CREATE TABLE `file_content` (
  `id` int(11) NOT NULL AUTO_INCREMENT,
  `bytes` longblob NOT NULL,
  PRIMARY KEY (`id`)
) ENGINE=InnoDB DEFAULT CHARSET=utf8 COLLATE=utf8_unicode_ci;

This throws what I'm talking about. Sorry don't know if you need something better.
tmpfile() is usage just for create stream resource which cause app fall

$pdoex = new \PDOException('SQLSTATE[08S01]: Communication link failure: 1153 Got a packet bigger than \'max_allowed_packet\' bytes');
$pdoex->errorInfo = ['08S01', 1153, 'Got a packet bigger than \'max_allowed_packet\' bytes'];
$ex = new \Doctrine\DBAL\Driver\PDOException($pdoex);

\Doctrine\DBAL\DBALException::driverExceptionDuringQuery(
			new \Doctrine\DBAL\Driver\PDOMySql\Driver(),
			$ex,
			"INSERT INTO file_content (`bytes`) VALUES (?)",
			[1 => tmpfile()]);

@holantomas commented on GitHub (Dec 5, 2017): Error is depend on your mysql settings of `max_allowed_packet`. By default it's set to 16M. I tried to upload PHP stream resource with 20MB file. Database definition ofr example. ```sql CREATE TABLE `file_content` ( `id` int(11) NOT NULL AUTO_INCREMENT, `bytes` longblob NOT NULL, PRIMARY KEY (`id`) ) ENGINE=InnoDB DEFAULT CHARSET=utf8 COLLATE=utf8_unicode_ci; ``` This throws what I'm talking about. Sorry don't know if you need something better. `tmpfile()` is usage just for create stream resource which cause app fall ```php $pdoex = new \PDOException('SQLSTATE[08S01]: Communication link failure: 1153 Got a packet bigger than \'max_allowed_packet\' bytes'); $pdoex->errorInfo = ['08S01', 1153, 'Got a packet bigger than \'max_allowed_packet\' bytes']; $ex = new \Doctrine\DBAL\Driver\PDOException($pdoex); \Doctrine\DBAL\DBALException::driverExceptionDuringQuery( new \Doctrine\DBAL\Driver\PDOMySql\Driver(), $ex, "INSERT INTO file_content (`bytes`) VALUES (?)", [1 => tmpfile()]); ```

admin commented 2026-01-22 15:18:13 +01:00

Author

Owner

Copy Link

@Ocramius commented on GitHub (Dec 5, 2017):

Right, and a test case would:

1. str_repeat('a', $db->select('SELECT setting FROM settings_table')->fetchColumn(0) * 1024 * 1024);
2. write it to a temporary resource (php://temp)
3. perform an insert on that table and cause an exception to be thrown

Marco Pivetta

http://twitter.com/Ocramius

http://ocramius.github.com/

On Tue, Dec 5, 2017 at 9:07 AM, Tomáš Holan notifications@github.com
wrote:

Error is depend on your mysql settings of max_allowed_packet. By default
it's set to 16M. I tried to upload PHP stream resource with 20MB file.

Database definition ofr example.

CREATE TABLE file_content (
id int(11) NOT NULL AUTO_INCREMENT,
bytes longblob NOT NULL,
PRIMARY KEY (id)
) ENGINE=InnoDB DEFAULT CHARSET=utf8 COLLATE=utf8_unicode_ci;

This throws what I'm talking about. Sorry don't know if you need something
better.
tmpfile() is usage just for create stream resource which cause app fall

$pdoex = new \PDOException('SQLSTATE[08S01]: Communication link failure: 1153 Got a packet bigger than 'max_allowed_packet' bytes');$pdoex->errorInfo = ['08S01', 1153, 'Got a packet bigger than 'max_allowed_packet' bytes'];$ex = new \Doctrine\DBAL\Driver\PDOException($pdoex);\Doctrine\DBAL\DBALException::driverExceptionDuringQuery( new \Doctrine\DBAL\Driver\PDOMySql\Driver(), $ex, "INSERT INTO file_content (bytes) VALUES (?)", [1 => tmpfile()]);

—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/doctrine/doctrine2/issues/6864#issuecomment-349227089,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAJakJKKJFgYKLTn3RSt3OKuQMCttRNeks5s9PnFgaJpZM4Q1x_R
.

@Ocramius commented on GitHub (Dec 5, 2017): Right, and a test case would: 1. `str_repeat('a', $db->select('SELECT setting FROM settings_table')->fetchColumn(0) * 1024 * 1024);` 2. write it to a temporary resource (`php://temp`) 3. perform an insert on that table and cause an exception to be thrown Marco Pivetta http://twitter.com/Ocramius http://ocramius.github.com/ On Tue, Dec 5, 2017 at 9:07 AM, Tomáš Holan <notifications@github.com> wrote: > Error is depend on your mysql settings of max_allowed_packet. By default > it's set to 16M. I tried to upload PHP stream resource with 20MB file. > > Database definition ofr example. > > CREATE TABLE `file_content` ( > `id` int(11) NOT NULL AUTO_INCREMENT, > `bytes` longblob NOT NULL, > PRIMARY KEY (`id`) > ) ENGINE=InnoDB DEFAULT CHARSET=utf8 COLLATE=utf8_unicode_ci; > > This throws what I'm talking about. Sorry don't know if you need something > better. > tmpfile() is usage just for create stream resource which cause app fall > > $pdoex = new \PDOException('SQLSTATE[08S01]: Communication link failure: 1153 Got a packet bigger than \'max_allowed_packet\' bytes');$pdoex->errorInfo = ['08S01', 1153, 'Got a packet bigger than \'max_allowed_packet\' bytes'];$ex = new \Doctrine\DBAL\Driver\PDOException($pdoex);\Doctrine\DBAL\DBALException::driverExceptionDuringQuery( new \Doctrine\DBAL\Driver\PDOMySql\Driver(), $ex, "INSERT INTO file_content (`bytes`) VALUES (?)", [1 => tmpfile()]); > > — > You are receiving this because you commented. > Reply to this email directly, view it on GitHub > <https://github.com/doctrine/doctrine2/issues/6864#issuecomment-349227089>, > or mute the thread > <https://github.com/notifications/unsubscribe-auth/AAJakJKKJFgYKLTn3RSt3OKuQMCttRNeks5s9PnFgaJpZM4Q1x_R> > . >

admin commented 2026-01-22 15:18:13 +01:00

Author

Owner

Copy Link

@holantomas commented on GitHub (Dec 5, 2017):

I really don't know what you want. So I found that this issue is DBAL problem so I will close this after your answer and recreate it in right repository. But for clean I did test case commit to Doctrine/DBAL. Is'nt that better?

9f1c15815b

@holantomas commented on GitHub (Dec 5, 2017): I really don't know what you want. So I found that this issue is DBAL problem so I will close this after your answer and recreate it in right repository. But for clean I did test case commit to Doctrine/DBAL. Is'nt that better? https://github.com/doctrine/dbal/commit/9f1c15815bec44bc198ce4aabe02680e1368249d

admin commented 2026-01-22 15:18:14 +01:00

Author

Owner

Copy Link

@holantomas commented on GitHub (Dec 5, 2017):

I'm ready to make PR for this commit.

Or better this can be fixed by editing Doctrine\DBAL\DBALException::formatParameters() to something like this:

private static function formatParameters(array $params)
    {
        return '[' . implode(', ', array_map(function ($param) {
            if (is_resource($param)) {
                return (string) $param;
            }

            $json = @json_encode($param);

            if (! is_string($json) || $json == 'null' && is_string($param)) {
                // JSON encoding failed, this is not a UTF-8 string.
                return '"\x' . implode('\x', str_split(bin2hex($param), 2)) . '"';
            }

            return $json;
        }, $params)) . ']';
    }

@holantomas commented on GitHub (Dec 5, 2017): I'm ready to make PR for this commit. Or better this can be fixed by editing `Doctrine\DBAL\DBALException::formatParameters()` to something like this: ```php private static function formatParameters(array $params) { return '[' . implode(', ', array_map(function ($param) { if (is_resource($param)) { return (string) $param; } $json = @json_encode($param); if (! is_string($json) || $json == 'null' && is_string($param)) { // JSON encoding failed, this is not a UTF-8 string. return '"\x' . implode('\x', str_split(bin2hex($param), 2)) . '"'; } return $json; }, $params)) . ']'; } ```

admin commented 2026-01-22 15:18:14 +01:00

Author

Owner

Copy Link

@Ocramius commented on GitHub (Dec 5, 2017):

@holantomas that test looks good, but indeed it is on the DBAL project. Open a PR and we'll close this one :-)

@Ocramius commented on GitHub (Dec 5, 2017): @holantomas that test looks good, but indeed it is on the DBAL project. Open a PR and we'll close this one :-)

admin commented 2026-01-22 15:18:14 +01:00

Author

Owner

Copy Link

@holantomas commented on GitHub (Dec 5, 2017):

https://github.com/doctrine/dbal/pull/2933

@holantomas commented on GitHub (Dec 5, 2017): https://github.com/doctrine/dbal/pull/2933

admin referenced this issue 2026-01-22 16:05:14 +01:00

[PR #5796] Catch `Throwable` instead of just catching `Exception` #9730

Sign in to join this conversation.

No Branch/Tag Specified

Branches Tags

3.6.x

2.20.x

dependabot/github_actions/2.20.x/doctrine/dot-github/dot-github/workflows/composer-lint.yml-14.0.0

3.7.x

4.0.x

2.21.x

old-prototype-3.x

3.6.2

3.6.1

3.6.0

2.20.9

3.5.8

3.5.7

3.5.6

3.5.5

2.20.8

3.5.4

3.5.3

2.20.7

3.5.2

2.20.6

3.5.1

3.5.0

3.4.4

3.4.3

3.4.2

2.20.5

3.4.1

3.4.0

3.3.4

2.20.4

3.3.3

2.20.3

3.3.2

2.20.2

3.3.1

2.20.1

3.3.0

3.2.3

2.20.0

2.19.8

3.2.2

2.19.7

3.2.1

2.19.6

3.2.0

3.1.4

3.1.3

2.19.5

3.1.2

2.19.4

3.1.1

2.19.3

2.19.2

2.19.1

3.1.0

2.19.0

3.0.3

2.18.3

3.0.2

2.18.2

3.0.1

2.18.1

3.0.0

2.18.0

2.17.5

3.0.0-RC1

2.17.4

2.17.3

2.17.2

2.17.1

3.0.0-beta2

2.17.0

2.16.3

3.0.0-beta1

2.16.2

2.16.1

2.16.0

2.15.5

2.15.4

2.15.3

2.15.2

2.15.1

2.15.0

2.14.3

2.14.2

2.14.1

2.14.0

2.13.5

2.13.4

2.13.3

2.13.2

2.13.1

2.13.0

2.12.4

2.12.3

2.12.2

2.12.1

2.12.0

2.11.3

2.11.2

2.11.1

2.11.0

2.10.5

2.10.4

2.10.3

2.10.2

2.10.1

2.10.0

2.9.6

2.9.5

2.9.4

2.9.3

2.9.2

2.9.1

2.9.0

2.8.5

2.8.4

2.8.3

2.8.2

2.8.1

2.8.0

2.7.5

2.7.4

v2.7.3

v2.7.2

v2.7.1

v2.7.0

v2.6.6

v2.6.5

v2.6.4

v2.6.3

v2.6.2

v2.6.1

v2.6.0

v2.5.14

v2.5.13

v2.5.12

v2.5.11

v2.5.10

v2.5.9

v2.5.8

v2.5.7

v2.5.6

v2.5.5

v2.5.4

v2.5.3

v2.5.2

v2.4.8

v2.5.1

v2.5.0

v2.5.0-RC2

v2.5.0-RC1

v2.5.0-beta1

v2.5.0-alpha2

v2.5.0-alpha1

v2.4.7

v2.4.6

v2.4.5

v2.4.4

v2.4.3

v2.3.6

v2.4.2

2.3.5

v2.4.1

v2.4.0

2.4.0-RC2

2.4.0-RC1

2.4.0-BETA2

2.3.4

2.3.3

2.4.0-BETA1

2.3.2

2.3.1

2.3.0

2.3.0-RC4

2.3.0-RC3

2.3.0-RC2

2.2.3

2.3.0-RC1

2.3.0-BETA1

2.1.7

2.2.2

2.2.1

2.1.6

2.2.0

2.2.0-RC1

2.2.0-BETA2

2.2.0-BETA1

2.1.5

2.1.4

2.1.3

2.1.2

2.1.1

2.0.7

2.1.0

2.1.0RC3

2.1.0RC2

2.1.0RC1

2.0.6

2.1.0BETA1

2.0.5

2.0.4

2.0.3

2.0.2

2.0.1

2.0.0

2.0.0RC2

2.0.0RC1

2.0.0-BETA4

2.0.0-BETA3

2.0.0-BETA2

2.0.0-BETA1

Labels

Clear labels

BC Break

Bug

Can't Fix

Deprecation

Documentation

DQL

Duplicate

Failing Test

Hacktoberfest

Improvement

Improvement

Incomplete

Invalid

Missing Tests

New Feature

PR Created

pull-request

Mirrored from GitHub Pull Request

Question

Regression

Status: Waiting feedback

Waiting feedback

WIP

Won't Fix

No Label Bug Duplicate

Milestone

No items

No Milestone

Projects

Clear projects

No project

Assignees

Clear assignees

admin

No Assignees

1 Participants

Notifications

Subscribe

Due Date

No due date set.

Dependencies

No dependencies set.

Reference: doctrine/archived-orm#5796