Allow alternative log level and break app on user not saved (#689)

* Allow RAILS_LOG_LEVEL to define alternative log level in production * Force server error when a user cannot be saved
pil0u · Nov 20, 2024 · 6ccfb5e · 6ccfb5e
1 parent d1ed082
commit 6ccfb5e
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 2 deletions.
diff --git a/app/models/user.rb b/app/models/user.rb
@@ -59,7 +59,7 @@ def self.from_kitt(auth)
     user.github_username = auth.info.github_nickname
     user.original_city_id = original_city.id
 
-    user.save
+    user.save!
     user
   end
 

diff --git a/config/environments/production.rb b/config/environments/production.rb
@@ -47,7 +47,7 @@
 
   # Include generic and useful information about system operation, but avoid logging too much
   # information to avoid inadvertent exposure of personally identifiable information (PII).
-  config.log_level = :info
+  config.log_level = ENV.fetch("RAILS_LOG_LEVEL", :info)
 
   # Prepend all log lines with the following tags.
   config.log_tags = [:request_id]