Description of problem: There are few parameters which are currently editable from settings submenu. For example: - General_tab - foreman_url - Auth_tab - oauth_consumer_key - oauth_secret_key - signo_url - ssl_client_dn_env - ssl_client_verify_env - Provisioning_tab - ssl_ca_file - ssl_certificate - ssl_priv_key I think config files are the best place to edit the values of these. Katello-installer itself sets the value of few. So these param shouldn't be user editable. Version-Release number of selected component (if applicable): sat6 GA snap3 How reproducible: Steps to Reproduce: 1. 2. 3. Actual results: Expected results: The above listed parameters which includes some ssl_key stuff or oauth stuff shouldn't be user editable. Additional info:
Settings that are set in /etc/foreman/settings.yml already are read-only, e.g. the oauth_* settings mentioned are showing as read-only here. Moving to the installer, maybe it's deliberately changing the live value so they remain mutable.
Yeah.. oauth_* settings are read-only. Thanks for confirming this.
Any reason why foreman_url and signo_url are editable ? I think these params should also be read-only. thanks.
(In reply to Sachin Ghai from comment #4) > Any reason why foreman_url and signo_url are editable ? I think these params > should also be read-only. thanks. There are many reasons you may need to change it, e.g. deploying the application behind a load balancer, or unusual DNS setups. I don't really follow why the user should *not* be able to update the URL. signo_url is unused here.
(In reply to Dominic Cleal from comment #5) > (In reply to Sachin Ghai from comment #4) > > Any reason why foreman_url and signo_url are editable ? I think these params > > should also be read-only. thanks. > > There are many reasons you may need to change it, e.g. deploying the > application behind a load balancer, or unusual DNS setups. I don't really > follow why the user should *not* be able to update the URL. signo_url is > unused here. thank you for clarifying. I'm just asking/checking which parameters should be editable and why as I'm not 100% sure about all params. This kind of reasoning helps to understand actual use of these params too.
This is a not a bug we are planning to fix. If you disagree, please reopen.