View Issue Details

IDProjectCategoryView StatusLast Update
0025446mantisbtuipublic2019-02-09 15:11
ReportercproensaAssigned Toatrol 
PrioritynormalSeverityminorReproducibilityhave not tried
Status assignedResolutionopen 
Product Version2.20.0 
Target Version2.20.0Fixed in Version 
Summary0025446: 'show_queries_count' is a global setting, but 'show_memory_usage', 'show_timer' are not

Seems like show_queries_count should have the same setting status than the other options: show_memory_usage, show_timer as they all affect the same behaviour in showing some info on the page.
Additionally, it's good to be able to set them individually, for example, only to some administrator users.
So i think show_queries_count could be taken outside of the global-only settings without harm.


TagsNo tags attached.




2019-02-07 03:15

developer   ~0061431




2019-02-08 22:17

manager   ~0061458

I believe config options that are related to instrumentation and diagnostics should be required to be global, since they can expose information about the server, overwrite a critical file with a log file, etc. These should be controlled via IT admin / hoster via config_inc.php and not by MantisBT administrator. Of course, if both personas are the same person, then it is not an issue.



2019-02-09 15:11

developer   ~0061468

In general, I prefer config_get_global for such kind of settings, because of better performance and security.

As mentioned by @cproensa there is no reason to prevent setting show_queries_count in database as long as we allow settings like show_memory_usage, show_timer to be set in database.
I created the PR to harmonize the settings.

I think we should merge the PR and start the general discussion in #25460 (IMO something that could just be changed in 3.x)

Issue History

Date Modified Username Field Change
2019-02-06 11:40 cproensa New Issue
2019-02-07 03:09 atrol Assigned To => atrol
2019-02-07 03:09 atrol Status new => assigned
2019-02-07 03:09 atrol Target Version => 2.20.0
2019-02-07 03:15 atrol Note Added: 0061431
2019-02-08 22:17 vboctor Note Added: 0061458
2019-02-09 15:11 atrol Note Added: 0061468