betterlog 0.12.0 → 0.14.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 881c76f1faf7c6783210c43f1c4c720d43cfc794da8385c27267080325f01af2
-  data.tar.gz: 6e266178a87eb7c1f526bcb9a70fc13b5bfc5ff642d8b2a63b0bb95e4b43053b
+  metadata.gz: ca69ec750626ef4b6a76e40dcfb19bc1a9334410ee60b2714c611afb21a804c6
+  data.tar.gz: 1ec3c97e0aac9f04452606c1af507c7102aac37033a8c371841435dcd152c6c7
 SHA512:
-  metadata.gz: 3036a50353157aece676091b0f245cbe30976fd707a214863546768bf3a20aa736c016c0e730f7cca11817fb410e293a40f20e2999620e6350b183d531e6d069
-  data.tar.gz: 4abee0552ed87ff53b35f562d5306c8edd24b18e45f4deed25e2b33098b9dea0d57e911f0a4b3914187c94f483557a163d3833ce1e386f1a63fc043b0dc53054
+  metadata.gz: aecbebf1fe2fdb3a9341ac6c0e9704bb8c20161ffb3c76fc11d2de386345a6f8d4d96cd103e97dc115e1b290d9f63144ae130984626e53d87b2b91feadf0b7f3
+  data.tar.gz: 01cab4c0d493f3b9ae5a884338121f1b758838709050e375c8d85a433bae129e9cb7d5c8ccd0664f8afa9a4fe99b3f8ad1513ee6b53f6853b745233cc18795ee

data/Dockerfile

@@ -1,6 +1,4 @@
-ARG BASE_IMAGE
-
-FROM ${BASE_IMAGE} AS builder
+FROM alpine:3.12.0 AS builder
 
 # Update/Upgrade/Add packages for building
 
@@ -20,7 +18,7 @@ RUN go get -u github.com/betterplace/go-init
 
 RUN make fetch all
 
-FROM ${BASE_IMAGE} AS runner
+FROM alpine:3.12.0 AS runner
 
 # Update/Upgrade/Add packages
 

data/Makefile

@@ -1,4 +1,3 @@
-# Pass BASE_IMAGE=alpine:3.13.3
 DOCKER_IMAGE_LATEST = betterlog
 DOCKER_IMAGE = $(DOCKER_IMAGE_LATEST):$(REVISION_SHORT)
 PROJECT_ID = betterplace-183212
@@ -48,15 +47,12 @@ tags: clean
 build-info:
 	@echo $(DOCKER_IMAGE)
 
-pull-base:
-	docker pull $(BASE_IMAGE)
-
-build: pull-base
-	docker build -t $(DOCKER_IMAGE) --build-arg=BASE_IMAGE=$(BASE_IMAGE) .
+build:
+	docker build -t $(DOCKER_IMAGE) .
 	$(MAKE) build-info
 
-build-force: pull-base
-	docker build -t $(DOCKER_IMAGE) --build-arg=BASE_IMAGE=$(BASE_IMAGE) --no-cache .
+build-force:
+	docker build -t $(DOCKER_IMAGE) --no-cache .
 	$(MAKE) build-info
 
 debug:

data/VERSION

@@ -1 +1 @@
-0.12.0
+0.14.0

data/betterlog.gemspec

@@ -1,14 +1,14 @@
 # -*- encoding: utf-8 -*-
-# stub: betterlog 0.12.0 ruby lib
+# stub: betterlog 0.14.0 ruby lib
 
 Gem::Specification.new do |s|
   s.name = "betterlog".freeze
-  s.version = "0.12.0"
+  s.version = "0.14.0"
 
   s.required_rubygems_version = Gem::Requirement.new(">= 0".freeze) if s.respond_to? :required_rubygems_version=
   s.require_paths = ["lib".freeze]
   s.authors = ["betterplace Developers".freeze]
-  s.date = "2020-04-02"
+  s.date = "2020-07-30"
   s.description = "This library provides structure json logging for our rails projects".freeze
   s.email = "developers@betterplace.org".freeze
   s.executables = ["betterlog".freeze, "betterlog_pusher".freeze, "betterlog_sink".freeze]

data/bin/betterlog_sink

@@ -7,14 +7,14 @@ require 'json'
 
 LOG_ENV = ENV.fetch('LOG_ENV', 'production')
 
+@config = complex_config.log(LOG_ENV).sink
+
 ENV['KUBECONFIG'] = File.join(
   ENV.fetch('HOME'),
   '.kube',
-  "betterlog-#{LOG_ENV}"
+  @config.cluster.name
 )
 
