Bug 562555 - create/update activation key page strips '()'
Summary: create/update activation key page strips '()'
Keywords:
Status: CLOSED CURRENTRELEASE
Alias: None
Product: Spacewalk
Classification: Community
Component: WebUI
Version: 0.7
Hardware: All
OS: Linux
low
medium
Target Milestone: ---
Assignee: Partha Aji
QA Contact: Red Hat Satellite QA List
URL:
Whiteboard:
Depends On:
Blocks: space11
TreeView+ depends on / blocked
 
Reported: 2010-02-07 10:51 UTC by Michael Mráka
Modified: 2010-08-19 08:24 UTC (History)
1 user (show)

Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed: 2010-08-19 08:24:37 UTC
Embargoed:


Attachments (Terms of Use)

Description Michael Mráka 2010-02-07 10:51:27 UTC
Description of problem:
Create/update activation key page strips certain characters from key description.
It's possible to create key with such description via API.

Version-Release number of selected component (if applicable):
spacewalk-java-0.8.10-1.fc12.noarch

How reproducible:
always

Steps to Reproduce:
a)
1. go to Systems > Activation keys > create new key
2. enter description with '()', e.g. 'RHEL5 (i386)' 
3. click Create Activation Key
b)
1. create activation key via API   activationkey.create having '()' in description, e.g. 'RHEL5 (i386)'
2. go to Systems > Activation keys and select the key
3. update e.g. usage limit and click Create Activation Key

Actual results:
created key is missing '()' characters from description

Expected results:
consistent constraints on description via webUI and API
if some characters are no allowed in description, print warning

Additional info:

Comment 2 Partha Aji 2010-08-11 21:20:14 UTC
So I have changed the API to also scrub the name and description to be consistent.
as of http://git.fedorahosted.org/git/?p=spacewalk.git;a=commit;h=9ece7b70a0b3f8b951b0cc5d13ec47697da5ebe5

the update rhn-java rpm should work..

Comment 3 Partha Aji 2010-08-12 14:28:33 UTC
Moving to on qa

Comment 4 Milan Zázrivec 2010-08-19 08:24:37 UTC
Spacewalk 1.1 has been released.


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