Improve logging (#274)

* Add some custom log messages
* Add original URL and remote IP to each log entry

app/controllers/application_controller.rb

@@ -58,6 +58,8 @@ class ApplicationController < ActionController::Base
   private
 
     def user_not_authorized
+      logger.error { "User not authorized: #{user_signed_in? ? current_user.inspect : 'unlogged user'}" }
+
       render json: {
         error: t('errors.unauthorized')
       }, status: :unauthorized

app/controllers/tenants_controller.rb

@@ -47,9 +47,13 @@ class TenantsController < ApplicationController
 
       CreateWelcomeEntitiesWorkflow.new().run
 
+      logger.info { "New tenant registration: #{Current.tenant.inspect}" }
+
       render json: @tenant, status: :created
 
     rescue ActiveRecord::RecordInvalid => exception
+      logger.error { "Error in tenant registration: #{exception}" }
+
       render json: { error: exception }, status: :unprocessable_entity
     end
   end

app/workflows/o_auth_exchange_auth_code_for_profile_workflow.rb

@@ -49,7 +49,8 @@ class OAuthExchangeAuthCodeForProfileWorkflow
 
       return profile_response
     rescue => error
-      print(error)
+      logger.error { "Error in OAuthExchangeAuthCodeForProfileWorkflow: #{error}, o_auth: #{@o_auth.inspect}" }
+
       return nil
     end
   end

app/workflows/o_auth_sign_in_user_workflow.rb

@@ -52,7 +52,7 @@ class OAuthSignInUserWorkflow
 
       return user
     rescue => error
-      print(error)
+      logger.error { "Error in OAuthSignInUserWorkflow: #{error}, o_auth: #{@o_auth.inspect}" }
       return nil
     end
   end

config/environments/production.rb

@@ -51,7 +51,7 @@ Rails.application.configure do
   config.log_level = :debug
 
   # Prepend all log lines with the following tags.
-  config.log_tags = [ :request_id ]
+  config.log_tags = [ :request_id, :original_url, :remote_ip ]
 
   # Use a different cache store in production.
   # config.cache_store = :mem_cache_store