Mysql Connections Count is not accurate #198

Saifallak · 2023-10-25T03:23:56Z

Saifallak
Oct 25, 2023

as you can see here:

laravel-health/src/Support/DbConnectionInfo.php

Line 15 in 1dee638

    
           $connection instanceof MySqlConnection => (int) $connection->selectOne('show status where variable_name = "threads_connected"')->Value,

we're using show status where variable_name = "threads_connected" which shows overall server processes not my user-only
this is wrong when running several apps on the same server and setting max_user_connections to like 60 per app.
it should warn me when my user exceeds the limit not the overall server.

so it should be changed to SELECT COUNT(*) FROM information_schema.PROCESSLIST;

this won't be breaking change for others, as if you're the only user on the server or using root mysql, this will return same data,
but if user is shared with other apps or other users (ex: Shared Hosting) it will return his processes only,
so he can be notified when his app is doing something wrong.

tell me what you think and i can open PR.