Edge case: Very long FBOS version names

FarmBot · Aug 28, 2021 · 248741b · 248741b
1 parent 39dca16
commit 248741b
Show file tree

Hide file tree

Showing 2 changed files with 1 addition and 2 deletions.
diff --git a/app/controllers/api/abstract_controller.rb b/app/controllers/api/abstract_controller.rb
@@ -261,7 +261,7 @@ def mark_as_seen(bot = (current_user && current_user.device))
           bot.last_saw_api = Time.now
           # Do _not_ set the FBOS version to 0.0.0 if the UA header is missing.
           if v > NULL && v < NOT_FBOS
-            bot.fbos_version = v.to_s
+            bot.fbos_version = (v.to_s || "").first(14)
             bot.save!
           end
         end

diff --git a/app/controllers/api/rmq_utils_controller.rb b/app/controllers/api/rmq_utils_controller.rb
@@ -186,7 +186,6 @@ def do_rate_limit
     end
 
     def deny(reason)
-      puts "=== DENIED: #{reason}"
       maybe_alert_user(reason)
       render json: "deny", status: 403
     end