singleton-client-test 0.7.7.5 → 0.7.7.6

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 6ab2a482cbb7c83aca190392287160802cae5488a7356dfd24a79476817f40ac
-  data.tar.gz: e691e2eca4f3c2a2f747970ff746b045e4c1cbaa3de23e73eeba84a7f812c2ec
+  metadata.gz: 3bd55bdad7f46bf9c0cffd42a710cfb6dcae97ecadbbb148af41d2b44be0afca
+  data.tar.gz: 026ef3a390e9a88691decb97b1131ac31e358c9ede7628a7b698d38899a80465
 SHA512:
-  metadata.gz: 619fed54ca5da856c0d8933724e69bb286deba7aebd94a6c98d674dc8c532dc2937cbee650ebb0e337351a3d0b8515dd06f1884066412009098f85c09a4373fc
-  data.tar.gz: 9500161a55b6e4a2c43f2955168895ddfbde82ae1fe68a6a2fcbe4aa1f1498bfaa99a3fe3f82962fac288f455f5a775e26c2193a945940eaaa4cdafbeca90fe8
+  metadata.gz: 5792e7d5f543603927732321cf440be37266fdc5a65fdd6075017068bb98fd9e5b8b4c52e8421fb0508166997993117d6c151921243b642ea448fb5ceea2e542
+  data.tar.gz: 7a8938ba9ffe0b6480a4b83e441b48d48eec98f5e2262019ff5db001e94fe50d404cc3c4277e464cab90ab7141076223b370d2c3a733058d95eecd940f8d8e88

data/lib/sgtn-client/api/translation.rb

@@ -43,7 +43,7 @@ module SgtnClient
         SgtnClient.logger.debug { "[#{method(__callee__).owner}.#{__callee__}] key: #{key}, component: #{component}, locale: #{locale}, args: #{kwargs}" }
 
         begin
-          best_match_locale = LocaleUtil.get_best_locale(locale || self.locale, component)
+          best_match_locale = LocaleUtil.get_best_locale(locale || SgtnClient.locale, component)
           messages, actual_locale = get_bundle_with_fallback(component, best_match_locale)
           result = messages&.fetch(key, nil)
         rescue StandardError => e
@@ -69,7 +69,7 @@ module SgtnClient
       def get_translations(component, locale = nil)
         SgtnClient.logger.debug { "[#{method(__callee__).owner}.#{__callee__}] component: #{component}, locale: #{locale}" }
 
-        best_match_locale = LocaleUtil.get_best_locale(locale || self.locale, component)
+        best_match_locale = LocaleUtil.get_best_locale(locale || SgtnClient.locale, component)
         messages, actual_locale = get_bundle_with_fallback(component, best_match_locale)
 
         { 'component' => component, 'locale' => actual_locale, 'messages' => messages } if messages
@@ -78,14 +78,6 @@ module SgtnClient
         nil
       end
 
-      def locale
-        RequestStore.store[:locale] ||= LocaleUtil.get_fallback_locale
-      end
-
-      def locale=(value)
-        RequestStore.store[:locale] = value
-      end
-
       private
 
       def get_bundle(component, locale)

data/lib/sgtn-client/core/config.rb

@@ -18,8 +18,10 @@ module SgtnClient
     def logger
       @logger ||= if log_file
                     puts "create log file: '#{log_file}', level: #{log_level}"
-                    require 'lumberjack'
-                    Lumberjack::Logger.new(log_file, level: log_level, max_size: '1M', keep: 4)
+                    require 'logging'
+                    logger = Logging.logger(log_file, 4, 1_048_576)
+                    logger.level = log_level
+                    logger
                   else
                     require 'logger'
                     Logger.new(STDOUT, level: log_level || Logger::INFO)

data/lib/sgtn-client/loader/cache.rb

@@ -15,7 +15,7 @@ module SgtnClient
 
       # get from cache, return expired data immediately
       def get_bundle(component, locale)
-        SgtnClient.logger.debug { "[#{caller[2]}] component=#{component}, locale=#{locale}" }
+        SgtnClient.logger.debug { "[#{__FILE__}][#{__callee__}] component=#{component}, locale=#{locale}" }
 
         result = @cache_hash[Common::BundleID.new(component, locale)] || load_bundle(component, locale)
       ensure
@@ -23,7 +23,7 @@ module SgtnClient
       end
 
       def available_bundles
-        SgtnClient.logger.debug { "[#{caller[2]}]" }
+        SgtnClient.logger.debug { "[#{__FILE__}][#{__callee__}]" }
 
         result = @cache_hash[CONSTS::AVAILABLE_BUNDLES_KEY] || super
       ensure
@@ -33,13 +33,13 @@ module SgtnClient
 
     module CacheFiller # :nodoc:
       def load_bundle(component, locale)
-        SgtnClient.logger.debug { "[#{caller[2]}] component=#{component}, locale=#{locale}" }
+        SgtnClient.logger.debug { "[#{__FILE__}][#{__callee__}] CacheFiller, component=#{component}, locale=#{locale}" }
 
         @cache_hash[Common::BundleID.new(component, locale)] = Common::BundleData.new(super)
       end
 
       def available_bundles
-        SgtnClient.logger.debug { "[#{caller[2]}]" }
+        SgtnClient.logger.debug { "[#{__FILE__}][#{__callee__}] CacheFiller" }
 
         item = super
         old_item = @cache_hash[CONSTS::AVAILABLE_BUNDLES_KEY]

data/lib/sgtn-client/loader/server.rb

@@ -9,27 +9,34 @@ require 'set'
 
 module SgtnClient
   module TranslationLoader
