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

IcingaHostForm: Fix crashing of form for invalid check command #2941

Merged
merged 1 commit into from
Dec 12, 2024

Conversation

raviks789
Copy link
Collaborator

fixes #2854

@cla-bot cla-bot bot added the cla/signed label Dec 10, 2024
@raviks789 raviks789 requested a review from lippserd December 10, 2024 12:27
@raviks789 raviks789 self-assigned this Dec 10, 2024
@raviks789 raviks789 added the bug label Dec 10, 2024
@raviks789 raviks789 force-pushed the fix/host-form-for-invalid-check-command branch from ac8b034 to 1d19d40 Compare December 10, 2024 12:47
@raviks789 raviks789 added this to the v1.11.3 milestone Dec 10, 2024
@lippserd lippserd force-pushed the fix/host-form-for-invalid-check-command branch from 1d19d40 to 8161a76 Compare December 12, 2024 12:11
@lippserd
Copy link
Member

I'm not entirely convinced by the solution, as now the error is shown in both the check_command and has_agent fields, but I haven't found a better solution in a quick way, so let's just merge it.

@lippserd lippserd merged commit 32a644f into master Dec 12, 2024
22 checks passed
@lippserd lippserd deleted the fix/host-form-for-invalid-check-command branch December 12, 2024 12:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Focusing out of the Check command field during the Host Object creation crashes site.
2 participants