Bug 569559 - If a user creates an alert, adding a condition with invalid input, the invalid input will be accepted
Summary: If a user creates an alert, adding a condition with invalid input, the invali...
Keywords:
Status: CLOSED CURRENTRELEASE
Alias: None
Product: RHQ Project
Classification: Other
Component: Alerts
Version: unspecified
Hardware: All
OS: Linux
low
medium
Target Milestone: ---
: ---
Assignee: Justin Harris
QA Contact: Chandrasekar Kannan
URL:
Whiteboard:
Depends On:
Blocks: rhq_sprint06
TreeView+ depends on / blocked
 
Reported: 2010-03-01 17:33 UTC by Steve Salevan
Modified: 2015-01-04 23:41 UTC (History)
3 users (show)

Fixed In Version: 2.4
Clone Of:
Environment:
Last Closed: 2010-08-12 16:54:49 UTC
Embargoed:


Attachments (Terms of Use)
JBoss Seam Debug page (91.24 KB, application/pdf)
2010-03-01 17:44 UTC, Steve Salevan
no flags Details

Description Steve Salevan 2010-03-01 17:33:49 UTC
Description of problem:
If a user creates an Alert for a platform, adding the following Measurement Threshold condition to it:

Active Thread Count > 1nonNum&^*^#1

the input will be accepted and the following error message will appear on the Alert creation page:

/rhq/resource/alert/listConditions.xhtml @276,132 value="#{alertConditionsUIBean.threshold}": Error writing 'threshold' on type org.rhq.enterprise.gui.alert.AlertConditionsUIBean_$$_javassist_14

Version-Release number of selected component (if applicable):
RHQ-alertPlugin, 2/23 git checkout

How reproducible:
Always

Steps to Reproduce:
1. Create an alert with invalid input similar to that reported
  
Actual results:
Similar to above

Expected results:
Input rejected

Additional info:

Comment 1 Steve Salevan 2010-03-01 17:44:11 UTC
Created attachment 397137 [details]
JBoss Seam Debug page

It appears that, if a user creates a Measurement Baseline alert as well, with:

Active Thread Count > -99 of max value

they will receive a traceback similar to that attached.

Comment 2 Justin Harris 2010-03-05 15:41:54 UTC
Current build of master results in proper error messages displayed for both cases.  This is a 'Works for me'.   Can you verify Steve?

Comment 3 Steve Salevan 2010-03-09 18:06:27 UTC
This issue no longer presents in a 5/9 master branch git pull build.  Moving to
VERIFIED.

Comment 4 wes hayutin 2010-03-10 14:19:44 UTC
moving verified bugs to rhq_sprint06, and removing from rhq_chainsaw

Comment 9 Corey Welton 2010-08-12 16:54:49 UTC
Mass-closure of verified bugs against JON.


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