-    class SgtnServer
+    class SgtnServer # :nodoc:
       ERROR_ILLEGAL_DATA = 'server returned illegal data.'
       ERROR_BUSINESS_ERROR = 'server returned business error.'
+      ERROR_PARTIAL_SUCCESS = 'the request to server was partially successful.'
+      ERROR_NO_DATA = 'no expected data in response from server. path: %s. Body is: %s'
 
       REQUEST_ARGUMENTS = { timeout: 10 }.freeze
 
       def initialize(config)
-        @server_url = config.vip_server
-
-        product_root = format('i18n/api/v2/translation/products/%s/versions/%s', config.product_name, config.version)
+        product_root = format('i18n/api/v2/translation/products/%<name>s/versions/%<version>s',
+                              name: config.product_name, version: config.version)
 
         @bundle_url = "#{product_root}/locales/%s/components/%s"
         @locales_url = "#{product_root}/localelist"
         @components_url = "#{product_root}/componentlist"
+
+        @conn = Faraday.new(config.vip_server, request: REQUEST_ARGUMENTS) do |f|
+          f.response :json # decode response bodies as JSON
+          f.use :gzip
+          f.response :raise_error
+          f.response :logger, config.logger, { log_level: :debug, headers: false, bodies: true }
+        end
       end
 
       def load_bundle(component, locale)
         SgtnClient.logger.debug { "[#{method(__callee__).owner}.#{__callee__}] component=#{component}, locale=#{locale}" }
 
-        messages = query_server(format(@bundle_url, locale, component), ['messages'])
-        messages
+        query_server(format(@bundle_url, locale, component), ['messages'])
       end
 
       def available_bundles
@@ -54,23 +61,12 @@ module SgtnClient
       end
 
       def query_server(url, path_to_data = [], queries = nil, headers = nil)
-        conn = Faraday.new(@server_url, request: REQUEST_ARGUMENTS) do |f|
-          f.response :json # decode response bodies as JSON
-          f.use :gzip
-          f.response :raise_error
-          f.response :logger, SgtnClient.logger, { log_level: :debug }
-        end
-        resp = conn.get(url, queries, headers)
+        resp = @conn.get(url, queries, headers)
 
         process_business_error(resp.body)
-        extract_data(resp.body, path_to_data)
-      end
-
-      def extract_data(parsedbody, path_to_data)
-        data = parsedbody.dig('data', *path_to_data)
-        raise SingletonError, "no expected data in response. Body is: #{parsedbody}" unless data
 
-        data
+        resp.body&.dig('data', *path_to_data) ||
+          (raise SingletonError, format(ERROR_NO_DATA, path_to_data, resp.body))
       end
 
       def process_business_error(parsedbody)
@@ -79,8 +75,8 @@ module SgtnClient
           raise SingletonError, "#{ERROR_BUSINESS_ERROR} #{parsedbody['response']}"
         end
 
-        # 600 means a successful response, 6xx means partial successful.
-        SgtnClient.logger.warn "#{ERROR_BUSINESS_ERROR} #{parsedbody['response']}" if b_code > 600
+        # 600/200 means a successful response, 6xx/2xx means partial successful.
+        SgtnClient.logger.warn "#{ERROR_PARTIAL_SUCCESS} #{parsedbody['response']}" if b_code != 600 && b_code != 200
       rescue TypeError, ArgumentError, NoMethodError => e
         raise SingletonError, "#{ERROR_ILLEGAL_DATA} #{e}. Body is: #{parsedbody}"
       end

data/lib/sgtn-client/sgtn-client.rb

@@ -38,5 +38,13 @@ module SgtnClient # :nodoc:
       config.update(config_hash)
       ValidateUtil.validate_config
     end
+
+    def locale
+      RequestStore.store[:locale] ||= LocaleUtil.get_fallback_locale
+    end
+
+    def locale=(value)
+      RequestStore.store[:locale] = value
+    end
   end
 end

data/lib/sgtn-client/util/locale-util.rb

@@ -29,7 +29,8 @@ module SgtnClient
       end
 
       component_result[locale] ||= begin
-                # component_result.shift if component_result.size >= 50
+                component_result.delete(component_result.keys[Random.rand(component_result.size)]) if component_result.size >= 50
+
                 if SgtnClient.config.available_locales(component).include?(locale)
                   locale
                 elsif locale.nil?

data/lib/singleton-client.rb

@@ -14,11 +14,26 @@ module Sgtn # :nodoc:
     end
 
     def_delegator SgtnClient::Config, :instance, :config
-    delegate %i[translate t get_translations locale locale=] => SgtnClient::Translation,
+    delegate %i[translate t get_translations] => SgtnClient::Translation,
+             %i[locale locale=] => SgtnClient,
              %i[logger product_name version vip_server translation_bundle
                 source_bundle cache_expiry_period log_file log_level].flat_map { |m|
                [m, "#{m}=".to_sym]
              } => :config
+
+    def with_locale(tmp_locale = nil)
+      if tmp_locale.nil?
+        yield
+      else
+        current_locale = locale
+        self.locale = tmp_locale
+        begin
+          yield
+        ensure
+          self.locale = current_locale
+        end
+      end
+    end
   end
 
   I18nBackend = SgtnClient::I18nBackend

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: singleton-client-test
 version: !ruby/object:Gem::Version
-  version: 0.7.7.5
+  version: 0.7.7.6
 platform: ruby
 authors:
 - VMware G11n Team
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2022-07-20 00:00:00.000000000 Z
+date: 2022-08-30 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: concurrent-ruby
@@ -67,7 +67,7 @@ dependencies:
       - !ruby/object:Gem::Version
         version: '0'
 - !ruby/object:Gem::Dependency
-  name: lumberjack
+  name: logging
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - ">="