PointToLineProtocol should output also additionaly set tags

[pxlfrk]

Proposal:
I'm using the InfluxDB-Client Library on a ESP8266 in addition with an Telegram-Bot to interact with the device. By sending a request-message to the telegram-bot /values, it should pass the sensor-values connected to the esp to the bot.

I have four different data points in my configuration with each 2 tag sets. There isn't any (non-deprecated) way to create a String of a data point including all tag sets right now.

Current behavior:
The data point class supports the method toLineProtocol and includes the optionally added tags - that's my desired behaviour - but it's deprecated :/

InfluxDB-Client-for-Arduino/src/Point.h

Lines 71 to 72 in 44f4b11

	// Creates line protocol with optionally added tags
	String toLineProtocol(String includeTags = "") const __attribute__ ((deprecated("Use InfluxDBClient::pointToLineProtocol()")));

The (regarding to the author) prefered method in the client-class only adds the default-tags, not the optionall added tags for each data point.

InfluxDB-Client-for-Arduino/src/InfluxDbClient.cpp

Lines 538 to 540 in 5b51bdb

	String InfluxDBClient::pointToLineProtocol(const Point& point) {
	return point.createLineProtocol(_writeOptions._defaultTags);
	}

Desired behavior:
Include the optionally configured tag sets.

Alternatives considered:
Describe other solutions or features you considered.

Use case:
Serial Output or (in my case) identify different dara points when forwarding their pointToLineProtocol-String to a Bot.

Kind regards,
Achim / pxlfrk

[pxlfrk]

Any updates?

[vlastahajek]

@pxlfrk, do I understand correctly, you are creating a line protocol with custom tags, that are not sent to an InfluxDB server via this library?
This a use case I didn't expect. :)

[pxlfrk]

@vlastahajek Yes and No. I am creating a line protocol string for a number of data points which i hav assigned custom tags. These Custom tags aren't added to the line string. i am using the created line string for logging purposes.

Would be great if they are added of if there is another option to achieve the same result. Hope you get my point!

[vlastahajek]

PR #157 removed deprecation of String Point::toLineProtocol(String includeTags = "")