diff options
author | GitLab Release Tools Bot <robert+release-tools@gitlab.com> | 2019-11-26 12:01:11 +0000 |
---|---|---|
committer | GitLab Release Tools Bot <robert+release-tools@gitlab.com> | 2019-11-26 12:01:11 +0000 |
commit | f83a4042c999aa0ebfd226e51855721df6da1552 (patch) | |
tree | eb66ac012fd3093cfe34f1bcbb234831d5aa31b8 | |
parent | 2e2b53ead7b2f8977b79871178da68e80e0e4a0a (diff) | |
parent | ab2801c4fd6282ff59afd98807454aafb61ea474 (diff) | |
download | gitlab-ce-f83a4042c999aa0ebfd226e51855721df6da1552.tar.gz |
Merge branch 'security-aws-secret-key-2937-ce-12-3' into '12-3-stable'
Hide AWS secret on Admin Integration page
See merge request gitlab/gitlabhq!3527
-rw-r--r-- | app/controllers/admin/application_settings_controller.rb | 1 |
1 files changed, 1 insertions, 0 deletions
diff --git a/app/controllers/admin/application_settings_controller.rb b/app/controllers/admin/application_settings_controller.rb index fbf63997b15..bf8b38ddd34 100644 --- a/app/controllers/admin/application_settings_controller.rb +++ b/app/controllers/admin/application_settings_controller.rb @@ -86,6 +86,7 @@ class Admin::ApplicationSettingsController < Admin::ApplicationController params[:application_setting][:import_sources]&.delete("") params[:application_setting][:restricted_visibility_levels]&.delete("") + params[:application_setting].delete(:elasticsearch_aws_secret_access_key) if params[:application_setting][:elasticsearch_aws_secret_access_key].blank? # TODO Remove domain_blacklist_raw in APIv5 (See https://gitlab.com/gitlab-org/gitlab-foss/issues/67204) params.delete(:domain_blacklist_raw) if params[:domain_blacklist_file] params.delete(:domain_blacklist_raw) if params[:domain_blacklist] |