Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Removed mysqlnd stats from phpinfo page #4164

Closed
wants to merge 1 commit into from
Closed

Removed mysqlnd stats from phpinfo page #4164

wants to merge 1 commit into from

Conversation

PeeHaa
Copy link
Contributor

@PeeHaa PeeHaa commented May 15, 2019

Currently all stats from mysqlnd are being displayed on our phpinfo() page.
Do we really need to display them here making the info page unnecessary long and hard to scroll past the long mysqlnd list.

mysqlnd-stats

All information can already be retrieved using the APIs at https://www.php.net/manual/en/mysqlnd.stats.php if really needed.

Closes https://bugs.php.net/bug.php?id=60594

@petk
Copy link
Member

petk commented May 15, 2019

Yes, the phpinfo page looks a bit strange the more you look at it... Wish we could one day remake it into a more usable page/info panel... I have no idea what is useful info and what not in this particular mysqlnd stats case. I've never checked these out to be honest. If the traffic/load changes, then static information at certain time is a bit pointless information. But with realtime changes, that would be more impressive I think.

Goals:

screen

Reality:

...phpinfo()... 😐

@petk petk added the Feature label May 16, 2019
@stollr
Copy link

stollr commented May 16, 2019

Maybe adding a tabbed view to the phpinfo page, with a tab that could be used for such information. Just a thought ^^

@krakjoe
Copy link
Member

krakjoe commented Jul 3, 2019

Merged as 6ab04a6

Thanks.

@krakjoe krakjoe closed this Jul 3, 2019
@PeeHaa PeeHaa deleted the remove-mysqlnd-stats-from-phpinfo branch July 3, 2019 14:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants