fluent-plugin-azure-storage-append-blob-lts 0.4.0 → 0.6.3

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 8f0868aba2220e9c160b9e8bec8854921864b580c1b7cb1d05845ca9e73c4606
-  data.tar.gz: 4d73dcf3a57b45018de76a862407062049735da53e96aa664db6304bc65b8718
+  metadata.gz: f9e6de27ebbf624d51968e415d502111523d3fd670d2345558381b03895934d6
+  data.tar.gz: 80fbe42cee9d82adb02afbaf2cfed1fd92b0d4fbbbfb1e67b50e328795d90b23
 SHA512:
-  metadata.gz: d72f2ccdfb8c92f13694c05f4fc9a349144dd4016b0c1eebee613d8ef536a5072a8635028a097cf21a546220cd11f870eddd4675b1e3241194b39d2a0ddf2d46
-  data.tar.gz: a7dfd1c386ed529d9ead3978fe436c7f69a653bd4ce206b5755c8af78142464c4779ab3d781fc14c035f650e5e8940c97a8a4dca240a933ffb5d733b924d9ae3
+  metadata.gz: d3c2e0296f88ae875f19043e322678aa9a7ea3a26e9bb41f9f4292991ff09a6a50022d03c1da48b36e0c724c2e9e83f70ef431e1b9a3fc628507774a8de32943
+  data.tar.gz: a86038ca1acc515dd05df9f86d014cef337c5cc8a70847a15dd94b8f86fe2233c5b0a49be988f26a3da86f298d1aa27465aab4fb904ddd8eeef9e16535291d99

data/.github/CODEOWNERS

@@ -0,0 +1 @@
+* @elsesiy

data/.github/workflows/publish.yaml

@@ -11,14 +11,12 @@ jobs:
   build:
     name: Build + Publish
     runs-on: ubuntu-latest
-
     steps:
     - uses: actions/checkout@v2
     - name: Set up Ruby 2.6
       uses: actions/setup-ruby@v1
       with:
         ruby-version: 2.6.x
-
     - name: Publish to RubyGems
       run: |
         mkdir -p $HOME/.gem

data/.github/workflows/test.yaml

@@ -8,9 +8,7 @@ on:
 
 jobs:
   test:
-
     runs-on: ubuntu-latest
-
     steps:
     - uses: actions/checkout@v2
     - name: Set up Ruby

data/Dockerfile

@@ -1,8 +1,8 @@
-FROM ruby:latest
+FROM ruby:2.7
 
 WORKDIR /plugin
 
-ADD . /plugin
+COPY . /plugin
 
 RUN gem install bundler && \
     gem install fluentd --no-doc && \

data/README.md

@@ -1,5 +1,9 @@
 # fluent-plugin-azure-storage-append-blob-lts
 
