FIX: catch specific exception

discourse · Aug 6, 2014 · b573b32 · eviltrout · Aug 6, 2014 · b573b32
1 parent 9dc96d1
commit b573b32
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 3 deletions.
diff --git a/app/models/incoming_domain.rb b/app/models/incoming_domain.rb
@@ -11,7 +11,7 @@ def self.add!(uri)
 
     begin
       current = create!(name: name, https: https, port: port)
-    rescue
+    rescue ActiveRecord::RecordNotUnique
       # duplicate key is just ignored
     end
 

diff --git a/app/models/incoming_referer.rb b/app/models/incoming_referer.rb
@@ -11,8 +11,8 @@ def self.add!(opts)
 
     begin
       current = create!(path: path, incoming_domain_id: domain_id)
-    rescue
-      # duplicates
+    rescue ActiveRecord::RecordNotUnique
+      # does not matter
     end
 
     current || find_by(path: path, incoming_domain_id: domain_id)