Project

General

Profile

Actions

Bug #28386

closed

hammer settings content_default_http_proxy does not have validation for provided values

Added by Oleh Fedorenko over 4 years ago. Updated about 3 years ago.

Status:
Closed
Priority:
Normal
Category:
-
Target version:
Difficulty:
easy
Triaged:
Yes
Fixed in Releases:
Found in Releases:

Description

Cloned from https://bugzilla.redhat.com/show_bug.cgi?id=1777730

Description of problem:

hammer setting content_default_http_proxy does not have validation for provided values

Version-Release number of selected component (if applicable):

@Satellite 6.7.0 snap 3,0

How reproducible:

Always

Steps to Reproduce:
1. Set settings as below:

[root@hp-dl180-04 ~]# hammer settings set --name content_default_http_proxy --value xyz
Setting [content_default_http_proxy] updated to [xyz].

[root@hp-dl180-04 ~]# hammer settings list |grep -i content_default_http_proxy
content_default_http_proxy | Default http proxy | xyz | Default HTTP Proxy for syncing content
[root@hp-dl180-04 ~]#

2. No check on UI for same setting , the value filed shows empty.

Administer => Settings => Content => 'Default HTTP Proxy'

Actual results:

Any value could pass with '--value'

Expected results:

It should only set the setting if provided value is correct (i.e. already created proxy name)

Additional info:


Subtasks 1 (0 open1 closed)

Bug #31923: unable to set "Default HTTP Proxy" field to "no global default".'DuplicateActions
Actions #1

Updated by Oleh Fedorenko over 4 years ago

  • Category changed from API to 98
  • Assignee set to Oleh Fedorenko
  • Difficulty set to easy
  • Triaged changed from No to Yes
Actions #2

Updated by The Foreman Bot over 4 years ago

  • Status changed from New to Ready For Testing
  • Pull request https://github.com/Katello/katello/pull/8456 added
Actions #3

Updated by The Foreman Bot over 4 years ago

  • Fixed in Releases Katello 3.15.0 added
Actions #4

Updated by Oleh Fedorenko over 4 years ago

  • Status changed from Ready For Testing to Closed
Actions #5

Updated by Jonathon Turel about 4 years ago

  • Target version set to Katello 3.15.0
Actions

Also available in: Atom PDF