Alias secret_key and secret_value to key and value

app/controllers/groups/variables_controller.rb

@@ -35,12 +35,7 @@ module Groups
     end
 
     def group_variables_params
-      filtered_params = params.permit(variables_attributes: [*variable_params_attributes])
-      filtered_params["variables_attributes"].each do |variable|
-        variable["key"] = variable.delete("secret_key")
-        variable["value"] = variable.delete("secret_value")
-      end
-      filtered_params
+      params.permit(variables_attributes: [*variable_params_attributes])
     end
 
     def variable_params_attributes

app/controllers/projects/variables_controller.rb

@@ -32,12 +32,7 @@ class Projects::VariablesController < Projects::ApplicationController
   end
 
   def variables_params
-    filtered_params = params.permit(variables_attributes: [*variable_params_attributes])
-    filtered_params["variables_attributes"].each do |variable|
-      variable["key"] = variable.delete("secret_key")
-      variable["value"] = variable.delete("secret_value")
-    end
-    filtered_params
+    params.permit(variables_attributes: [*variable_params_attributes])
   end
 
   def variable_params_attributes

app/models/ci/group_variable.rb

@@ -6,6 +6,9 @@ module Ci
 
     belongs_to :group
 
+    alias_attribute :secret_key, :key
+    alias_attribute :secret_value, :value
+
     validates :key, uniqueness: {
       scope: :group_id,
       message: "(%{value}) has already been taken"

app/models/ci/variable.rb

@@ -6,6 +6,9 @@ module Ci
 
     belongs_to :project
 
+    alias_attribute :secret_key, :key
+    alias_attribute :secret_value, :value
+
     validates :key, uniqueness: {
       scope: [:project_id, :environment_scope],
       message: "(%{value}) has already been taken"