glassfish
  1. glassfish
  2. GLASSFISH-15711

The Default Provider checkbox is displaying wrong info and was ignored when creating/saving the provider config.

    Details

    • Type: Bug Bug
    • Status: Closed
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: None
    • Fix Version/s: 3.1.1_dev, 4.0_dev
    • Component/s: admin_gui
    • Labels:
      None

      Description

      The defaultProvider checkbox in Provider Config is wrong.
      The logic of calculating the value is wrong.

      gf.getEntityAttrs(endpoint="#

      {sessionScope.REST_URL}

      /configs/config/#

      {pageSession.configName}

      /security-service/message-security-config/#

      {pageSession.msgSecurityName}

      .json",
      valueMap="#

      {pageSession.msgSecurityMap}

      ");
      mapPut(map="#

      {pageSession.valueMap}" key="defaultProvider" value="false");

      if ("#{pageSession.msgSecurityMap['defaultClientProvider']}=true || #{pageSession.msgSecurityMap['defaultProvider']}=true")
      mapPut(map="#{pageSession.valueMap}

      " key="defaultProvider" value="true");

      It should test the name of the defaultClientProvider and defaultProvider and see if it match the one you are trying to edit.

      There is no code to save this checkbox value also.

      Since it is too late to fix and add the logic, I am removing the checkbox for 3.1.
      User can select that to be the default in the MessageSecurity page itself.

      Filing this but exclude from 3.1

        Activity

        Hide
        srinik76 added a comment -

        Checked in the fix into branch v3.1.1

        Sending SecurityHandler.java
        Transmitting file data .
        Committed revision 46444.

        Show
        srinik76 added a comment - Checked in the fix into branch v3.1.1 Sending SecurityHandler.java Transmitting file data . Committed revision 46444.
        Hide
        srinik76 added a comment -

        Closing the issue as fixed

        Show
        srinik76 added a comment - Closing the issue as fixed
        Hide
        srinik76 added a comment -

        Checked in the test case into the branch v3.1.1

        Sending SecurityTest.java
        Transmitting file data .
        Committed revision 46445.

        Show
        srinik76 added a comment - Checked in the test case into the branch v3.1.1 Sending SecurityTest.java Transmitting file data . Committed revision 46445.
        Hide
        srinik76 added a comment -

        Need to add the fix version 3.2_b03

        Show
        srinik76 added a comment - Need to add the fix version 3.2_b03
        Hide
        srinik76 added a comment -

        Added the fix version 3.2_b03

        Show
        srinik76 added a comment - Added the fix version 3.2_b03

          People

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

            Dates

            • Created:
              Updated:
              Resolved: