log_file_path and http client CartoDB/cartodb-platform/issues/2355

app/models/user/db_service.rb

@@ -983,7 +983,7 @@ module CartoDB
       end
 
       def monitor_user_notification
-        FileUtils.touch(users_modifications_path)
+        FileUtils.touch(log_file_path('users_modifications'))
         if !Cartodb.config[:signups].nil? && !Cartodb.config[:signups]["service"].nil? &&
            !Cartodb.config[:signups]["service"]["port"].nil?
           enable_remote_db_user

lib/carto/configuration.rb

@@ -7,12 +7,8 @@ module Carto::Configuration
     @app_config = YAML.load_file(app_config_file)
   end
 
-  def users_modifications_path
-    log_files_root.join('log', 'users_modifications')
-  end
-
-  def resque_metrics_path
-    "#{log_files_root}/log/resque_metrics.log")
+  def log_file_path(filename)
+    "#{log_files_root}/log/#{filename}")
   end
 
   private

lib/carto/http/response_logger.rb

@@ -1,11 +1,13 @@
 # encoding: utf-8
 
 require 'socket'
+require 'carto/configuration'
 
 module Carto
   module Http
 
     class ResponseLogger
+      include Carto::Configuration
 
       def self.enabled?
         defined?(Rails) && Rails.respond_to?(:root) && Rails.root.present? && Cartodb.config[:http_client_logs]
@@ -30,7 +32,7 @@ module Carto
       end
 
       def logger
-        @@logger ||= Logger.new("#{Rails.root}/log/http_client.log")
+        @@logger ||= Logger.new(log_file_path('http_client.log'))
       end
     end
 

lib/resque/metrics_logger.rb

@@ -3,7 +3,7 @@ require 'carto/configuration'
 module CartoDB
   module ResqueMetrics
     def self.logger
-      @metric_logger ||= ::Logger.new(Carto::Configuration.new.resque_metrics_path)
+      @metric_logger ||= ::Logger.new(Carto::Configuration.new.log_file_path('resque_metrics.log'))
     end
   end
 end