Possible incorrect InfluxDB line protocol escaping in `centreon-certified/influxdb/influxdb2-metrics-apiv2.lua`

[aeblyve]

The code writes out the data in this way:

  local event = self.sc_event.event
  local tags = 'host.name=' .. event.cache.host.name .. ',poller=' .. self:escape_special_characters(event.cache.poller)

This appears to cause issues when event.cache.host.name contains spaces. From influxdb:

https://docs.influxdata.com/influxdb/v1/write_protocols/line_protocol_reference/#special-characters

In tag keys, tag values, and field keys, you must escape:

    commas
    equal signs
    spaces

I am not certain that this applies in the same way to influx v2, but adding

  local trpl = string.gsub(event.cache.host.name, " ", "\\ ")
  local tags = 'host.name=' .. trpl .. ',poller=' .. self:escape_special_characters(event.cache.poller)

Seemed like a quick fix to get the broker working again.

[tanguyvda]

hello,

thank you for your feedback. You're totally right. On our side, we used this documentation and it is the same than for v1.

I totally forgot that the space character was allowed for hosts (, and = aren't allowed in centreon so I didn't care for them). Anyway, your quick fix is fine, what we are going to do is to use the self:escape_special_characters function that was made for this purpose and escapes things the way you did (with the benefit to have parameters to better control how you'd like to escape those characters)