Project

General

Profile

Actions

Bug #31923

closed

Bug #28386: hammer settings content_default_http_proxy does not have validation for provided values

unable to set "Default HTTP Proxy" field to "no global default".'

Added by Marek Hulán about 3 years ago. Updated about 3 years ago.

Status:
Duplicate
Priority:
Normal
Assignee:
-
Category:
Repositories
Target version:
Difficulty:
Triaged:
Yes
Fixed in Releases:
Found in Releases:

Description

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

Description of problem:
unable to set "Default HTTP Proxy" field to "no global default".

Version-Release number of selected component (if applicable):
- Satellite 6.9.0 snap 9

How reproducible:
- Always

Steps to Reproduce:
1. Create a Http Proxy (Go to Infrastructure > Http Proxies)
2. Go to Administer > Settings > Content tab
3. Set "Default HTTP Proxy"
4. Now try to set "Default HTTP Proxy" to "no global default".

Actual results:
Oh no! Something went wrong while submitting the form, the server returned the following error: Error: Request failed with status code 422

Expected results:
- User is able to set "Default HTTP Proxy" to "no global default".

Additional info:

2021-01-20T03:42:34 [E|app|9a9bad14] Unprocessable entity Setting::Content (id: 198):
9a9bad14 | There is no such HTTP Proxy
9a9bad14 |
2021-01-20T03:42:34 [I|app|9a9bad14] Rendering api/v2/errors/unprocessable_entity.json.rabl within api/v2/layouts/error_layout
2021-01-20T03:42:34 [I|app|9a9bad14] Rendered api/v2/errors/unprocessable_entity.json.rabl within api/v2/layouts/error_layout (Duration: 4.8ms | Allocations: 6575)
2021-01-20T03:42:34 [D|app|9a9bad14] Body: {
9a9bad14 | "error": {"id":198,"errors":{"base":["There is no such HTTP Proxy"]},"full_messages":["There is no such HTTP Proxy"]}
9a9bad14 | }
9a9bad14 |
2021-01-20T03:42:34 [I|app|9a9bad14] Completed 422 Unprocessable Entity in 26ms (Views: 8.4ms | ActiveRecord: 3.1ms | Allocations: 18645)

Workaround:
- Use hammer CLI to unset content_default_http_proxy
hammer> settings set --name content_default_http_proxy --value ""'

Actions #1

Updated by The Foreman Bot about 3 years ago

  • Status changed from New to Ready For Testing
  • Assignee set to Marek Hulán
  • Pull request https://github.com/Katello/katello/pull/9183 added
Actions #2

Updated by Marek Hulán about 3 years ago

  • Subject changed from unable to set "Default HTTP Proxy" field to "no global default".' to unable to set "Default HTTP Proxy" field to "no global default".'
  • Category deleted (98)
  • Status changed from Ready For Testing to New
  • Assignee deleted (Marek Hulán)

Ok I give up, the problem is that the value coming from the form is not empty string, it's the actual `no global default` string. Therefore the detection fails. `:include_blank` no longer sets empty string as a value for this select <option>. Settings is now in react and I didn't find the proper place to fix it. At this point I'm not sure if this needs to be fixed in core or Katello, the only core setting with the similar logic is the host owner where it seems to work fine. So leaving in Katello

Actions #3

Updated by Marek Hulán about 3 years ago

Ah, this seems to be fixed in upstream already. Seems to be originally introduced by #28386 but there's no linked fix.

Actions #4

Updated by Jonathon Turel about 3 years ago

  • Category set to Repositories
  • Target version set to Katello 4.1.0
  • Triaged changed from No to Yes
Actions #5

Updated by Chris Roberts about 3 years ago

  • Status changed from New to Duplicate
  • Target version changed from Katello 4.1.0 to Katello Recycle Bin
  • Parent task set to #28386
Actions

Also available in: Atom PDF