diff options
| author | Mathieu Parent <math.parent@gmail.com> | 2019-07-25 16:10:30 +0300 |
|---|---|---|
| committer | Mathieu Parent <math.parent@gmail.com> | 2019-09-09 16:14:52 +0300 |
| commit | 85776fa3ffba6f641cf981cb0107f0e4ba882f3e (patch) | |
| tree | dd571294c8ee15639c371c1a0f6a71799eea632e /app/controllers | |
| parent | ea14d17da4d3625eaf6a70a3b38660e5753c5f77 (diff) | |
Settings API: domain_{black,white}list should be arrays
As in documentation.
Fixes: #58180.
Also remove the requirement between domain_blacklist_enabled and domain_blacklist.
Diffstat (limited to 'app/controllers')
| -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 53ceffd6bea..f2f72bea5b4 100644 --- a/app/controllers/admin/application_settings_controller.rb +++ b/app/controllers/admin/application_settings_controller.rb @@ -85,6 +85,7 @@ class Admin::ApplicationSettingsController < Admin::ApplicationController params[:application_setting][:import_sources]&.delete("") params[:application_setting][:restricted_visibility_levels]&.delete("") + # TODO Remove domain_blacklist_raw in APIv5 (See https://gitlab.com/gitlab-org/gitlab-ce/issues/67204) params.delete(:domain_blacklist_raw) if params[:domain_blacklist_file] params.delete(:domain_blacklist_raw) if params[:domain_blacklist] params.delete(:domain_whitelist_raw) if params[:domain_whitelist] |