icinga-powershell-framework icon indicating copy to clipboard operation
icinga-powershell-framework copied to clipboard

Automate check_by_powershell configuration for Icinga 2 and Director

Open mwaldmueller opened this issue 4 years ago • 1 comments

mwaldmueller avatar Feb 03 '21 12:02 mwaldmueller

I mean the missing of strict mode is not really bug in itself. Enabling it, at best, changes nothing, at worst, it exposes other bugs. In doubt, exposing would mean crashing the Icinga DB process. So better just do this as a whole in 1.2.0.

julianbrost avatar Aug 01 '23 13:08 julianbrost

#699 sets TRADITIONAL in addition to ANSI_QUOTES to enable strict mode, while this PR eliminates the latter by introducing driver-specific quoting. I've removed this PR from the next release and will leave it open until we decide whether to support Vitess, which would be the only reason to change quoting.

lippserd avatar Mar 18 '24 09:03 lippserd

This PR

  • introduces Quoter that provides utility functions for quoting table names and columns, where the quotation mark depends on the database driver used
  • and replaces the ANSI_QUOTES SQL mode with TRADITIONAL which enables strict mode.

Accidentally disabling strict mode was already fixed in #699

So only the other point remains which seems to only be relevant if we'd want to so support Vitess (#606).

julianbrost avatar Jul 31 '24 08:07 julianbrost

I doubt that these changes will be necessary in the near future, so I am closing this PR.

lippserd avatar Oct 02 '24 08:10 lippserd