influxdb-client 1.17.0.pre.3024 → 2.0.0.pre.3196

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: d3c00b85066bf6f6d4043e66fd9747633b2ad20e92ea5e9683e00d33ca77a035
-  data.tar.gz: 2d109a3a326aab34d512cc8a93565e15b35ea7958af021492afce78f63362019
+  metadata.gz: 33f286d2a281bd72623ee0284de96e20aebfc433a39642478c43f31d0258ee50
+  data.tar.gz: 98d9e3ddc4d6e892daa5bfeba7601af15e4c4da0440ad1ea8fa0d7740e38e21a
 SHA512:
-  metadata.gz: 1a94053a10547a696297576832a79986e38804d5acd77db0a46821689f4cd10ec6da33b1ab8c7152ce560608283722af4e235cdb5a8f695ba8baeb4da49421e7
-  data.tar.gz: 0a303636ce48ffe795a4369c80fbf3f6c87fc7786186fc0cbdb5ec5d228a3766ee3ebf7e941e77f16445d56307765f39d9258c0643e92dcdea7249865382bb88
+  metadata.gz: f2f655f21aa5f143abc469b653a90bc9ca381371ad4c60d2bf35fc7019e0e1b6f8ce3d4403f30acaff03a134b0d31bc42c7001dc893dd5db9ad7d08b705dbfbc
+  data.tar.gz: 95f5b211418be1dd31067dd7f266cbb2a57dc8736e66147908ae6c5edf48036ed021ae299915b04c3d19783d1962f487623f01881bceb27dba08f8d60696170e

data/CHANGELOG.md

@@ -1,4 +1,19 @@
-## 1.17.0 [unreleased]
+## 2.0.0 [unreleased]
+
+### Breaking Changes
+Due to a security reason `Authorization` header is not forwarded when redirect leads to a different domain.
+To overcome this limitation you have to set the client property `redirect_forward_authorization` to `true`.
+
+### Features
+1. [#89](https://github.com/influxdata/influxdb-client-ruby/pull/89): `Authorization` header is not forwarded when redirect leads to a different domain
+
+### Bug Fixes
+1. [#89](https://github.com/influxdata/influxdb-client-ruby/pull/89): Correct redirect location
+ 
+## 1.17.0 [2021-08-20]
+
+### Bug Fixes
+1. [#87](https://github.com/influxdata/influxdb-client-ruby/pull/87): Parsing infinite numbers
 
 ## 1.16.0 [2021-07-09]
 

data/README.md

@@ -23,6 +23,7 @@ This repository contains the reference Ruby client for the InfluxDB 2.0.
     - [Management API](#management-api)
 - [Advanced Usage](#advanced-usage)
     - [Default Tags](#default-tags)
+    - [Proxy configuration](#proxy-configuration)
 - [Contributing](#contributing)
 - [License](#license)
 
@@ -63,13 +64,13 @@ The client can be installed manually or with bundler.
 To install the client gem manually:
 
 ```
-gem install influxdb-client -v 1.16.0
+gem install influxdb-client -v 1.17.0
 ```
 
 For management API:
 
 ```
-gem install influxdb-client-apis -v 1.16.0
+gem install influxdb-client-apis -v 1.17.0
 ```
 
 ## Usage
@@ -93,6 +94,7 @@ client = InfluxDB2::Client.new('https://localhost:8086', 'my-token')
 | write_timeout | Number of seconds to wait for one block of data to be written | Integer | 10 |
 | read_timeout | Number of seconds to wait for one block of data to be read | Integer | 10 |
 | max_redirect_count | Maximal number of followed HTTP redirects | Integer | 10 |
+| redirect_forward_authorization | Pass Authorization header to different domain during HTTP redirect. | bool | false |
 | use_ssl | Turn on/off SSL for HTTP communication | bool | true |
 | verify_mode | Sets the flags for the certification verification at beginning of SSL/TLS session. | `OpenSSL::SSL::VERIFY_NONE` or `OpenSSL::SSL::VERIFY_PEER` | none |
 
@@ -400,6 +402,20 @@ client.close!
 
 Server availability can be checked using the `client.health` method. That is equivalent of the [influx ping](https://v2.docs.influxdata.com/v2.0/reference/cli/influx/ping/).
 
+### Proxy configuration
+
+You can configure the client to tunnel requests through an HTTP proxy. To configure the proxy use a `http_proxy` environment variable. 
+
+```ruby
+ENV['HTTP_PROXY'] = 'http://my-user:my-password@my-proxy:8099'
+```
+
+Client automatically follows HTTP redirects. The default redirect policy is to follow up to 10 consecutive requests.
+You can configure redirect counts by the client property: `max_redirect_count`. 
+
+Due to a security reason `Authorization` header is not forwarded when redirect leads to a different domain. 
+To overcome this limitation you have to set the client property `redirect_forward_authorization` to `true`.
+
 ### InfluxDB 1.8 API compatibility
 
 [InfluxDB 1.8.0 introduced forward compatibility APIs](https://docs.influxdata.com/influxdb/v1.8/tools/api/#influxdb-2-0-api-compatibility-endpoints) for InfluxDB 2.0. This allow you to easily move from InfluxDB 1.x to InfluxDB 2.0 Cloud or open source.

data/lib/influxdb2/client/client.rb

@@ -42,6 +42,8 @@ module InfluxDB2
     # @option options [Integer] :write_timeout Number of seconds to wait for one block of data to be written
     # @option options [Integer] :read_timeout Number of seconds to wait for one block of data to be read
     # @option options [Integer] :max_redirect_count Maximal number of followed HTTP redirects
+    # @option options [bool] :redirect_forward_authorization Pass Authorization header to different domain
+    #   during HTTP redirect.
     # @option options [bool] :use_ssl Turn on/off SSL for HTTP communication
     # @option options [Integer] :verify_mode Sets the flags for the certification verification
     #   at beginning of SSL/TLS session. Could be one of `OpenSSL::SSL::VERIFY_NONE` or `OpenSSL::SSL::VERIFY_PEER`.

data/lib/influxdb2/client/default_api.rb

@@ -73,21 +73,24 @@ module InfluxDB2
       _post(payload, uri, headers: headers.merge(HEADER_CONTENT_TYPE => 'text/plain'))
     end
 
-    def _post(payload, uri, limit: @max_redirect_count, headers: {})
-      _request(payload, uri, limit: limit, headers: headers, request: Net::HTTP::Post)
+    def _post(payload, uri, limit: @max_redirect_count, add_authorization: true, headers: {})
+      _request(payload, uri, limit: limit, add_authorization: add_authorization,
+                             headers: headers, request: Net::HTTP::Post)
     end
 
-    def _get(uri, limit: @max_redirect_count, headers: {})
-      _request(nil, uri, limit: limit, headers: headers.merge('Accept' => 'application/json'), request: Net::HTTP::Get)
+    def _get(uri, limit: @max_redirect_count, add_authorization: true, headers: {})
+      _request(nil, uri, limit: limit, add_authorization: add_authorization,
+                         headers: headers.merge('Accept' => 'application/json'), request: Net::HTTP::Get)
     end
 
-    def _request(payload, uri, limit: @max_redirect_count, headers: {}, request: Net::HTTP::Post)
+    def _request(payload, uri, limit: @max_redirect_count, add_authorization: true, headers: {},
+                 request: Net::HTTP::Post)
       raise InfluxError.from_message("Too many HTTP redirects. Exceeded limit: #{@max_redirect_count}") if limit.zero?
 
       http = _prepare_http_client(uri)
 
       request = request.new(uri.request_uri)
-      request['Authorization'] = "Token #{@options[:token]}"
+      request['Authorization'] = "Token #{@options[:token]}" if add_authorization
       request['User-Agent'] = "influxdb-client-ruby/#{InfluxDB2::VERSION}"
       headers.each { |k, v| request[k] = v }
 
@@ -100,7 +103,16 @@ module InfluxDB2
           response
         when Net::HTTPRedirection then
           location = response['location']
-          _post(payload, URI.parse(location), limit: limit - 1, headers: headers)
+          redirect_forward_authorization = @options[:redirect_forward_authorization] || false
+
+          uri_redirect = URI.parse(location)
+          uri_redirect.query = uri.query
+          uri_redirect.path = File.join(uri_redirect.path, uri.path)
+
+          redirect_forward_authorization ||= (uri_redirect.host == uri.host) && (uri_redirect.port == uri.port)
+
+          _post(payload, uri_redirect, limit: limit - 1, add_authorization: redirect_forward_authorization,
+                                       headers: headers)
         else
           raise InfluxError.from_response(response)
         end

data/lib/influxdb2/client/flux_csv_parser.rb

@@ -240,7 +240,14 @@ module InfluxDB2
       when 'unsignedLong', 'long', 'duration'
         str_val.to_i
       when 'double'
-        str_val.to_f
+        case str_val
+        when '+Inf'
+          Float::INFINITY
+        when '-Inf'
+          -Float::INFINITY
+        else
+          str_val.to_f
+        end
       when 'base64Binary'
         Base64.decode64(str_val)
       when 'dateTime:RFC3339', 'dateTime:RFC3339Nano'

data/lib/influxdb2/client/version.rb

@@ -19,5 +19,5 @@
 # THE SOFTWARE.
 
 module InfluxDB2
-  VERSION = '1.17.0'.freeze
+  VERSION = '2.0.0'.freeze
 end

data/test/influxdb/flux_csv_parser_test.rb

@@ -1,4 +1,4 @@
-# The MIT License
+# The MIT
 #
 # Permission is hereby granted, free of charge, to any person obtaining a copy
 # of this software and associated documentation files (the "Software"), to deal
@@ -166,7 +166,7 @@ class FluxCsvParserTest < MiniTest::Test
       "#default,mean,,,,,,,,,,,\n" \
       ",result,table,_start,_stop,_time,_value,_field,_measurement,language,license,name,owner\n" \
       ',,0,2020-11-02T07:29:49.55050738Z,2020-12-02T07:29:49.55050738Z,2020-11-02T09:00:00Z,9,' \
-      "stars,github_repository,Ruby,MIT License,influxdb-client-ruby,influxdata\n"
+      "stars,gh,Ruby,MIT,influxdb-client-ruby,influxdata\n"
 
     tables = InfluxDB2::FluxCsvParser.new(data).parse.tables
     records = tables[0].records
@@ -464,4 +464,29 @@ class FluxCsvParserErrorTest < MiniTest::Test
     assert_equal true, tables[0].columns[2].group
     assert_equal 1, tables[1].records.size
   end
+
+  def test_parse_infinity
+    data = '#group,false,false,true,true,true,true,true,true,true,true,false,false
+#datatype,string,long,dateTime:RFC3339,dateTime:RFC3339,string,string,string,string,string,string,double,double
+#default,_result,,,,,,,,,,,
+,result,table,_start,_stop,_field,_measurement,language,license,name,owner,le,_value
+,,0,2021-06-23T06:50:11.897825012Z,2021-06-25T06:50:11.897825012Z,stars,gh,C#,MIT,ruby,influxdata,0,0
+,,0,2021-06-23T06:50:11.897825012Z,2021-06-25T06:50:11.897825012Z,stars,gh,C#,MIT,ruby,influxdata,10,0
+,,0,2021-06-23T06:50:11.897825012Z,2021-06-25T06:50:11.897825012Z,stars,gh,C#,MIT,ruby,influxdata,20,0
+,,0,2021-06-23T06:50:11.897825012Z,2021-06-25T06:50:11.897825012Z,stars,gh,C#,MIT,ruby,influxdata,30,0
+,,0,2021-06-23T06:50:11.897825012Z,2021-06-25T06:50:11.897825012Z,stars,gh,C#,MIT,ruby,influxdata,40,0
+,,0,2021-06-23T06:50:11.897825012Z,2021-06-25T06:50:11.897825012Z,stars,gh,C#,MIT,ruby,influxdata,50,0
+,,0,2021-06-23T06:50:11.897825012Z,2021-06-25T06:50:11.897825012Z,stars,gh,C#,MIT,ruby,influxdata,60,0
+,,0,2021-06-23T06:50:11.897825012Z,2021-06-25T06:50:11.897825012Z,stars,gh,C#,MIT,ruby,influxdata,70,0
+,,0,2021-06-23T06:50:11.897825012Z,2021-06-25T06:50:11.897825012Z,stars,gh,C#,MIT,ruby,influxdata,80,0
+,,0,2021-06-23T06:50:11.897825012Z,2021-06-25T06:50:11.897825012Z,stars,gh,C#,MIT,ruby,influxdata,90,0
+,,0,2021-06-23T06:50:11.897825012Z,2021-06-25T06:50:11.897825012Z,stars,gh,C#,MIT,ruby,influxdata,+Inf,15
+,,0,2021-06-23T06:50:11.897825012Z,2021-06-25T06:50:11.897825012Z,stars,gh,C#,MIT,ruby,influxdata,-Inf,15'
+
+    tables = InfluxDB2::FluxCsvParser.new(data).parse.tables
+    assert_equal 1, tables.size
+    assert_equal 12, tables[0].records.size
+    assert_equal tables[0].records[10].values['le'], Float::INFINITY
+    assert_equal tables[0].records[11].values['le'], -Float::INFINITY
+  end
 end

data/test/influxdb/redirect_test.rb

@@ -0,0 +1,145 @@
+# The MIT License
+#
+# Permission is hereby granted, free of charge, to any person obtaining a copy
+# of this software and associated documentation files (the "Software"), to deal
+# in the Software without restriction, including without limitation the rights
+# to use, copy, modify, merge, publish, distribute, sublicense, and/or sell
+# copies of the Software, and to permit persons to whom the Software is
+# furnished to do so, subject to the following conditions:
+#
+# The above copyright notice and this permission notice shall be included in
+# all copies or substantial portions of the Software.
+#
+# THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND, EXPRESS OR
+# IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF MERCHANTABILITY,
+# FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT. IN NO EVENT SHALL THE
+# AUTHORS OR COPYRIGHT HOLDERS BE LIABLE FOR ANY CLAIM, DAMAGES OR OTHER
+# LIABILITY, WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING FROM,
+# OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS IN
+# THE SOFTWARE.
+
+require 'test_helper'
+
+class WriteApiTest < MiniTest::Test
+  def setup
+    WebMock.disable_net_connect!
+  end
+
+  def test_redirect_same
+    stub_request(:any, 'http://localhost:8086/api/v2/write?bucket=my-bucket&org=my-org&precision=ns')
+      .to_return(status: 307, headers:
+        { 'location' => 'http://localhost:8086' })
+      .then.to_return(status: 204)
+    stub_request(:any, 'http://localhost:8086/api/v2/write?bucket=my-bucket&org=my-org&precision=ns')
+      .to_return(status: 204)
+
+    client = InfluxDB2::Client.new('http://localhost:8086', 'my-token',
+                                   bucket: 'my-bucket',
+                                   org: 'my-org',
+                                   precision: InfluxDB2::WritePrecision::NANOSECOND,
+                                   use_ssl: false)
+
+    client.create_write_api.write(data: 'h2o,location=west value=33i 15')
+
+    headers = {
+      'Authorization' => 'Token my-token',
+      'User-Agent' => "influxdb-client-ruby/#{InfluxDB2::VERSION}",
+      'Content-Type' => 'text/plain'
+    }
+
+    assert_requested(:post, 'http://localhost:8086/api/v2/write?bucket=my-bucket&org=my-org&precision=ns',
+                     times: 1, body: 'h2o,location=west value=33i 15', headers: headers)
+  end
+
+  def test_redirect_301
+    stub_request(:any, 'http://localhost:8086/api/v2/write?bucket=my-bucket&org=my-org&precision=ns')
+      .to_return(status: 301, headers:
+        { 'location' => 'http://localhost:9090/' })
+      .then.to_return(status: 204)
+    stub_request(:any, 'http://localhost:9090/api/v2/write?bucket=my-bucket&org=my-org&precision=ns')
+      .to_return(status: 204)
+
+    client = InfluxDB2::Client.new('http://localhost:8086', 'my-token',
+                                   bucket: 'my-bucket',
+                                   org: 'my-org',
+                                   precision: InfluxDB2::WritePrecision::NANOSECOND,
+                                   use_ssl: false)
+
+    client.create_write_api.write(data: 'h2o,location=west value=33i 15')
+
+    headers = {
+      'Authorization' => 'Token my-token',
+      'User-Agent' => "influxdb-client-ruby/#{InfluxDB2::VERSION}",
+      'Content-Type' => 'text/plain'
+    }
+
+    assert_requested(:post, 'http://localhost:8086/api/v2/write?bucket=my-bucket&org=my-org&precision=ns',
+                     times: 1, body: 'h2o,location=west value=33i 15', headers: headers)
+
+    assert_not_requested(:post, 'http://localhost:9090/api/v2/write?bucket=my-bucket&org=my-org&precision=ns',
+                         times: 1, body: 'h2o,location=west value=33i 15', headers: headers)
+
+    assert_requested(:post, 'http://localhost:9090/api/v2/write?bucket=my-bucket&org=my-org&precision=ns',
+                     times: 1, body: 'h2o,location=west value=33i 15')
+  end
+
+  def test_redirect_301_allow
+    stub_request(:any, 'http://localhost:8086/api/v2/write?bucket=my-bucket&org=my-org&precision=ns')
+      .to_return(status: 301, headers:
+        { 'location' => 'http://localhost:9090/' })
+      .then.to_return(status: 204)
+    stub_request(:any, 'http://localhost:9090/api/v2/write?bucket=my-bucket&org=my-org&precision=ns')
+      .to_return(status: 204)
+
+    client = InfluxDB2::Client.new('http://localhost:8086', 'my-token',
+                                   bucket: 'my-bucket',
+                                   org: 'my-org',
+                                   precision: InfluxDB2::WritePrecision::NANOSECOND,
+                                   use_ssl: false,
+                                   redirect_forward_authorization: true)
+
+    client.create_write_api.write(data: 'h2o,location=west value=33i 15')
+
+    headers = {
+      'Authorization' => 'Token my-token',
+      'User-Agent' => "influxdb-client-ruby/#{InfluxDB2::VERSION}",
+      'Content-Type' => 'text/plain'
+    }
+
+    assert_requested(:post, 'http://localhost:8086/api/v2/write?bucket=my-bucket&org=my-org&precision=ns',
+                     times: 1, body: 'h2o,location=west value=33i 15', headers: headers)
+
+    assert_requested(:post, 'http://localhost:9090/api/v2/write?bucket=my-bucket&org=my-org&precision=ns',
+                     times: 1, body: 'h2o,location=west value=33i 15', headers: headers)
+  end
+
+  def test_redirect_different_path
+    stub_request(:any, 'http://localhost:8086/api/v2/write?bucket=my-bucket&org=my-org&precision=ns')
+      .to_return(status: 301, headers:
+        { 'location' => 'http://localhost:8086/influxdb/' })
+      .then.to_return(status: 204)
+    stub_request(:any, 'http://localhost:8086/influxdb/api/v2/write?bucket=my-bucket&org=my-org&precision=ns')
+      .to_return(status: 204)
+
+    client = InfluxDB2::Client.new('http://localhost:8086', 'my-token',
+                                   bucket: 'my-bucket',
+                                   org: 'my-org',
+                                   precision: InfluxDB2::WritePrecision::NANOSECOND,
+                                   use_ssl: false,
+                                   redirect_forward_authorization: true)
+
+    client.create_write_api.write(data: 'h2o,location=west value=33i 15')
+
+    headers = {
+      'Authorization' => 'Token my-token',
+      'User-Agent' => "influxdb-client-ruby/#{InfluxDB2::VERSION}",
+      'Content-Type' => 'text/plain'
+    }
+
+    assert_requested(:post, 'http://localhost:8086/api/v2/write?bucket=my-bucket&org=my-org&precision=ns',
+                     times: 1, body: 'h2o,location=west value=33i 15', headers: headers)
+
+    assert_requested(:post, 'http://localhost:8086/influxdb/api/v2/write?bucket=my-bucket&org=my-org&precision=ns',
+                     times: 1, body: 'h2o,location=west value=33i 15', headers: headers)
+  end
+end

data/test/influxdb/write_api_test.rb

@@ -201,7 +201,7 @@ class WriteApiTest < MiniTest::Test
   def test_follow_redirect
     stub_request(:any, 'http://localhost:8086/api/v2/write?bucket=my-bucket&org=my-org&precision=ns')
       .to_return(status: 307, headers:
-          { 'location' => 'http://localhost:9090/api/v2/write?bucket=my-bucket&org=my-org&precision=ns' })
+          { 'location' => 'http://localhost:9090/' })
       .then.to_return(status: 204)
     stub_request(:any, 'http://localhost:9090/api/v2/write?bucket=my-bucket&org=my-org&precision=ns')
       .to_return(status: 204)
@@ -223,7 +223,7 @@ class WriteApiTest < MiniTest::Test
   def test_follow_redirect_max
     stub_request(:any, 'http://localhost:8086/api/v2/write?bucket=my-bucket&org=my-org&precision=ns')
       .to_return(status: 307, headers:
-          { 'location' => 'http://localhost:8086/api/v2/write?bucket=my-bucket&org=my-org&precision=ns' })
+          { 'location' => 'http://localhost:8086/' })
 
     client = InfluxDB2::Client.new('http://localhost:8086', 'my-token',
                                    bucket: 'my-bucket',

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: influxdb-client
 version: !ruby/object:Gem::Version
-  version: 1.17.0.pre.3024
+  version: 2.0.0.pre.3196
 platform: ruby
 authors:
 - Jakub Bednar
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2021-07-27 00:00:00.000000000 Z
+date: 2021-08-31 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: bundler
@@ -161,6 +161,7 @@ files:
 - test/influxdb/query_api_integration_test.rb
 - test/influxdb/query_api_stream_test.rb
 - test/influxdb/query_api_test.rb
+- test/influxdb/redirect_test.rb
 - test/influxdb/write_api_batching_test.rb
 - test/influxdb/write_api_integration_test.rb
 - test/influxdb/write_api_test.rb
@@ -201,6 +202,7 @@ test_files:
 - test/influxdb/query_api_integration_test.rb
 - test/influxdb/query_api_stream_test.rb
 - test/influxdb/query_api_test.rb
+- test/influxdb/redirect_test.rb
 - test/influxdb/write_api_batching_test.rb
 - test/influxdb/write_api_integration_test.rb
 - test/influxdb/write_api_test.rb