Merge pull request #183 from datacite/investigate-500-errors

fix zeitwerk load errors

README.md

@@ -37,6 +37,12 @@ We use Rspec for unit and acceptance testing:
 bundle exec rspec
 ```
 
+To run the Rails console:
+
+```
+DISABLE_SPRING=true bundle exec rails console
+```
+
 ## Technical Description
 
 ### Resource components

app/controllers/application_controller.rb

@@ -79,6 +79,7 @@ def transform_params
     elsif status == 409
       message = "The resource already exists."
     else
+      Raven.capture_exception(exception)
       message = exception.message
     end
 

config/application.rb

@@ -55,12 +55,11 @@ module Sashimi
   class Application < Rails::Application
     # Initialize configuration defaults for originally generated Rails version.
     config.load_defaults 7.1
-    config.autoload_paths << Rails.root.join('lib')
+    config.autoload_once_paths << Rails.root.join('lib')
     config.autoload_paths << Rails.root.join("app", "models", "concerns")
-    config.autoload_paths += %W(#{config.root}/lib #{config.root}/lib/middleware)
 
-    # Allow middleware to be loaded. (compressed_requests)
-    config.autoload_lib(ignore: nil)
+    config.eager_load_paths << Rails.root.join("lib")
+    config.eager_load_paths << Rails.root.join("app", "models", "concerns")
 
     # Settings in config/environments/* take precedence over those specified here.
     # Application configuration should go into files in config/initializers
@@ -76,8 +75,8 @@ class Application < Rails::Application
     config.lograge.enabled = true
     config.lograge.formatter = Lograge::Formatters::Logstash.new
     config.lograge.logger = LogStashLogger.new(type: :stdout)
-    config.logger = config.lograge.logger        ## LogStashLogger needs to be pass to rails logger, see roidrage/lograge#26
-    config.log_level = ENV["LOG_LEVEL"].to_sym   ## Log level in a config level configuration
+    config.logger = config.lograge.logger
+    config.log_level = ENV["LOG_LEVEL"].to_sym
 
     config.lograge.ignore_actions = ["HeartbeatController#index", "IndexController#index"]
     config.lograge.ignore_custom = lambda do |event|
@@ -93,6 +92,8 @@ class Application < Rails::Application
       }
     end
 
+    config.action_dispatch.debug_exception_log_level = :error
+
     # configure caching
     config.cache_store = :mem_cache_store, ENV["MEMCACHE_SERVERS"], { namespace: ENV["APPLICATION"] }
 

config/environments/development.rb

@@ -66,6 +66,8 @@
     url: "/report_files/:filename",
     use_timestamp: false,
   }
+
+  config.action_dispatch.show_exceptions = false
 end
 
 #BetterErrors::Middleware.allow_ip! ENV['TRUSTED_IP']

config/initializers/middlewares.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require "./lib/middleware/compressed_requests"
 
 Rails.application.configure do

lib/middleware/compressed_requests.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module Middleware
   class CompressedRequests
     def initialize(app)