Bug 988283 - CLI GUI - write attribute dialog for string value should enclose value in generated command to double quotes
Summary: CLI GUI - write attribute dialog for string value should enclose value in gen...
Keywords:
Status: CLOSED CURRENTRELEASE
Alias: None
Product: JBoss Enterprise Application Platform 6
Classification: JBoss
Component: CLI
Version: 6.1.1
Hardware: Unspecified
OS: Unspecified
unspecified
medium
Target Milestone: DR2
: EAP 6.3.0
Assignee: Alexey Loubyansky
QA Contact:
URL:
Whiteboard:
: 1054214 1066228 (view as bug list)
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2013-07-25 09:01 UTC by Petr Sakař
Modified: 2014-06-28 15:42 UTC (History)
8 users (show)

Fixed In Version:
Clone Of:
Environment:
Last Closed: 2014-06-28 15:42:11 UTC
Type: Bug
Embargoed:


Attachments (Terms of Use)


Links
System ID Private Priority Status Summary Last Updated
Red Hat Issue Tracker WFLY-2051 0 Major Resolved CLI write attribute dialog for string value should enclose value in generated command to double quotes 2017-10-18 16:36:05 UTC

Description Petr Sakař 2013-07-25 09:01:55 UTC
Description of problem:
String value entered in write attribute dialog is written to command without double quotes, which is wrong in many cases

Eg. when user enteres in dialog value ${jboss.bind.address:127.0.0.1}
following command is generated:

/subsystem=webservices/:write-attribute(name=wsdl-host,value=${jboss.bind.address:127.0.0.1})

Correctly should be generated command 
/subsystem=webservices/:write-attribute(name=wsdl-host,value="${jboss.bind.address:127.0.0.1}")

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


How reproducible:


Steps to Reproduce:
1. start AS
  ./bin/standalone.sh
2. start CLI GUI 
  ./bin/jboss-cli.sh --gui &
3. click on node subsystem=webservices of /
4. right click on wsdl-host and select write-attribute
5. enter value ${jboss.bind.address:127.0.0.1} and click OK
6. submit generated command
7. reload node
  click on node subsystem=webservices of / (close node)
  click on node subsystem=webservices of / (open node)

Actual results:
  wsdl-host=$

Expected results:
  wsdl-host=${jboss.bind.address:127.0.0.1}


Workaround
  in dialog box enclose the value in double quotes

Additional info:

Comment 2 Chao Wang 2013-09-12 10:11:14 UTC
Issue exists in wildfly as well, and also see this with normal CLI write-attribute operation for attributes like STRING type and expression-allowed => true, 
Example: /interface=management:write-attribute(name=inet-address,value=${jboss.bind.address.management:127.0.0.1})

From https://github.com/wildfly/wildfly/blob/master/cli/src/main/java/org/jboss/as/cli/ArgumentValueConverter.java?source=c#L71

toSet = ModelNode.fromString(value); This returns 'IllegalArgumentException: Invalid character: $' then '$' is set by ArgumentValueCallbackHandler

Create an upstream issue as well https://issues.jboss.org/browse/WFLY-2051

Comment 4 JBoss JIRA Server 2013-09-17 14:20:27 UTC
Alexey Loubyansky <alex> made a comment on jira WFLY-2051

It's actually a bug in the CLI command line parser. I.e. it parses any expression like ${xxxx} to simply $. This would fix it 
https://github.com/aloubyansky/wildfly/commit/255fa8e50a98cae9bffbe7dd60adc6d518548341

Comment 5 JBoss JIRA Server 2013-09-17 14:21:04 UTC
Alexey Loubyansky <alex> made a comment on jira WFLY-2051

It's actually a bug in the CLI command line parser. I.e. it parses any expression like ${xxxx} to simply $. This would fix it 
https://github.com/aloubyansky/wildfly/commit/255fa8e50a98cae9bffbe7dd60adc6d518548341

Comment 6 JBoss JIRA Server 2013-10-08 14:53:53 UTC
Alexey Loubyansky <alex> updated the status of jira WFLY-2051 to Resolved

Comment 7 JBoss JIRA Server 2013-10-08 14:53:53 UTC
Alexey Loubyansky <alex> made a comment on jira WFLY-2051

Fixed.

Comment 9 James Perkins 2014-03-05 17:35:53 UTC
*** Bug 1054214 has been marked as a duplicate of this bug. ***

Comment 10 Petr Kremensky 2014-03-07 11:44:07 UTC
Verified on EAP 6.2.0.DR2

Comment 11 Brian Stansberry 2014-05-05 20:19:24 UTC
*** Bug 1066228 has been marked as a duplicate of this bug. ***


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