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

[dev.icinga.com #8006] Beautify command transport error exceptions #1261

Closed
icinga-migration opened this issue Dec 5, 2014 · 2 comments
Closed
Labels
area/monitoring Affects the monitoring module bug Something isn't working
Milestone

Comments

@icinga-migration
Copy link

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

Created by mfriedrich on 2014-12-05 10:44:03 +00:00

Assignee: (none)
Status: Resolved (closed on 2015-05-28 14:35:04 +00:00)
Target Version: 2.0.0-rc1
Last Update: 2015-05-28 14:35:04 +00:00 (in Redmine)


While it is good to see which command fails, next to the actual error ("permission") denied, it's really not required from a users perspective to see which php file triggered the exception. Errors should be made user and human readable. This might affect more errors, but this is certainly the most common one.

Auswahl_161.png

Attachments

Changesets

2015-05-28 14:13:35 +00:00 by elippmann a4a1a29

monitoring: Reduce error output when transporting a command to a local command file failed

refs #8006

2015-05-28 14:33:35 +00:00 by elippmann 9f155ea

monitoring: Don't show SSH command when transporting a command to a remote command file failed

refs #8006

2015-05-28 14:34:12 +00:00 by elippmann 068bfc0

Merge branch 'bugfix/monitoring-beautify-command-transport-errors-8006'

fixes #8006
@icinga-migration
Copy link
Author

Updated by elippmann on 2015-03-31 14:14:50 +00:00

  • Subject changed from Beautify command pipe error exception to Beautify command transport error exceptions
  • Target Version set to 2.0.0-rc1

@icinga-migration
Copy link
Author

Updated by elippmann on 2015-05-28 14:35:04 +00:00

  • Status changed from New to Resolved
  • Done % changed from 0 to 100

Applied in changeset 068bfc0.

@icinga-migration icinga-migration added bug Something isn't working area/monitoring Affects the monitoring module labels Jan 17, 2017
@icinga-migration icinga-migration added this to the 2.0.0-rc1 milestone Jan 17, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/monitoring Affects the monitoring module bug Something isn't working
Projects
None yet
Development

No branches or pull requests

1 participant