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

check_ssl_cert: integrate new option to set maximum validity #9881

Merged

Conversation

cycloon
Copy link
Contributor

@cycloon cycloon commented Oct 18, 2023

check_ssl_cert has a new option with a new default: --maximum-validity. This change allows for services to use that option to override the default of 397 days. It is needed, if you have internal certificates that have a longer duration.

@cla-bot cla-bot bot added the cla/signed label Oct 18, 2023
Copy link
Member

@Al2Klimov Al2Klimov left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please also update the documentation.

itl/plugins-contrib.d/web.conf Outdated Show resolved Hide resolved
Copy link
Member

@Al2Klimov Al2Klimov left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please undo the master merge via:

  • git reset --hard 6fc8bb611b60673a9a2a913d08e77a6a33dced07
  • git push -f

@cycloon cycloon force-pushed the feature/check-ssl-cert-maximum-validity branch from f39e964 to 6fc8bb6 Compare January 22, 2024 09:12
@cycloon
Copy link
Contributor Author

cycloon commented Jan 22, 2024

Sorry, I just did not see the notification for this PR. I did what you suggested. Please let me know if that is still okay or if I can do anything else.

@Al2Klimov Al2Klimov assigned Al2Klimov and unassigned cycloon Jan 22, 2024
@Al2Klimov Al2Klimov removed their assignment Jan 22, 2024
Copy link
Member

@Al2Klimov Al2Klimov left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I meant the ITL documentation under doc/

itl/plugins-contrib.d/web.conf Outdated Show resolved Hide resolved
@cycloon cycloon requested a review from Al2Klimov January 24, 2024 09:54
doc/10-icinga-template-library.md Outdated Show resolved Hide resolved
Al2Klimov
Al2Klimov previously approved these changes Jan 26, 2024
Copy link
Member

@Al2Klimov Al2Klimov left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Note

Despite, technically speaking, multiple committers this PR may (and should) be squashed during merge.

@Al2Klimov Al2Klimov added the area/itl Template Library CheckCommands label Jan 26, 2024
@Al2Klimov Al2Klimov added this to the 2.15.0 milestone Jan 26, 2024
cycloon and others added 4 commits May 24, 2024 12:05
check_ssl_cert has a new option with a new default: --maximum-validity. This change allows for services to use that option to override the default of 397 days. It is needed, if you have internal certificates that have a longer duration.
Update documentation as requested
@Al2Klimov Al2Klimov force-pushed the feature/check-ssl-cert-maximum-validity branch from ae8044c to d41b3f0 Compare May 24, 2024 10:24
Copy link
Member

@Al2Klimov Al2Klimov left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Rebased it for you, but there's a lot of trailing whitespace.

itl/plugins-contrib.d/web.conf Outdated Show resolved Hide resolved
@Al2Klimov Al2Klimov merged commit 7cc8d81 into Icinga:master May 27, 2024
26 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/itl Template Library CheckCommands cla/signed
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants