Project

General

Profile

Bug #28386

hammer settings content_default_http_proxy does not have validation for provided values

Added by Oleh Fedorenko about 2 months ago. Updated about 2 months ago.

Status:
Closed
Priority:
Normal
Category:
Settings
Target version:
-
Difficulty:
easy
Triaged:
Yes
Bugzilla link:
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:

Associated revisions

Revision 23db0e06 (diff)
Added by Oleh Fedorenko about 2 months ago

Fixes #28386 - Validate default_http_proxy setting for API/hammer

History

#1 Updated by Oleh Fedorenko about 2 months ago

  • Triaged changed from No to Yes
  • Difficulty set to easy
  • Assignee set to Oleh Fedorenko
  • Category changed from API to Settings

#2 Updated by The Foreman Bot about 2 months ago

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

#3 Updated by The Foreman Bot about 2 months ago

  • Fixed in Releases Katello 3.15.0 added

#4 Updated by Oleh Fedorenko about 2 months ago

  • Status changed from Ready For Testing to Closed

Also available in: Atom PDF