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.

Sign up for GitHub

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

Jump to bottom

IcingaDB::PrepareObject(): round Notification#times.{begin,end} not to crash Go daemon #9794

Merged
merged 2 commits into from
Jun 27, 2023
Merged

IcingaDB::PrepareObject(): round Notification#times.{begin,end} not to crash Go daemon #9794

merged 2 commits into from
Jun 27, 2023

Commits on Jun 27, 2023

  1. IcingaDB::PrepareObject(): round Notification#times.{begin,end} not t… 

    …o crash Go daemon

The latter expects ints, not floats - not to mention strings.
Luckily Icinga already enforces numeric strings so that we can cast it to number.
    @Al2Klimov
    Al2Klimov committed Jun 27, 2023
    Configuration menu
    Copy the full SHA
    415b810 View commit details
    Browse the repository at this point in the history

  2. IcingaDB::PrepareObject(): cut off (null) negative Notification#times… 

    ….{begin,end} not to crash Go daemon

At least our PostgreSQL schema enforces positive values.
    @Al2Klimov
    Al2Klimov committed Jun 27, 2023
    Configuration menu
    Copy the full SHA
    dccb678 View commit details
    Browse the repository at this point in the history