summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorPieter Hordijk <info@pieterhordijk.com>2019-05-15 17:13:34 +0300
committerJoe Watkins <krakjoe@php.net>2019-07-03 08:20:54 +0200
commit6ab04a6eef0750ea2d4365be7f9bcfd9caa33449 (patch)
tree01ec4b91b99eda5d50ce4ca5d7a239b5844418b8
parenta45192e183d456b3288e466b51137e7e2a5e2cf6 (diff)
downloadphp-git-6ab04a6eef0750ea2d4365be7f9bcfd9caa33449.tar.gz
Removed mysqlnd stats from phpinfo page
All information can already be retrieved using the APIs at https://www.php.net/manual/en/mysqlnd.stats.php. Closes https://bugs.php.net/bug.php?id=60594
-rw-r--r--NEWS3
-rw-r--r--ext/mysqli/tests/mysqli_phpinfo.phpt11
-rw-r--r--ext/mysqlnd/php_mysqlnd.c27
3 files changed, 3 insertions, 38 deletions
diff --git a/NEWS b/NEWS
index 179935c8ab..47a738036c 100644
--- a/NEWS
+++ b/NEWS
@@ -12,6 +12,9 @@ PHP NEWS
- Opcache:
. Fixed #78202 (Opcache stats for cache hits are capped at 32bit NUM). (cmb)
+- mysqlnd:
+ . Fixed #60594 (mysqlnd exposes 160 lines of stats in phpinfo). (PeeHaa)
+
- PDO:
. Implemented FR #78033 (PDO - support username & password specified in
DSN). (sjon)
diff --git a/ext/mysqli/tests/mysqli_phpinfo.phpt b/ext/mysqli/tests/mysqli_phpinfo.phpt
index 02913daa69..4f856f2b65 100644
--- a/ext/mysqli/tests/mysqli_phpinfo.phpt
+++ b/ext/mysqli/tests/mysqli_phpinfo.phpt
@@ -46,17 +46,6 @@ require_once('skipifconnectfailure.inc');
if ($IS_MYSQLND) {
$expected = array(
- 'mysqlnd statistics',
- 'bytes_sent', 'bytes_received', 'packets_sent', 'packets_received',
- 'protocol_overhead_in', 'protocol_overhead_out', 'result_set_queries',
- 'non_result_set_queries', 'no_index_used', 'bad_index_used',
- 'buffered_sets', 'unbuffered_sets', 'ps_buffered_sets', 'ps_unbuffered_sets',
- 'flushed_normal_sets', 'flushed_ps_sets', 'rows_fetched_from_server',
- 'rows_fetched_from_client', 'rows_skipped', 'copy_on_write_saved',
- 'copy_on_write_performed', 'command_buffer_too_small', 'connect_success',
- 'connect_failure', 'connection_reused', 'explicit_close', 'implicit_close',
- 'disconnect_close', 'in_middle_of_command_close', 'explicit_free_result',
- 'implicit_free_result', 'explicit_stmt_close', 'implicit_stmt_close',
'size',
'mysqli.allow_local_infile',
'mysqli.allow_persistent', 'mysqli.max_persistent'
diff --git a/ext/mysqlnd/php_mysqlnd.c b/ext/mysqlnd/php_mysqlnd.c
index bf509724ca..276d8968cd 100644
--- a/ext/mysqlnd/php_mysqlnd.c
+++ b/ext/mysqlnd/php_mysqlnd.c
@@ -51,29 +51,6 @@ mysqlnd_minfo_print_hash(zval *values)
/* }}} */
-/* {{{ mysqlnd_minfo_dump_plugin_stats */
-static int
-mysqlnd_minfo_dump_plugin_stats(zval *el, void * argument)
-{
- struct st_mysqlnd_plugin_header * plugin_header = (struct st_mysqlnd_plugin_header *)Z_PTR_P(el);
- if (plugin_header->plugin_stats.values) {
- char buf[64];
- zval values;
- snprintf(buf, sizeof(buf), "%s statistics", plugin_header->plugin_name);
-
- mysqlnd_fill_stats_hash(plugin_header->plugin_stats.values, plugin_header->plugin_stats.names, &values ZEND_FILE_LINE_CC);
-
- php_info_print_table_start();
- php_info_print_table_header(2, buf, "");
- mysqlnd_minfo_print_hash(&values);
- php_info_print_table_end();
- zend_array_destroy(Z_ARR(values));
- }
- return ZEND_HASH_APPLY_KEEP;
-}
-/* }}} */
-
-
/* {{{ mysqlnd_minfo_dump_loaded_plugins */
static int
mysqlnd_minfo_dump_loaded_plugins(zval *el, void * buf)
@@ -161,10 +138,6 @@ PHP_MINFO_FUNCTION(mysqlnd)
}
php_info_print_table_end();
-
-
- /* Print client stats */
- mysqlnd_plugin_apply_with_argument(mysqlnd_minfo_dump_plugin_stats, NULL);
}
/* }}} */