Bug 1402881 - Mandatory options in kickstart advanced options accepted even if empty
Summary: Mandatory options in kickstart advanced options accepted even if empty
Keywords:
Status: CLOSED DUPLICATE of bug 1357824
Alias: None
Product: Red Hat Satellite 5
Classification: Red Hat
Component: WebUI
Version: 570
Hardware: Unspecified
OS: Unspecified
unspecified
low
Target Milestone: ---
Assignee: Grant Gainey
QA Contact: Red Hat Satellite QA List
URL:
Whiteboard:
Depends On:
Blocks: sat580-low
TreeView+ depends on / blocked
 
Reported: 2016-12-08 14:28 UTC by Radovan Drazny
Modified: 2017-02-15 20:00 UTC (History)
1 user (show)

Fixed In Version:
Doc Type: If docs needed, set a value
Doc Text:
Clone Of:
Environment:
Last Closed: 2017-02-15 20:00:18 UTC
Target Upstream Version:


Attachments (Terms of Use)

Description Radovan Drazny 2016-12-08 14:28:42 UTC
Description of problem:
In advanced options for a kickstart profile, there are some mandatory options marked with asterisk. If user unchecks mandatory option checkbox and attempts to update the kickstart, Satellite will return error about missing mandatory option. If user checks the mandatory option, but leaves text field empty, kickstart file is updated, even if there is no parameter for the options. 


Version-Release number of selected component (if applicable):
spacewalk-java-2.3.8-157.el6sat

How reproducible:
always

Steps to Reproduce:
1. Have a kickstart profile created.
2. Select a profile in Kickstart profiles, go to Kickstart Details -> Advanced Options. 
3. Verify that "auth" option is checked, and delete content of its text field. 
4. Press Update Kickstart button. 

Actual results:
Satellite happily accepts the update and the kickstart file now contains "auth" option without any parameters.

Expected results:
Validating that parameters for each and every (mandatory) option are correct would be almost impossible, of course. Still, there should be at least some kind of warning that option is enabled but without any parameters, and that the resulting kickstart file may not work as expected because of this.

Additional info:

Comment 2 Grant Gainey 2017-02-15 20:00:18 UTC

*** This bug has been marked as a duplicate of bug 1357824 ***


Note You need to log in before you can comment on or make changes to this bug.