ensure useragent is not null

app/controllers/application_controller.rb

@@ -73,7 +73,7 @@ class ApplicationController < ActionController::Base
     APIUsage.create!(
       user_id:    CurrentUser.is_anonymous? ? nil : CurrentUser.id,
       api_key:    @apikey.nil? || @apikey.is_anon? ? nil : @apikey,
-      user_agent: request.headers["user-agent"],
+      user_agent: request.headers["user-agent"] || "",
       method:     request.method,
       path:       request.path,
       params:     request.params.to_json,

app/controllers/exceptions_controller.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 class ExceptionsController < ApplicationController
   def index
     @pagy, @exceptions = pagy(ExceptionLog.order(id: :desc), items: 25)

app/logical/pool_index.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module PoolIndex
   module_function
 

app/models/api_image.rb

@@ -35,7 +35,7 @@ class APIImage < ApplicationRecord
     invalidate_cache
   end
 
-  def  file_header_info(file_path)
+  def file_header_info(file_path)
     File.open(file_path) do |bin|
       mime_type = Marcel::MimeType.for(bin)
       ext = case mime_type

app/models/e621/pool.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module E621
   class Pool < ApplicationRecord
     self.table_name = "e621.pools"

app/models/e621/post.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module E621
   class Post < ApplicationRecord
     self.table_name = "e621.posts"

app/models/e621/tag.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module E621
   class Tag < ApplicationRecord
     self.table_name = "e621.tags"

app/models/e621/tag_alias.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module E621
   class TagAlias < ApplicationRecord
     self.table_name = "e621.tag_aliases"

app/models/e621/tag_implication.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module E621
   class TagImplication < ApplicationRecord
     self.table_name = "e621.tag_implications"

app/models/e621/wiki_page.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module E621
   class WikiPage < ApplicationRecord
     self.table_name = "e621.wiki_pages"

app/models/exception_log.rb

@@ -8,7 +8,7 @@ class ExceptionLog < ApplicationRecord
       host:       Socket.gethostname,
       params:     request.filtered_parameters,
       referrer:   request.referrer,
-      user_agent: request.user_agent,
+      user_agent: request.user_agent || "",
     }
 
     # Required to unwrap exceptions that occur inside template rendering.

config/initializers/cache_store.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 def cache_store
   [:memory_store, { size: 64.megabytes }]
 end

config/initializers/git_helper.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
-#
+
 Rails.configuration.to_prepare do
   GitHelper.init
 end

db/migrate/20240603083428_create_e621_export_tables.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 class CreateE621ExportTables < ActiveRecord::Migration[7.1]
   def change
     reversible do |r|

db/migrate/20240604105520_add_e621_pools_tag_string.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 class AddE621PoolsTagString < ActiveRecord::Migration[7.1]
   def change
     add_column(:"e621.pools", :tag_string, :string, null: false, default: "")