Rakefile

@@ -43,7 +43,7 @@ end
 
 #task :default => :test
 
-require 'rdoc/task'
+require 'rake/rdoctask'
 Rake::RDocTask.new do |rdoc|
   if File.exist? 'VERSION'
     version = File.read 'VERSION'

VERSION

@@ -1 +1 @@
-0.0.10
+0.0.4

lib/semlogger.rb

@@ -2,10 +2,7 @@ require 'json'
 
 class Object
 	def to_semlogger
-		[
-			self.class.name.to_sym,
-			self.respond_to?( :serializable_hash) ? self.serializable_hash : self
-		]
+		[self.class.name.to_sym, self.respond_to?( :serializable_hash) ? self.serializable_hash : self ]
 	end
 end
 
@@ -30,30 +27,7 @@ end
 end
 
 class Semlogger < ::Logger
-	class Base
-		class <<self
-			attr_accessor :logger
-		end
-		attr_accessor :logger
-
-		def initialize
-			@logger = self.class.logger
-		end
-
-		def add severity, logger = nil, &block
-			(logger || @logger).add severity, self, &block
-		end
-
-		::Semlogger::Severity.constants.each do |severity|
-			module_eval <<-EOC, __FILE__, __LINE__+1
-				def #{severity.downcase} *a, &e
-					add #{::Semlogger::Severity.const_get severity}, *a, &e
-				end
-			EOC
-		end
-	end
-
-	class CustomType < Base
+	class CustomType
 		def initialize name, *obj
 			@name, @obj = name.to_s.to_sym, obj
 		end
@@ -64,36 +38,17 @@ class Semlogger < ::Logger
 	end
 
 	attr_accessor :logdev, :level, :progname
-
-	# some libs use #log_level
-	def log_level=( level)  @level = level  end
-	def log_level()  @level  end
-
 	class <<self
-		attr_accessor :progname, :logger
+		attr_accessor :progname
 
-		def new_rails_logger config, logdev = nil
-			require 'semlogger/rack'
-			logdev ||= ::Rails.root.join 'log', "#{::Rails.env.to_s.gsub('%', '%%')}.%Y-%m-%d.%$.log"
-			logdev = logdev.to_s
-			logger = nil
-			if Rails.env.production?
-				logger = new logdev
-				logger.level = Semlogger::INFO
-			elsif Rails.env.development?
-				logger = new Semlogger::Multiplex.new( Semlogger::FInfo.new( Semlogger::Printer.new), Semlogger::Writer.new( logdev))
-				logger.level = Semlogger::DEBUG
-			else
-				logger = new logdev
-				logger.level = Semlogger::DEBUG
-			end
-			config.middleware.swap Rails::Rack::Logger, Semlogger::Rack, [], {reqid: :uuid}
-			config.logger = logger
+		def custom *a
+			CustomType.new *a
 		end
-
-		def custom( *a)  CustomType.new( *a).tap {|t| t.logger = self.logger }  end
 	end
-	def custom( *a)  CustomType.new( *a).tap {|t| t.logger = self }  end
+
+	def custom *a
+		CustomType.new *a
+	end
 
 	@@progname = nil
 
@@ -103,21 +58,21 @@ class Semlogger < ::Logger
 		end
 		@progname = a[0] || @@progname
 		@level, @data, @tags, @logdev = DEBUG, {}, [], logdev
-		self.class.logger = self  if !self.class.logger && self.class.logger.is_a?( Semlogger::Default)
 	end
 
-	def tagged *tags, &e
-		@tags += tags.flatten.compact
+	def tagged tags, &e
+		@tags += tags
 		tags = tags.size
 		yield
 	ensure
-		#tags.times { @tags.pop }
-		@tags.slice! -tags .. -1
+		tags.times { @tags.pop }
 	end
 
 	def add severity, message = nil, progname = nil, &block
 		severity ||= UNKNOWN
-		return true  if @logdev.nil? or severity < @level
+		if @logdev.nil? or severity < @level
+			return true
+		end
 		progname ||= @progname
 		if message.nil?
 			if block_given?
@@ -175,11 +130,6 @@ end
 
 require 'semlogger/rotate'
 require 'semlogger/multiplex'
+require 'semlogger/rack'
 require 'semlogger/filter'
 require 'semlogger/writer'
-
-class Semlogger
-	class Default < Semlogger
-	end
-	self.logger ||= Default.new
-end

lib/semlogger/rack.rb

@@ -1,43 +1,29 @@
 class Semlogger::Rack < Rails::Rack::Logger
 	def initialize app, tags = nil, data = nil
 		super app, tags
-		@data = data || {}
+		@data = data
 	end
 
-	def call_app request, env
+	def call_app env
+		request = ActionDispatch::Request.new env
 		path = request.filtered_path
-		Rails.logger.custom( :connection, request.ip, Thread.current.object_id, request.request_method, path).info
+		Rails.logger.info Semlogger.custom :connection, request.ip, Thread.current.object_id, request.request_method, path
 		@app.call env
 	ensure
 		ActiveSupport::LogSubscriber.flush_all!
 	end
 
 	def call env
+		if @data
+			Rails.logger.data( compute_data( env)) { super env }
+		else
+			super env
+		end
+	end
+
+	def compute_data env
 		request = ActionDispatch::Request.new env
-		compute_tagged_ request do
-			compute_data_ request do
-				call_app request, env
-			end
-		end
-	end
 
-	def compute_tagged_ request
-		if Rails.logger.respond_to? :tagged
-			Rails.logger.tagged( compute_tags( request)) { yield }
-		else
-			yield
-		end
-	end
-
-	def compute_data_ request
-		if Rails.logger.respond_to? :data
-			Rails.logger.data( compute_data( request)) { yield }
-		else
-			yield
-		end
-	end
-
-	def compute_data request
 		data = @data.dup
 		data.each do |k, v|
 			case v

lib/semlogger/rotate.rb

@@ -9,7 +9,7 @@ class Semlogger::Rotate
 
 	def open_if
 		name = Time.now.strftime( @filename).gsub /%\$/, $$.to_s
-		close  if @file and name != @file.path
+		@file.close  if @file and name != @file.path
 		@file = File.open name, 'a'  unless @file
 		@file
 	end

lib/semlogger/writer.rb

@@ -3,11 +3,7 @@ end
 
 class Semlogger::Writer < Semlogger::Output
 	def initialize logdev = nil
-		@logdev = case logdev
-			when String then ::Semlogger::Rotate.new logdev
-			when nil then ::Semlogger::Rotate.new "log/#{File.basename $0}.%Y-%m-%d.%$.log"
-			else logdev
-			end
+		@logdev = logdev || ::Semlogger::Rotate.new( ::Rails.root.join( 'log', ::Rails.env).to_s.gsub('%', '%%') + '.%Y-%m-%d.%$.log')
 	end
 
 	def add severity, time, progname, data, tags, message

semlogger.gemspec

@@ -4,13 +4,13 @@
 # -*- encoding: utf-8 -*-
 
 Gem::Specification.new do |s|
-  s.name = "semlogger"
-  s.version = "0.0.10"
+  s.name = %q{semlogger}
+  s.version = "0.0.4"
 
   s.required_rubygems_version = Gem::Requirement.new(">= 0") if s.respond_to? :required_rubygems_version=
   s.authors = ["Denis Knauf"]
-  s.date = "2012-11-21"
-  s.description = "(Semi-)Structured Logger for Ruby (and Rails)"
+  s.date = %q{2012-09-17}
+  s.description = %q{(Semi-)Structured Logger for Ruby (and Rails)}
   s.email = ["Denis.Knauf@gmail.com"]
   s.extra_rdoc_files = [
     "LICENSE",
@@ -30,12 +30,13 @@ Gem::Specification.new do |s|
     "lib/semlogger/tailer.rb",
     "lib/semlogger/writer.rb"
   ]
-  s.homepage = "http://github.com/DenisKnauf/semlogger"
+  s.homepage = %q{http://github.com/DenisKnauf/semlogger}
   s.require_paths = ["lib"]
-  s.rubygems_version = "1.8.23"
-  s.summary = "(Semi-)Structured Logger"
+  s.rubygems_version = %q{1.3.7}
+  s.summary = %q{(Semi-)Structured Logger}
 
   if s.respond_to? :specification_version then
+    current_version = Gem::Specification::CURRENT_SPECIFICATION_VERSION
     s.specification_version = 3
 
     if Gem::Version.new(Gem::VERSION) >= Gem::Version.new('1.2.0') then