Bug 770354

Summary: No validation for Priority field in Add Provider Page
Product: [Retired] CloudForms Cloud Engine Reporter: Shveta <ssachdev>
Component: aeolus-conductorAssignee: Maros Zatko <mzatko>
Status: CLOSED CURRENTRELEASE QA Contact: wes hayutin <whayutin>
Severity: unspecified Docs Contact:
Priority: unspecified    
Version: 1.0.0CC: akarol, deltacloud-maint, mzatko, slinaber, ssachdev
Target Milestone: rc   
Target Release: ---   
Hardware: Unspecified   
OS: Unspecified   
Whiteboard:
Fixed In Version: Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: Type: ---
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: --- RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: --- Target Upstream Version:
Attachments:
Description Flags
priority_validation none

Description Shveta 2011-12-26 06:27:19 UTC 
Description of problem:


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


How reproducible:


Steps to Reproduce:
1. Priority field in Add provider page 
2. Takes any input like all special chars (@$^IU$#$%^%^%)
3. Or it can be left blank too 

What is the significance of that field ?
  
Actual results:


Expected results:


Additional info:

 rpm -qa|grep aeolus
aeolus-conductor-doc-0.8.0-0.20111222233342gitd98cb57.el6.noarch
aeolus-configure-2.5.0-0.20111222173430git17b704a.el6.noarch
rubygem-aeolus-image-0.3.0-0.20111222173411gitc13b654.el6.noarch
rubygem-aeolus-cli-0.3.0-0.20111222173356git3cd6277.el6.noarch
aeolus-all-0.8.0-0.20111222233342gitd98cb57.el6.noarch
aeolus-conductor-0.8.0-0.20111222233342gitd98cb57.el6.noarch
aeolus-conductor-daemons-0.8.0-0.20111222233342gitd98cb57.el6.noarch
Comment 1 wes hayutin 2012-01-10 17:11:36 UTC 
adding to ce-sprint-next
Comment 2 wes hayutin 2012-01-10 17:15:04 UTC 
adding to ce-sprint-next
Comment 3 wes hayutin 2012-01-12 16:36:40 UTC 
adding to ce-sprint
Comment 4 wes hayutin 2012-01-12 16:42:44 UTC 
removing ce-sprint-next tracker
Comment 5 wes hayutin 2012-01-12 16:44:29 UTC 
taking off ce-sprint-next..
Comment 6 Maros Zatko 2012-01-24 13:17:42 UTC 
fixed in

commit 21bef1c79f0ab8753c072a5871555f8f39d78987
Author: Maros Zatko <mzatko>
Date:   Mon Jan 23 16:22:32 2012 +0100

    BZ#770354 added tests for priority field validation

commit 7aab61434b45dbc22be4c107f1fb7a22846e18bd
Author: Maros Zatko <mzatko>
Date:   Mon Jan 23 16:21:45 2012 +0100

    BZ#770354 added validation for priority field
Comment 7 Steve Linabery 2012-01-25 00:09:53 UTC 
21bef1c  7aab614  in aeolus-conductor-0.8.0-11.el6
Comment 8 Shveta 2012-01-25 03:44:32 UTC 
Created attachment 557362 [details]
priority_validation
Comment 9 Shveta 2012-01-25 03:46:55 UTC 
Verified in
rpm -qa|grep aeolus
aeolus-conductor-0.8.0-11.el6.noarch
aeolus-conductor-doc-0.8.0-11.el6.noarch
rubygem-aeolus-image-0.3.0-3.el6.noarch
rubygem-aeolus-cli-0.3.0-5.el6.noarch
aeolus-all-0.8.0-11.el6.noarch
aeolus-configure-2.5.0-7.el6.noarch
aeolus-conductor-daemons-0.8.0-11.el6.noarch