Project

General

Profile

Actions

Bug #14364

closed

no validation for default_download_policy setting value

Added by Partha Aji about 8 years ago. Updated almost 6 years ago.

Status:
Closed
Priority:
Normal
Assignee:
Category:
Repositories
Target version:
Difficulty:
Triaged:
Fixed in Releases:
Found in Releases:

Description

Cloned from https://bugzilla.redhat.com/show_bug.cgi?id=1321045
Description of problem:
No validation at all applies to default_download_policy setting, so user can input whatever value.

```
  1. hammer -u admin -p changeme settings set --name default_download_policy --value they-told-me-i-could-be-anything-i-wanted-so-i-became-a-download-policy-value
    Setting [default_download_policy] updated to [they-told-me-i-could-be-anything-i-wanted-so-i-became-a-download-policy-value]
  2. hammer -u admin -p changeme settings list | grep policy
    default_download_policy | they-told-me-i-could-be-anything-i-wanted-so-i-became-a-download-policy-value | Default download policy for repositories (either 'immediate', 'on_demand', or...

```

- applies to UI and API as well.

Version-Release number of selected component (if applicable):
Satellite 6.2.0 Snap 5.1

How reproducible:

Actual results:
any value (including Null) accepted

Expected results:
should accept only: 'immediate', 'on_demand', or 'background'

Additional info:


Related issues 1 (0 open1 closed)

Related to Katello - Feature #14366: Show a select in the UI for the different download policy options on the Settings pageDuplicateDavid Davis03/28/2016Actions
Actions #1

Updated by The Foreman Bot about 8 years ago

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

Updated by David Davis about 8 years ago

  • Related to Feature #14366: Show a select in the UI for the different download policy options on the Settings page added
Actions #3

Updated by David Davis about 8 years ago

  • Status changed from Ready For Testing to Closed
  • % Done changed from 0 to 100
Actions #4

Updated by Eric Helms about 8 years ago

  • translation missing: en.field_release set to 86
Actions

Also available in: Atom PDF