fix: Fix support for InfluxDB 1.8.x in InfluxQLQueryAPI

[btasker]

Proposed Changes

Switches to sending InfluxQL queries as a urlencoded form-body to ensure compatibility with 1.8.x's InfluxQL implementation.

See (#458)

Checklist

• CHANGELOG.md updated
• Rebased/mergeable
• mvn test completes successfully
• Commit messages are conventional
• Sign CLA (if not already signed)

[bednar]

Thanks for your PR 👍

Please add test for your changes:

@Test void testInfluxDB18() { // create database String db = "testing_database"; influxDBQuery("CREATE DATABASE " + db, db); // connect to InfluxDB 1.8 influxDBClient.close(); influxDBClient = InfluxDBClientFactory.createV1(getInfluxDbUrl(), "username", "password".toCharArray(), db, "autogen"); influxQLQueryApi = influxDBClient.getInfluxQLQueryApi(); // test query to InfluxDB 1.8 InfluxQLQueryResult result = influxQLQueryApi.query(new InfluxQLQuery("SHOW DATABASES", db)); assertSingleSeriesRecords(result) .map(record -> record.getValueByKey("name")) .contains(db); // drop database influxDBQuery("DROP DATABASE " + db, db); }

to

influxdb-client-java/client/src/test/java/com/influxdb/client/ITInfluxQLQueryApi.java

Line 135 in 5712b48

[codecov-commenter]

Codecov Report

Base: 88.40% // Head: 88.40% // No change to project coverage 👍

Coverage data is based on head (f748c64) compared to base (f137cd9).
Patch has no changes to coverable lines.

Additional details and impacted files

@@ Coverage Diff @@ ## master #459 +/- ## ========================================= Coverage 88.40% 88.40% Complexity 777 777 ========================================= Files 172 172 Lines 7014 7014 Branches 378 378 ========================================= Hits 6201 6201 Misses 693 693 Partials 120 120 

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

☔ View full report at Codecov.
📢 Do you have feedback about the report comment? Let us know in this issue.

[bednar]

LGTM 🚀