-
Notifications
You must be signed in to change notification settings - Fork 5
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
Error for single quote in alias #55
Comments
No work-around for that one I'm afraid. I'll need to add code to make it a url encoded field. |
I've got the code in place. It works but just need to test border cases. |
Hey, thanks for the quick answer! |
Aha :) Hi!
Also, nrcq has been updated so it will work too:
Jessie .deb's are at sourceforge but I've not tried them so backup your configuration using the GUI before upgrading. I tested Centos 6 RPMS, which seemed to work. Cheers! |
ok got it in my build/test routine and everything is working as expected on nagrestconf side! :) |
Yep, that's what the fix was, an extra url encoding. Thanks for testing. |
Perfect then, i'll do what needed on my app side, |
Hey there, |
Coded the modification part Quickly: working perfectly! |
Thanks for that Woulf. I'll open this again and fix it for show/hosts and delete/hosts. Even though having the alias field for show and delete is unnecessary, I'd still like to be able to reuse an add or modify line and only have to change the 'add' to 'delete' or 'show'. |
Hello,
as the title say even tough it is accepted by nagios, and valid in JSON string, i can't use single quote ( ' ) in alias (big problem for french).
I tried escaping it with the "'" form and with the "\u0027" forms with the same result.
I'm always getting an http 400 error so i guess that's on the nagrestconf webserver side that the problem lay?
Any idea how to solve that?
thanks
The text was updated successfully, but these errors were encountered: