Merge pull request #408 from technicalpickles/logger-should-use-env

Update Gemstsah::Logging.logger to use configured logfile

Author: segiddins

lib/gemstash/config.ru

@@ -4,9 +4,10 @@ require "gemstash"
 require "puma/commonlogger"
 
 use Rack::Deflater
-use Gemstash::Logging::RackMiddleware
 
 use Gemstash::Env::RackMiddleware, Gemstash::Env.current
+use Gemstash::Logging::RackMiddleware
+
 use Gemstash::GemSource::RackMiddleware
 use Gemstash::Health::RackMiddleware
 run Gemstash::Web.new(gemstash_env: Gemstash::Env.current)

lib/gemstash/logging.rb

@@ -12,6 +12,8 @@
 module Gemstash
   # :nodoc:
   module Logging
+    extend Gemstash::Env::Helper
+
     LEVELS = {
       debug: Logger::DEBUG,
       info: Logger::INFO,
@@ -45,7 +47,7 @@ def self.setup_logger(logfile)
     end
 
     def self.logger
-      @logger ||= setup_logger($stdout)
+      @logger ||= setup_logger(gemstash_env.log_file)
     end
 
     def self.reset
@@ -61,6 +63,7 @@ def initialize(app)
 
       def call(env)
         env["rack.logger"] = Gemstash::Logging.logger
+        env["rack.errors"] = Gemstash::Env.current.log_file
         @app.call(env)
       end
     end