+![Tests](https://github.com/elsesiy/fluent-plugin-azure-storage-append-blob-lts/workflows/Test/badge.svg?branch=master)
+[![Gem Version](https://badge.fury.io/rb/fluent-plugin-azure-storage-append-blob-lts.svg)](https://badge.fury.io/rb/fluent-plugin-azure-storage-append-blob-lts)
+[![Twitter](https://img.shields.io/badge/twitter-@elsesiy-blue.svg)](http://twitter.com/elsesiy)
+
 Azure Storage Append Blob output plugin buffers logs in local file and uploads them to Azure Storage Append Blob periodically.
 This plugin is a fork of [microsoft/fluent-plugin-azure-storage-append-blob](https://github.com/microsoft/fluent-plugin-azure-storage-append-blob) which is not actively maintained.
 
@@ -24,6 +28,8 @@ And then execute:
     <match pattern>
       type azure-storage-append-blob
 
+      azure_cloud                       <azure cloud environment>
+      azure_storage_dns_suffix          <your azure storage dns suffix> # only used for Azure Stack Cloud
       azure_storage_account             <your azure storage account>
       azure_storage_access_key          <your azure storage access key> # leave empty to use MSI
       azure_storage_connection_string   <your azure storage connection string> # leave empty to use MSI
@@ -36,6 +42,7 @@ And then execute:
       path                              logs/
       azure_object_key_format           %{path}%{time_slice}_%{index}.log
       time_slice_format                 %Y%m%d-%H
+      compute_checksums                 true
       # if you want to use %{tag} or %Y/%m/%d/ like syntax in path / azure_blob_name_format,
       # need to specify tag for %{tag} and time for %Y/%m/%d in <buffer> argument.
       <buffer tag,time>
@@ -47,6 +54,18 @@ And then execute:
       </buffer>
     </match>
 
+### `azure_cloud` (Optional)
+
+Default: `AZUREPUBLICCLOUD`
+
+Cloud environment used to determine the storage endpoint suffix to use, see [here](https://github.com/Azure/go-autorest/blob/master/autorest/azure/environments.go).
+
+Use `AZURESTACKCLOUD` for Azure Stack Cloud.
+
+### `azure_storage_dns_suffix` (Required only for Azure Stack Cloud)
+
+Your Azure Storage endpoint suffix. This can be retrieved from Azure Storage connection string, `EndpointSuffix` section.
+
 ### `azure_storage_account` (Required)
 
 Your Azure Storage Account Name. This can be retrieved from Azure Management portal.
@@ -60,15 +79,15 @@ If all are empty, the plugin will use the local Managed Identity endpoint to obt
 
 ### `azure_imds_api_version` (Optional, only for MSI)
 
-Default: 2019-08-15
+Default: `2020-12-01`
 
 The Instance Metadata Service is used during the OAuth flow to obtain an access token. This API is versioned and specifying the version is mandatory.
 
-See [here](https://docs.microsoft.com/en-us/azure/virtual-machines/linux/instance-metadata-service#versioning) for more details.
+See [here](https://docs.microsoft.com/en-us/azure/virtual-machines/linux/instance-metadata-service?tabs=windows#versioning) for more details.
 
 ### `azure_token_refresh_interval` (Optional, only for MSI)
 
-Default: 60 (1 hour)
+Default: `60` (1 hour)
 
 When using MSI, the initial access token needs to be refreshed periodically.
 
@@ -82,8 +101,9 @@ Azure Identity Client ID to use for accessing Azure Blob service.
 
 ### `auto_create_container`
 
+Default: `true`
+
 This plugin creates the Azure container if it does not already exist exist when you set 'auto_create_container' to true.
-The default value is `true`
 
 ### `azure_object_key_format`
 
@@ -127,6 +147,14 @@ The [fluent-mixin-config-placeholders](https://github.com/tagomoris/fluent-mixin
 
 Format of the time used in the file name. Default is '%Y%m%d'. Use '%Y%m%d%H' to split files hourly.
 
+### `compute_checksums`
+
+Default: `true`
+
+Whether to compute MD5 checksum of the blob contents during append operation and provide it in a header for the blob service.
+
+You want to set it to `false` in FIPS-enabled environments.
+
 ### Run tests
 
     gem install bundler

data/fluent-plugin-azure-storage-append-blob-lts.gemspec

@@ -3,12 +3,12 @@ $LOAD_PATH.unshift(lib) unless $LOAD_PATH.include?(lib)
 
 Gem::Specification.new do |spec|
   spec.name = 'fluent-plugin-azure-storage-append-blob-lts'
-  spec.version = '0.4.0'
+  spec.version = '0.6.3'
   spec.authors = ['Jonas-Taha El Sesiy']
-  spec.email = ['']
+  spec.email = ['github@elsesiy.com']
 
   spec.summary = 'Azure Storage Append Blob output plugin for Fluentd event collector'
-  spec.description = 'Fluentd plugin to upload logs to Azure Storage append blobs. Fork of https://github.com/elsesiy/fluent-plugin-azure-storage-append-blob'
+  spec.description = 'Fluentd plugin to upload logs to Azure Storage append blobs. Fork of https://github.com/microsoft/fluent-plugin-azure-storage-append-blob'
   spec.homepage = 'https://github.com/elsesiy/fluent-plugin-azure-storage-append-blob-lts'
   spec.license = 'MIT'
 

data/lib/fluent/plugin/out_azure-storage-append-blob.rb

@@ -11,7 +11,7 @@ require 'json'
 
 module Fluent
   module Plugin
-    class AzureStorageAppendBlobOut < Fluent::Plugin::Output
+    class AzureStorageAppendBlobOut < Output
       Fluent::Plugin.register_output('azure-storage-append-blob', self)
 
       helpers :formatter, :inject
@@ -19,18 +19,33 @@ module Fluent
       DEFAULT_FORMAT_TYPE = 'out_file'.freeze
       AZURE_BLOCK_SIZE_LIMIT = 4 * 1024 * 1024 - 1
 
+      def initialize
+        super
+        @use_msi = false
+
+        # Storage endpoint suffixes for various environments, see https://github.com/Azure/go-autorest/blob/master/autorest/azure/environments.go
+        @storage_endpoint_mapping = {
+          'AZURECHINACLOUD' => 'core.chinacloudapi.cn',
+          'AZUREGERMANCLOUD' => 'core.cloudapi.de',
+          'AZUREPUBLICCLOUD' => 'core.windows.net',
+          'AZUREUSGOVERNMENTCLOUD' => 'core.usgovcloudapi.net'
+        }
+      end
+
       config_param :path, :string, default: ''
       config_param :azure_storage_account, :string, default: nil
       config_param :azure_storage_access_key, :string, default: nil, secret: true
-      config_param :azure_storage_sas_token, :string, default: nil, secret: true
       config_param :azure_storage_connection_string, :string, default: nil, secret: true
+      config_param :azure_storage_dns_suffix, :string, default: nil
+      config_param :azure_storage_sas_token, :string, default: nil, secret: true
+      config_param :azure_cloud, :string, default: 'AZUREPUBLICCLOUD'
       config_param :azure_msi_client_id, :string, default: nil
       config_param :azure_container, :string, default: nil
-      config_param :azure_imds_api_version, :string, default: '2019-08-15'
+      config_param :azure_imds_api_version, :string, default: '2020-12-01'
       config_param :azure_token_refresh_interval, :integer, default: 60
-      config_param :use_msi, :bool, default: false
       config_param :azure_object_key_format, :string, default: '%{path}%{time_slice}-%{index}.log'
       config_param :auto_create_container, :bool, default: true
+      config_param :compute_checksums, :bool, default: true
       config_param :format, :string, default: DEFAULT_FORMAT_TYPE
       config_param :time_slice_format, :string, default: '%Y%m%d'
       config_param :localtime, :bool, default: false
@@ -61,15 +76,31 @@ module Fluent
                          end
                        end
 
-        raise ConfigError, 'azure_storage_account needs to be specified' if @azure_storage_account.nil?
-
-        raise ConfigError, 'azure_container needs to be specified' if @azure_container.nil?
+        if @azure_cloud == 'AZURESTACKCLOUD'
+          if @azure_storage_dns_suffix.nil?
+            raise ConfigError, 'azure_storage_dns_suffix invalid, must not be empty for AZURESTACKCLOUD'
+          end
+        else
+          @azure_storage_dns_suffix = @storage_endpoint_mapping[@azure_cloud]
+          if @azure_storage_dns_suffix.nil?
+            raise ConfigError, 'azure_cloud invalid, must be either of AZURECHINACLOUD, AZUREGERMANCLOUD, AZUREPUBLICCLOUD, AZUREUSGOVERNMENTCLOUD, AZURESTACKCLOUD'
+          end
+        end
 
         if (@azure_storage_access_key.nil? || @azure_storage_access_key.empty?) &&
            (@azure_storage_sas_token.nil? || @azure_storage_sas_token.empty?) &&
            (@azure_storage_connection_string.nil? || @azure_storage_connection_string.empty?)
-          log.info 'Using MSI since neither azure_storage_access_key nor azure_storage_sas_token was provided.'
+          log.info 'Using MSI since neither of azure_storage_access_key, azure_storage_sas_token, azure_storage_connection_string was provided.'
           @use_msi = true
+        elsif @azure_storage_connection_string.nil? || @azure_storage_connection_string.empty?
+          raise ConfigError, 'azure_storage_account needs to be specified' if @azure_storage_account.nil?
+          raise ConfigError, 'azure_container needs to be specified' if @azure_container.nil?
+        end
+
+        @blob_options = {}
+
+        if !@compute_checksums
+          @blob_options[:content_md5] = ''
         end
       end
 
@@ -81,13 +112,13 @@ module Fluent
         access_key_request = Faraday.new('http://169.254.169.254/metadata/identity/oauth2/token?' \
                                          "api-version=#{@azure_imds_api_version}" \
                                          '&resource=https://storage.azure.com/' \
-                                         "#{! azure_msi_client_id.nil? ? "&client_id=#{azure_msi_client_id}" : ''}",
+                                         "#{!azure_msi_client_id.nil? ? "&client_id=#{azure_msi_client_id}" : ''}",
                                          headers: { 'Metadata' => 'true' }).get
 
         if access_key_request.status == 200
           JSON.parse(access_key_request.body)['access_token']
         else
-          raise "Access token request was not sucssessful. Possibly due to missing azure_msi_client_id config parameter."
+          raise 'Access token request was not sucssessful. Possibly due to missing azure_msi_client_id config parameter.'
         end
       end
 
@@ -96,7 +127,11 @@ module Fluent
         if @use_msi
           token_credential = Azure::Storage::Common::Core::TokenCredential.new get_access_token
           token_signer = Azure::Storage::Common::Core::Auth::TokenSigner.new token_credential
-          @bs = Azure::Storage::Blob::BlobService.new(storage_account_name: @azure_storage_account, signer: token_signer)
+          @bs = Azure::Storage::Blob::BlobService.new(
+            storage_account_name: @azure_storage_account,
+            storage_dns_suffix: @azure_storage_dns_suffix,
+            signer: token_signer
+          )
 
           refresh_interval = @azure_token_refresh_interval * 60
           cancelled = false
@@ -113,7 +148,7 @@ module Fluent
         elsif !@azure_storage_connection_string.nil? && !@azure_storage_connection_string.empty?
           @bs = Azure::Storage::Blob::BlobService.create_from_connection_string(@azure_storage_connection_string)
         else
-          @bs_params = { storage_account_name: @azure_storage_account }
+          @bs_params = { storage_account_name: @azure_storage_account, storage_dns_suffix: @azure_storage_dns_suffix }
 
           if !@azure_storage_access_key.nil? && !@azure_storage_access_key.empty?
             @bs_params.merge!({ storage_access_key: @azure_storage_access_key })
@@ -137,23 +172,26 @@ module Fluent
       end
 
       def write(chunk)
-        metadata = chunk.metadata
         tmp = Tempfile.new('azure-')
         begin
           chunk.write_to(tmp)
 
-          generate_log_name(metadata, @current_index)
+          generate_log_name(chunk, @current_index)
           if @last_azure_storage_path != @azure_storage_path
             @current_index = 0
-            generate_log_name(metadata, @current_index)
+            generate_log_name(chunk, @current_index)
           end
 
           content = File.open(tmp.path, 'rb', &:read)
 
-          append_blob(content, metadata)
+          append_blob(content, chunk)
           @last_azure_storage_path = @azure_storage_path
         ensure
-          tmp.close(true) rescue nil
+          begin
+            tmp.close(true)
+          rescue StandardError
+            nil
+          end
         end
       end
 
@@ -182,7 +220,8 @@ module Fluent
         end
       end
 
-      def generate_log_name(metadata, index)
+      def generate_log_name(chunk, index)
+        metadata = chunk.metadata
         time_slice = if metadata.timekey.nil?
                        ''.freeze
                      else
@@ -196,41 +235,39 @@ module Fluent
           '%{index}' => index
         }
         storage_path = @azure_object_key_format.gsub(/%{[^}]+}/, values_for_object_key)
-        @azure_storage_path = extract_placeholders(storage_path, metadata)
+        @azure_storage_path = extract_placeholders(storage_path, chunk)
       end
 
-      def append_blob(content, metadata)
+      def append_blob(content, chunk)
         position = 0
         log.debug "azure_storage_append_blob: append_blob.start: Content size: #{content.length}"
         loop do
-          begin
-            size = [content.length - position, AZURE_BLOCK_SIZE_LIMIT].min
-            log.debug "azure_storage_append_blob: append_blob.chunk: content[#{position}..#{position + size}]"
-            @bs.append_blob_block(@azure_container, @azure_storage_path, content[position..position + size - 1])
-            position += size
-            break if position >= content.length
-          rescue Azure::Core::Http::HTTPError => e
-            status_code = e.status_code
-
-            if status_code == 409 # exceeds azure block limit
-              @current_index += 1
-              old_azure_storage_path = @azure_storage_path
-              generate_log_name(metadata, @current_index)
-
-              # If index is not a part of format, rethrow exception.
-              if old_azure_storage_path == @azure_storage_path
-                log.warn 'azure_storage_append_blob: append_blob: blocks limit reached, you need to use %{index} for the format.'
-                raise
-              end
-
-              log.debug "azure_storage_append_blob: append_blob: blocks limit reached, creating new blob #{@azure_storage_path}."
-              @bs.create_append_blob(@azure_container, @azure_storage_path)
-            elsif status_code == 404 # blob not found
-              log.debug "azure_storage_append_blob: append_blob: #{@azure_storage_path} blob doesn't exist, creating new blob."
-              @bs.create_append_blob(@azure_container, @azure_storage_path)
-            else
+          size = [content.length - position, AZURE_BLOCK_SIZE_LIMIT].min
+          log.debug "azure_storage_append_blob: append_blob.chunk: content[#{position}..#{position + size}]"
+          @bs.append_blob_block(@azure_container, @azure_storage_path, content[position..position + size - 1], options=@blob_options)
+          position += size
+          break if position >= content.length
+        rescue Azure::Core::Http::HTTPError => e
+          status_code = e.status_code
+
+          if status_code == 409 # exceeds azure block limit
+            @current_index += 1
+            old_azure_storage_path = @azure_storage_path
+            generate_log_name(chunk, @current_index)
+
+            # If index is not a part of format, rethrow exception.
+            if old_azure_storage_path == @azure_storage_path
+              log.warn 'azure_storage_append_blob: append_blob: blocks limit reached, you need to use %{index} for the format.'
               raise
             end
+
+            log.debug "azure_storage_append_blob: append_blob: blocks limit reached, creating new blob #{@azure_storage_path}."
+            @bs.create_append_blob(@azure_container, @azure_storage_path)
+          elsif status_code == 404 # blob not found
+            log.debug "azure_storage_append_blob: append_blob: #{@azure_storage_path} blob doesn't exist, creating new blob."
+            @bs.create_append_blob(@azure_container, @azure_storage_path)
+          else
+            raise
           end
         end
         log.debug 'azure_storage_append_blob: append_blob.complete'

data/test/plugin/test_out_azure_storage_append_blob.rb

@@ -11,6 +11,7 @@ class AzureStorageAppendBlobOutTest < Test::Unit::TestCase
   end
 
   CONFIG = %(
+    azure_cloud AZUREGERMANCLOUD
     azure_storage_account test_storage_account
     azure_storage_access_key MY_FAKE_SECRET
     azure_container test_container
@@ -18,6 +19,12 @@ class AzureStorageAppendBlobOutTest < Test::Unit::TestCase
     path log
   ).freeze
 
+  CONNSTR_CONFIG = %(
+    azure_storage_connection_string https://test
+    time_slice_format        %Y%m%d-%H
+    path log
+  ).freeze
+
   MSI_CONFIG = %(
     azure_storage_account test_storage_account
     azure_container test_container
@@ -27,6 +34,16 @@ class AzureStorageAppendBlobOutTest < Test::Unit::TestCase
     path log
   ).freeze
 
+  AZURESTACKCLOUD_CONFIG = %(
+    azure_cloud AZURESTACKCLOUD
+    azure_storage_dns_suffix test.storage.dns.suffix
+    azure_storage_account test_storage_account
+    azure_storage_access_key MY_FAKE_SECRET
+    azure_container test_container
+    time_slice_format        %Y%m%d-%H
+    path log
+  ).freeze
+
   def create_driver(conf: CONFIG, service: nil)
     d = Fluent::Test::Driver::Output.new(Fluent::Plugin::AzureStorageAppendBlobOut).configure(conf)
     d.instance.instance_variable_set(:@bs, service)
@@ -36,7 +53,7 @@ class AzureStorageAppendBlobOutTest < Test::Unit::TestCase
 
   sub_test_case 'test config' do
     test 'config should reject with no azure container' do
-      assert_raise Fluent::ConfigError do
+      assert_raise Fluent::ConfigError.new('azure_container needs to be specified') do
         create_driver conf: %(
           azure_storage_account test_storage_account
           azure_storage_access_key MY_FAKE_SECRET
@@ -47,8 +64,25 @@ class AzureStorageAppendBlobOutTest < Test::Unit::TestCase
       end
     end
 
+    test 'config should reject for invalid cloud ' do
+      assert_raise Fluent::ConfigError.new('azure_cloud invalid, must be either of AZURECHINACLOUD, AZUREGERMANCLOUD, AZUREPUBLICCLOUD, AZUREUSGOVERNMENTCLOUD, AZURESTACKCLOUD') do
+        create_driver conf: %(
+          azure_cloud INVALIDCLOUD
+        )
+      end
+    end
+
+    test 'config should reject for Azure Stack Cloud with no azure storage dns suffix' do
+      assert_raise Fluent::ConfigError.new('azure_storage_dns_suffix invalid, must not be empty for AZURESTACKCLOUD') do
+        create_driver conf: %(
+          azure_cloud AZURESTACKCLOUD
+        )
+      end
+    end
+
     test 'config with access key should set instance variables' do
       d = create_driver
+      assert_equal 'core.cloudapi.de', d.instance.instance_variable_get(:@azure_storage_dns_suffix)
       assert_equal 'test_storage_account', d.instance.azure_storage_account
       assert_equal 'MY_FAKE_SECRET', d.instance.azure_storage_access_key
       assert_equal 'test_container', d.instance.azure_container
@@ -60,12 +94,29 @@ class AzureStorageAppendBlobOutTest < Test::Unit::TestCase
       d = create_driver conf: MSI_CONFIG
       assert_equal 'test_storage_account', d.instance.azure_storage_account
       assert_equal 'test_container', d.instance.azure_container
-      assert_equal true, d.instance.use_msi
+      assert_equal true, d.instance.instance_variable_get(:@use_msi)
       assert_equal true, d.instance.auto_create_container
       assert_equal '%{path}%{time_slice}-%{index}.log', d.instance.azure_object_key_format
       assert_equal 120, d.instance.azure_token_refresh_interval
       assert_equal '1970-01-01', d.instance.azure_imds_api_version
     end
+
+    test 'config with connection string should set instance variables' do
+      d = create_driver conf: CONNSTR_CONFIG
+      assert_equal 'https://test', d.instance.azure_storage_connection_string
+      assert_equal false, d.instance.instance_variable_get(:@use_msi)
+      assert_equal true, d.instance.auto_create_container
+    end
+  
+    test 'config for Azure Stack Cloud should set instance variables' do
+      d = create_driver conf: AZURESTACKCLOUD_CONFIG
+      assert_equal 'test.storage.dns.suffix', d.instance.instance_variable_get(:@azure_storage_dns_suffix)
+      assert_equal 'test_storage_account', d.instance.azure_storage_account
+      assert_equal 'MY_FAKE_SECRET', d.instance.azure_storage_access_key
+      assert_equal 'test_container', d.instance.azure_container
+      assert_equal true, d.instance.auto_create_container
+      assert_equal '%{path}%{time_slice}-%{index}.log', d.instance.azure_object_key_format
+    end
   end
 
   sub_test_case 'test path slicing' do
@@ -92,15 +143,13 @@ class AzureStorageAppendBlobOutTest < Test::Unit::TestCase
   # This class is used to create an Azure::Core::Http::HTTPError. HTTPError parses
   # a response object when it is created.
   class FakeResponse
-    def initialize(status=404)
+    def initialize(status = 404)
       @status = status
-      @body = "body"
+      @body = 'body'
       @headers = {}
     end
 
-    attr_reader :status
-    attr_reader :body
-    attr_reader :headers
+    attr_reader :status, :body, :headers
   end
 
   # This class is used to test plugin functions which interact with the blob service
@@ -111,11 +160,11 @@ class AzureStorageAppendBlobOutTest < Test::Unit::TestCase
     end
     attr_reader :blocks
 
-    def append_blob_block(container, path, data)
+    def append_blob_block(_container, _path, data, options={})
       @blocks.append(data)
     end
 
-    def get_container_properties(container)
+    def get_container_properties(_container)
       unless @response.status_code == 200
         raise Azure::Core::Http::HTTPError.new(@response)
       end
@@ -125,18 +174,18 @@ class AzureStorageAppendBlobOutTest < Test::Unit::TestCase
   sub_test_case 'test container_exists' do
     test 'container 404 returns false' do
       d = create_driver service: FakeBlobService.new(404)
-      assert_false d.instance.container_exists? "anything"
+      assert_false d.instance.container_exists? 'anything'
     end
 
     test 'existing container returns true' do
       d = create_driver service: FakeBlobService.new(200)
-      assert_true d.instance.container_exists? "anything"
+      assert_true d.instance.container_exists? 'anything'
     end
 
     test 'unexpected exception raises' do
       d = create_driver service: FakeBlobService.new(500)
       assert_raise_kind_of Azure::Core::Http::HTTPError do
-        d.instance.container_exists? "anything"
+        d.instance.container_exists? 'anything'
       end
     end
   end
@@ -175,10 +224,10 @@ class AzureStorageAppendBlobOutTest < Test::Unit::TestCase
 
     test 'long buffer appends multiple times' do
       limit = Fluent::Plugin::AzureStorageAppendBlobOut::AZURE_BLOCK_SIZE_LIMIT
-      buf_1 = 'a' * limit
-      buf_2 = 'a' * 3
-      blocks = fake_appended_blocks buf_1 + buf_2
-      assert_equal [buf_1, buf_2], blocks
+      buf1 = 'a' * limit
+      buf2 = 'a' * 3
+      blocks = fake_appended_blocks buf1 + buf2
+      assert_equal [buf1, buf2], blocks
     end
   end
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: fluent-plugin-azure-storage-append-blob-lts
 version: !ruby/object:Gem::Version
-  version: 0.4.0
+  version: 0.6.3
 platform: ruby
 authors:
 - Jonas-Taha El Sesiy
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2021-01-20 00:00:00.000000000 Z
+date: 2021-04-03 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: bundler
@@ -87,13 +87,14 @@ dependencies:
       - !ruby/object:Gem::Version
         version: '2'
 description: Fluentd plugin to upload logs to Azure Storage append blobs. Fork of
-  https://github.com/elsesiy/fluent-plugin-azure-storage-append-blob
+  https://github.com/microsoft/fluent-plugin-azure-storage-append-blob
 email:
-- ''
+- github@elsesiy.com
 executables: []
 extensions: []
 extra_rdoc_files: []
 files:
+- ".github/CODEOWNERS"
 - ".github/workflows/publish.yaml"
 - ".github/workflows/test.yaml"
 - ".gitignore"