Skip to content
This repository has been archived by the owner. It is now read-only.

[dev.icinga.com #1624] Command expansion feature broken #646

Closed
icinga-migration opened this Issue Jun 8, 2011 · 5 comments

Comments

Projects
None yet
1 participant
@icinga-migration
Copy link
Member

icinga-migration commented Jun 8, 2011

This issue has been migrated from Redmine: https://dev.icinga.com/issues/1624

Created by tontonitch on 2011-06-08 12:15:24 +00:00

Assignee: mfriedrich
Status: Resolved (closed on 2011-06-26 15:35:51 +00:00)
Target Version: 1.4.2
Last Update: 2014-12-08 09:42:16 +00:00 (in Redmine)

Icinga Version: 1.10.0
OS Version: any

Hi,
After upgrading from icinga 1.4.0 to 1.4.1, the command expansion became broken.
On my install:
icinga_1.4.1_command_expansion_broken_perso.png
Checked on classic.demo.icinga.org.png: same problem:
icinga_1.4.1_command_expansion_broken_classic.demo.icinga.org.png
It seems that cgi is looking for a check command containing the arguments.
Regards,
Yannick

Attachments

Changesets

2011-06-08 21:25:49 +00:00 by mfriedrich 757be9b

re-fix xss vulnerability and string escaping for command expansion #1605 #1624

refs #1605
refs #1624

2011-06-09 11:59:14 +00:00 by mfriedrich bd1ae15

re-fix xss vulnerability and string escaping for command expansion #1605 #1624

refs #1605
refs #1624
@icinga-migration

This comment has been minimized.

Copy link
Member Author

icinga-migration commented Jun 8, 2011

Updated by mfriedrich on 2011-06-08 21:23:06 +00:00

  • Status changed from New to Feedback
  • Assigned to set to mfriedrich

well. i'd appreciate more testing on the r1.4 but anyhow. this relates to #1605 introducing a hard fix, not having the right effect.

@icinga-migration

This comment has been minimized.

Copy link
Member Author

icinga-migration commented Jun 9, 2011

Updated by tontonitch on 2011-06-09 10:45:50 +00:00

Tested and working on my setup.
Thanks Michael
Regards,
Yannick

@icinga-migration

This comment has been minimized.

Copy link
Member Author

icinga-migration commented Jun 15, 2011

Updated by mfriedrich on 2011-06-15 16:49:11 +00:00

  • Target Version changed from 1.4.1 to 1.4.2
@icinga-migration

This comment has been minimized.

Copy link
Member Author

icinga-migration commented Jun 26, 2011

Updated by mfriedrich on 2011-06-26 15:35:51 +00:00

  • Status changed from Feedback to Resolved
  • Done % changed from 0 to 100
@icinga-migration

This comment has been minimized.

Copy link
Member Author

icinga-migration commented Dec 8, 2014

Updated by mfriedrich on 2014-12-08 09:42:16 +00:00

  • Project changed from 19 to Core, Classic UI, IDOUtils
  • Category changed from 52 to Classic UI
  • Icinga Version set to 1
  • OS Version set to any
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
You can’t perform that action at this time.
You signed in with another tab or window. Reload to refresh your session. You signed out in another tab or window. Reload to refresh your session.