mysql-xplain-xplain icon indicating copy to clipboard operation
mysql-xplain-xplain copied to clipboard

About port

Open duckjiangwei opened this issue 3 years ago • 3 comments

If not use the default port(3306),Will report an error。

duckjiangwei avatar Aug 17 '22 06:08 duckjiangwei

@duckjiangwei I just downloaded this tool having some normal "let's try to patch up this old code base for modern times" issues.

I'm using a non-standard port as well. But I seem to be deeper down the rabbit hole. I'm on PHP 8.1.9 hitting a MySQL 8.0 server cluster.

Error I'm getting in litespeed log is:

2022-09-04 10:33:23.930423 [NOTICE] [408276] [162.158.62.218:18538:HTTP2-1#webserver.com] [STDERR] PHP Fatal error:  Declaration of Jasny\MySQL\DB::query($query) must be compatible with mysqli::query(string $query, int $result_mode = MYSQLI_STORE_RESULT): mysqli_result|bool in /var/www/webserver.com.com/public_html/xplain/vendor/jasny/db-mysql/src/Jasny/MySQL/DB.php on line 102

I hacked in and modified line 102:

#[\ReturnTypeWillChange]
public function query($query,int $result_mode = MYSQLI_STORE_RESULT): mysqli_result|bool

After that hack the error changed to:

2022-09-04 10:41:01.524645 [NOTICE] [408277] [162.158.62.218:63082:HTTP2-1#webserver.com] [STDERR] PHP Fatal error:  Class Jasny\MySQL\DB_Exception cannot extend final class mysqli_sql_exception in /var/www/webserver.com/public_html/xplain/vendor/jasny/db-mysql/src/Jasny/MySQL/DB/Exception.php on line 0

Any of these the ones you have seen?

stephenfeather avatar Sep 04 '22 14:09 stephenfeather

@stephenfeather Look at this:https://github.com/rap2hpoutre/mysql-xplain-xplain/issues/12 So, you can try the version below php7 .eg: php5.6

duckjiangwei avatar Sep 04 '22 15:09 duckjiangwei

I'll see if I can spin up something with php5 on it. Thanks for the xref to #12

This is becoming a "thing" across OSS. I have 3 3rd party Wordpress plugins in use, one is essentially abandoned, and 2 use unupdated vendor libraries that I'm repatching with every update. (yes, the old-school diff and patch is still useful!)

stephenfeather avatar Sep 04 '22 17:09 stephenfeather