[jboss-jira] [JBoss JIRA] (WFCORE-3180) ElytronSubsystemMessages uses double quotes to surround string literals

Brian Stansberry (JIRA) issues at jboss.org
Thu Aug 10 10:06:01 EDT 2017


Brian Stansberry created WFCORE-3180:
----------------------------------------

             Summary: ElytronSubsystemMessages uses double quotes to surround string literals
                 Key: WFCORE-3180
                 URL: https://issues.jboss.org/browse/WFCORE-3180
             Project: WildFly Core
          Issue Type: Bug
          Components: Security
            Reporter: Brian Stansberry
            Assignee: Darran Lofthouse
            Priority: Minor


Lots of things like this:

@Message(id = 910, value = "Password cannot be resolved for key-store \"%s\"")

Use ' instead of \".  If the string ends up in a DMR ModelNode, which it often does, the output looks bad when double quotes are present.

For this ModelNode

ModelNode node = new ModelNode("Password cannot be resolved for key-store \"%s\"");
       
A call to node.toString() produces

"Password cannot be resolved for key-store \"%s\""

node.asString() is better, but that requires a special call vs the standard Object.toString

Password cannot be resolved for key-store "%s"

This is minor since we can try to ensure asString is used or perhaps even consider changing how toString works. But I think single quotes are a better practice.



--
This message was sent by Atlassian JIRA
(v7.2.3#72005)


More information about the jboss-jira mailing list