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

Fix Slices with whitespace or umlaut in data field #17

Merged
merged 1 commit into from Oct 20, 2017

Conversation

@sant-swedge
Copy link
Contributor

commented Apr 24, 2017

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

@kobmaki

This comment has been minimized.

Copy link

commented May 25, 2017

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

This comment has been minimized.

Copy link

commented Sep 5, 2017

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

@Thomas-Gelf Thomas-Gelf merged commit 556850d into Icinga:master Oct 20, 2017

@ysijason ysijason referenced this pull request Jan 27, 2017

Closed

sql error #12

@nilmerg nilmerg added this to the 1.1.0 milestone Apr 4, 2019

@nilmerg nilmerg added the bug label May 20, 2019

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
5 participants
You can’t perform that action at this time.