-@config = complex_config.log(LOG_ENV).sink
-
 def authenticate
   return if File.exist?(ENV['KUBECONFIG'])
   context =  [

data/cloudbuild.yaml

@@ -1,8 +1,6 @@
 steps:
 - name: eu.gcr.io/betterplace-183212/better-builder
-  args: ['pull', '${_BASE_IMAGE}']
-- name: eu.gcr.io/betterplace-183212/better-builder
-  args: ['build', '-t', 'eu.gcr.io/$PROJECT_ID/${_IMAGE_NAME}:$SHORT_SHA', '-t', 'eu.gcr.io/$PROJECT_ID/${_IMAGE_NAME}', '--build-arg=BASE_IMAGE=${_BASE_IMAGE}', '.']
+  args: ['build', '-t', 'eu.gcr.io/$PROJECT_ID/${_IMAGE_NAME}:$SHORT_SHA', '-t', 'eu.gcr.io/$PROJECT_ID/${_IMAGE_NAME}', '.']
 images:
   - 'eu.gcr.io/$PROJECT_ID/${_IMAGE_NAME}:$SHORT_SHA'
   - 'eu.gcr.io/$PROJECT_ID/${_IMAGE_NAME}'

data/lib/betterlog.rb

@@ -2,6 +2,7 @@ require 'tins/xt'
 require 'json'
 require 'logger'
 require 'time'
+require 'complex_config'
 require 'term/ansicolor'
 
 module Betterlog

data/lib/betterlog/log.rb

@@ -7,12 +7,16 @@ require 'betterlog/log/severity'
 module Betterlog
   class Log
     include Tins::SexySingleton
+    extend ComplexConfig::Provider::Shortcuts
 
     class_attr_accessor :default_logger
     self.default_logger = Logger.new(STDERR)
+    if level = cc.log.level?
+      default_logger.level = level
+    end
 
     def logger
-      defined?(Rails) && Rails.respond_to?(:logger) ? Rails.logger : ::Logger.new(STDERR)
+      defined?(Rails) && Rails.respond_to?(:logger) ? Rails.logger : self.class.default_logger
     end
 
     # Logs a message on severity info.
@@ -98,7 +102,7 @@ module Betterlog
       result = time_block(&block)
     rescue => error
       e = Log::Event.ify(error)
-      rest |= e.as_hash.subhash(:error_class, :backtrace, :message)
+      rest |= e.as_json.subhash(:error_class, :backtrace, :message)
       rest[:message] = "#{rest[:message].inspect} while measuring metric #{name}"
       raise error
     ensure
@@ -125,6 +129,21 @@ module Betterlog
       instance.context(data_hash)
     end
 
+    def emit(event)
+      l = caller_locations.reverse_each.each_cons(3).find { |c, n1, n2|
+        n2.absolute_path =~ /betterlog\/log\.rb/ and break c # TODO check if this still works
+      }
+      if l
+        event[:location] = [ l.absolute_path, l.lineno ] * ?:
+      end
+      event[:emitter] = self.class.name.downcase
+      notify(event)
+      logger.send(event.severity.to_sym, event.to_json)
+      self
+    ensure
+      GlobalMetadata.data.clear
+    end
+
     private
 
     def protect
@@ -158,21 +177,6 @@ module Betterlog
       )
     end
 
-    def emit(event)
-      l = caller_locations.reverse_each.each_cons(3).find { |c, n1, n2|
-        n2.absolute_path =~ /betterlog\/log\.rb/ and break c # TODO check if this still works
-      }
-      if l
-        event[:location] = [ l.absolute_path, l.lineno ] * ?:
-      end
-      event[:emitter] = self.class.name.downcase
-      notify(event)
-      logger.send(event.severity.to_sym, event.to_json)
-      self
-    ensure
-      GlobalMetadata.data.clear
-    end
-
     def notify(event)
       if event.notify?
         Notifiers.notify(event)

data/lib/betterlog/log/event.rb

@@ -70,12 +70,12 @@ module Betterlog
         @data = Hash[data.sort_by(&:first)]
       end
 
-      def as_hash(*a)
+      def as_json(*a)
         @data.dup
       end
 
       def to_json(*a)
-        as_hash.to_json(*a)
+        as_json.to_json(*a)
       end
 
       def format(*args)

data/lib/betterlog/log/event_formatter.rb

@@ -2,6 +2,7 @@ module Betterlog
   class Log
     class EventFormatter
       include Term::ANSIColor
+      include ComplexConfig::Provider::Shortcuts
 
       def initialize(event)
         @event = event
@@ -52,7 +53,8 @@ module Betterlog
             invisible = $1.full?
             directive = $2
             key       = $3
-            if value = @event[key]
+            value     = @event[key]
+            unless value.nil?
               formatted_value =
                 if directive
                   case directive

data/lib/betterlog/log_event_formatter.rb

@@ -1,6 +1,7 @@
 module Betterlog
   class LogEventFormatter < ActiveSupport::Logger::Formatter
     include ActiveSupport::TaggedLogging::Formatter
+    include ComplexConfig::Provider::Shortcuts
 
     def emitter
       'legacy'

data/lib/betterlog/logger.rb

@@ -2,9 +2,15 @@ require 'redis'
 
 module Betterlog
   class Logger < ::Logger
+    include ComplexConfig::Provider::Shortcuts
+
     def initialize(redis, shift_age = 0, shift_size = 1048576, name: nil, buffer_size: nil, **opts)
       @redis       = redis
       @fallback    = ::Logger.new(STDERR)
+      if level = cc.log.level?
+        self.level      = level
+        @fallback.level = level
+      end
       @name        = name || self.class.name
       @buffer_size = determine_buffer_size(buffer_size)
       super(nil, shift_age, shift_size, **opts)
@@ -83,7 +89,7 @@ module Betterlog
       rescue Redis::BaseConnectionError
       end
 
-      @redis.exists(@name) or return Enumerator.new {}
+      @redis.exists?(@name) or return Enumerator.new {}
 
       Enumerator.new do |y|
         name_tmp = "#{@name}_#{rand}"
@@ -92,7 +98,9 @@ module Betterlog
         s = 0
         e = @redis.strlen(name_tmp) - 1
         until s > e
-          y.yield @redis.getrange(name_tmp, s, s + chunk_size - 1)
+          range = @redis.getrange(name_tmp, s, s + chunk_size - 1)
+          range.force_encoding 'ASCII-8BIT'
+          y.yield range
           s += chunk_size
         end
 
@@ -115,9 +123,10 @@ module Betterlog
       chunk_size > 0 or raise ArgumentError, 'chunk_size > 0 required'
       Enumerator.new do |y|
         buffer = ''
+        buffer.encode! 'ASCII-8BIT'
         each_chunk(chunk_size: chunk_size) do |chunk|
           buffer << chunk
-          buffer.gsub!(/\A(.*?#$/)/) do |line|
+          buffer.gsub!(/\A(.*?#$/)/n) do |line|
             y.yield(line)
             ''
           end

data/lib/betterlog/notifiers.rb

@@ -14,7 +14,7 @@ module Betterlog
 
     def self.notify(event)
       notifiers.each do |notifier|
-        notifier.notify(event.notify?, event.as_hash)
+        notifier.notify(event.notify?, event.as_json)
       end
     end
 

data/lib/betterlog/version.rb

@@ -1,6 +1,6 @@
 module Betterlog
   # Betterlog version
-  VERSION         = '0.12.0'
+  VERSION         = '0.14.0'
   VERSION_ARRAY   = VERSION.split('.').map(&:to_i) # :nodoc:
   VERSION_MAJOR   = VERSION_ARRAY[0] # :nodoc:
   VERSION_MINOR   = VERSION_ARRAY[1] # :nodoc:

data/spec/betterlog/log/event_spec.rb

@@ -1,10 +1,10 @@
 require 'spec_helper'
 
 describe Betterlog::Log::Event do
-  describe '#as_hash' do
+  describe '#as_json' do
     it 'dups hash before returning it' do
       event = described_class.new
-      expect(event.as_hash).not_to equal event.instance_variable_get(:@data)
+      expect(event.as_json).not_to equal event.instance_variable_get(:@data)
     end
   end
 

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: betterlog
 version: !ruby/object:Gem::Version
-  version: 0.12.0
+  version: 0.14.0
 platform: ruby
 authors:
 - betterplace Developers
-autorequire: 
+autorequire:
 bindir: bin
 cert_chain: []
-date: 2020-04-02 00:00:00.000000000 Z
+date: 2020-07-30 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: gem_hadar
@@ -248,7 +248,7 @@ files:
 homepage: http://github.com/betterplace/betterlog
 licenses: []
 metadata: {}
-post_install_message: 
+post_install_message:
 rdoc_options:
 - "--title"
 - Betterlog
@@ -268,7 +268,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
       version: '0'
 requirements: []
 rubygems_version: 3.1.2
-signing_key: 
+signing_key:
 specification_version: 4
 summary: Structured logging support for bp
 test_files: