glassfish
  1. glassfish
  2. GLASSFISH-16339

[UB]sun-acc.xml (and associated DTD) are now glassfish-acc.xml

    Details

    • Type: Bug Bug
    • Status: Open
    • Priority: Major Major
    • Resolution: Unresolved
    • Affects Version/s: 3.1
    • Fix Version/s: 4.0
    • Component/s: docs
    • Labels:
      None

      Description

      During 3.1 we converted Glassfish-specific descriptors and DTDs from "sun-xxx" names to "glassfish-xxx" but I did not do this for the sun-acc.xml and the sun-application-client-container_1_2.dtd files.

      I have opened issue 16302 which is for the engineering changes to make this change.

      The docs should change to change references to sun-application-client-container.dtd to glassfish-application-client-container_1_3.dtd and from sun-acc.sml to glassfish-acc.xml.

      Note that customers might have scripts which refer to the sun-acc.xml that has historically been generated into $

      {domainDir}

      /config, so 3.2 will continue to provide that file as well as glassfish-acc.xml. Officially the use of sun-acc.xml will be deprecated as of 3.2. But the documentation should steer users toward glassfish-acc.xml instead of sun-acc.xml.

        Issue Links

          Activity

          Hide
          Tim Quinn added a comment -

          Linking to 16302 - the issue covering the engineering changes.

          Show
          Tim Quinn added a comment - Linking to 16302 - the issue covering the engineering changes.
          Hide
          Rebecca Parks added a comment -

          Changed target release to 3.2 per Tim Quinn.

          Show
          Rebecca Parks added a comment - Changed target release to 3.2 per Tim Quinn.
          Hide
          Rebecca Parks added a comment -

          Build 3 is when Tim made the fix in the 3.2 code.

          Show
          Rebecca Parks added a comment - Build 3 is when Tim made the fix in the 3.2 code.
          Hide
          Tim Quinn added a comment -

          I have made the coding changes in 3.1.2 (which should appear in promoted build 11) for 16302.

          The doc or release notes should mention that glassfish-acc.xml becomes the default ACC config file (in the domain's config directory) and if the user does not specify "-xml path-to-acc-config-file" on the appclient command then the ACC will look for glassfish-acc.xml in the domain's config directory as the default. (Prior to these changes it would look for sun-acc.xml.)

          If a user does an in-place update from 3.1 to 3.1.2, then the glassfish-acc.xml file will not appear in the domain's config directory. To support this use case the ACC will continue to use the sun-acc.xml file as the default ACC config file if it cannot find glassfish-acc.xml. Note that the user can always control what config file the ACC will read using the -xml option on the appclient command.

          Show
          Tim Quinn added a comment - I have made the coding changes in 3.1.2 (which should appear in promoted build 11) for 16302. The doc or release notes should mention that glassfish-acc.xml becomes the default ACC config file (in the domain's config directory) and if the user does not specify "-xml path-to-acc-config-file" on the appclient command then the ACC will look for glassfish-acc.xml in the domain's config directory as the default. (Prior to these changes it would look for sun-acc.xml.) If a user does an in-place update from 3.1 to 3.1.2, then the glassfish-acc.xml file will not appear in the domain's config directory. To support this use case the ACC will continue to use the sun-acc.xml file as the default ACC config file if it cannot find glassfish-acc.xml. Note that the user can always control what config file the ACC will read using the -xml option on the appclient command.
          Hide
          Tim Quinn added a comment -

          Rebecca,

          Upon further review, the small incompatibility that we'd introduce by making these changes in 3.1.2, given that this is a bug release, is one we should not create.

          So I'm going to revert the changes for this in he 3.1.2 code base. We will not make the file name changes in 3.1.2.

          Show
          Tim Quinn added a comment - Rebecca, Upon further review, the small incompatibility that we'd introduce by making these changes in 3.1.2, given that this is a bug release, is one we should not create. So I'm going to revert the changes for this in he 3.1.2 code base. We will not make the file name changes in 3.1.2.
          Hide
          Mike Fitch added a comment -

          Changed fix version to 4.0 so this issue gets picked up by OSE 4.0 release queries.

          Show
          Mike Fitch added a comment - Changed fix version to 4.0 so this issue gets picked up by OSE 4.0 release queries.

            People

            • Assignee:
              Paul Davies
              Reporter:
              Tim Quinn
            • Votes:
              0 Vote for this issue
              Watchers:
              0 Start watching this issue

              Dates

              • Created:
                Updated: