16826: Fixes the bug.
authorLucas Di Pentima <lucas@di-pentima.com.ar>
Fri, 11 Sep 2020 21:44:20 +0000 (18:44 -0300)
committerLucas Di Pentima <lucas@di-pentima.com.ar>
Fri, 11 Sep 2020 21:44:20 +0000 (18:44 -0300)
The issue is that the config loading code on Rails uses
ActiveSupport::OrderedOptions on config hashes, and this class always
returns keys as symbols. We were incorrectly testing the feature by
assigning a normal Hash to the config.

Arvados-DCO-1.1-Signed-off-by: Lucas Di Pentima <lucas@di-pentima.com.ar>

services/api/app/models/arvados_model.rb

index 6fb8ff2b33549af8e4e512a1374363f8dee8fa64..709b4b4d1c1304e456c494a4079ca2ea0cccc8fb 100644 (file)
@@ -454,7 +454,7 @@ class ArvadosModel < ApplicationRecord
   end
 
   def logged_attributes
-    attributes.except(*Rails.configuration.AuditLogs.UnloggedAttributes.keys)
+    attributes.except(*Rails.configuration.AuditLogs.UnloggedAttributes.stringify_keys.keys)
   end
 
   def self.full_text_searchable_columns