Icinga / icingaweb2-module-cube

Drill-down view for Icinga web 2 based on custom variables
GNU General Public License v2.0
45 stars 12 forks source link

Fix Slices with whitespace or umlaut in data field #17

Closed sant-swedge closed 6 years ago

sant-swedge commented 7 years ago

Fixes issue related to URL encoding (or in this case decoding) in cubeFromParams($params) function.

kobmaki commented 7 years ago

When will this patch be be applied? I vote for the patch as I have also problems with variables (vars) that contains whitespaces.

You can verify the problem when you modify the host template

 /etc/icinga2/conf.d/templates.conf

in the template part

 template Host "generic-host" 

with

   vars.DimTyp = type+" "+Math.round(Math.random()*4)

you get maximum five different types for a dimension.

Without the patch, you see an error in the details (link to the monitoring/host). With the patch it works as expected.

wolf-in-sheeps-clothing commented 7 years ago

Tested in my productive environment. Everything works fine. Now finally my vars with umlauten and whitespaces work. Please merge to master. Thanks.