Fox-Alpha / nagioschecker

Automatically exported from code.google.com/p/nagioschecker
0 stars 0 forks source link

Backslash in service url is missing #120

Closed GoogleCodeExporter closed 9 years ago

GoogleCodeExporter commented 9 years ago
What steps will reproduce the problem?

1. take a service with backslash in the name, like "C:\ Drive Space"
2. get this service in warning or critical state to show up in nagioschecker
3. select this service in nagioschecker

What is the expected output? What do you see instead?

the browser should show the state page of this service, instead there is a
nagios error message shown.

reason is a wrong URL. Instead of (for example)

http://.../nagios/cgi-bin/extinfo.cgi?type=2&host=SRDEAS042&service=C%3A\+Drive+
Space

following URL is produced by nagioschecker

http://.../nagios/cgi-bin/extinfo.cgi?type=2&host=SRDEAS042&service=C%3A%20Drive
%20Space

What version of the product are you using (Nagios Checker and Nagios)? On
what operating system?

nagioschecker 0.14.3 on Windows XP, nagios 3.1.0 on linux debian etch

Original issue reported on code.google.com by christ...@vieser.de on 27 Apr 2009 at 8:23

GoogleCodeExporter commented 9 years ago
Thank you for your report. Could you please test fixed version I attached there 
if it
works?
Thank you very much.

Original comment by petr.si...@gmail.com on 27 Apr 2009 at 1:10

Attachments:

GoogleCodeExporter commented 9 years ago
I'm sorry, the backslash isn't there.

0.14.3 :           ...service=C%3A%20Drive%20Space
0.14.3.escapefix : ...service=C%3A+Drive Space

it should be one of these:
...service=C%3A\%20Drive%20Space    or
...service=C%3A\+Drive+Space        or
...service=C%3A%5c%20Drive%20Space  or
...service=C%3A%5c+Drive+Space      

Original comment by christ...@vieser.de on 27 Apr 2009 at 1:59

GoogleCodeExporter commented 9 years ago
I am sorry I mistook myself while reading your report.
Could you please test the new fix?
Thank you.

Original comment by petr.si...@gmail.com on 27 Apr 2009 at 9:50

Attachments:

GoogleCodeExporter commented 9 years ago
It works!!!

Thank you very much for the quick response. Well done!

Original comment by christ...@vieser.de on 28 Apr 2009 at 7:14

GoogleCodeExporter commented 9 years ago
Thank you for testing.
Fix will be included in just released 0.14.4 public version.

Original comment by petr.si...@gmail.com on 28 Apr 2009 at 11:37