glassfish
  1. glassfish
  2. GLASSFISH-18799

Admin gui can not save backward slash in a property.

    Details

    • Type: Bug Bug
    • Status: Resolved
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: 4.0
    • Fix Version/s: 4.0_b77, 4.0
    • Component/s: admin_gui
    • Labels:
      None
    • Environment:

      Windows

      Description

      In the page of Admin gui > Resources > New Resource Adapter Config, if a property's value contains backward slash '\', then the backward slash will be dropped after saving.

      Reproduction steps:
      1. start domain.
      2. Under Resources > New Resource Adapter Config, Add a new Property with name "TestProperty" and value "abc\def\ghi". Then click Save button.
      3. Under Resources > Resource Adapter Configs, edit the jmsra config which was just created. Then we can see the value of TestProperty has been changed to "abcdefghi" - the backward slash is lost.

      Is it possible to keep the backward slash when saving via admin gui? It is useful when the property is a path string on windows(for example, C:\glassfish3\glassfish\lib) which is generated by program as default value.

        Activity

        Hide
        Anissa Lam added a comment -

        Fix by HCF (3/25)

        Show
        Anissa Lam added a comment - Fix by HCF (3/25)
        Hide
        Anissa Lam added a comment -

        Issues need to be addressed before 4.0 HCF (3/25)

        Show
        Anissa Lam added a comment - Issues need to be addressed before 4.0 HCF (3/25)
        Hide
        Anissa Lam added a comment -

        The property value wasn't properly escaped. Its now fixed.

        Log Message:
        ------------
        GLASSFISH-18799. Need to escape property value properly.

        Revisions:
        ----------
        59665

        Modified Paths:
        ---------------
        trunk/main/appserver/admingui/common/src/main/java/org/glassfish/admingui/common/handlers/UtilHandlers.java
        trunk/main/appserver/admingui/common/src/main/java/org/glassfish/admingui/common/handlers/InstanceHandler.java

        Show
        Anissa Lam added a comment - The property value wasn't properly escaped. Its now fixed. Log Message: ------------ GLASSFISH-18799 . Need to escape property value properly. Revisions: ---------- 59665 Modified Paths: --------------- trunk/main/appserver/admingui/common/src/main/java/org/glassfish/admingui/common/handlers/UtilHandlers.java trunk/main/appserver/admingui/common/src/main/java/org/glassfish/admingui/common/handlers/InstanceHandler.java

          People

          • Assignee:
            Anissa Lam
            Reporter:
            David Zhao
          • Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved: