glassfish
  1. glassfish
  2. GLASSFISH-3844

Treat default-web.xml as runtime instead of a deploytime option

    Details

    • Type: Bug Bug
    • Status: Resolved
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: 9.1pe
    • Fix Version/s: 9.1.1_dev
    • Component/s: deployment
    • Labels:
      None
    • Environment:

      Operating System: All
      Platform: Sun

    • Issuezilla Id:
      3,844
    • Status Whiteboard:
      Hide

      as91ur1-na,as911-na

      Show
      as91ur1-na,as911-na

      Description

      Currently, the default-web.xml is treated as a deploytime option, in that it is
      consulted only during the deployment of a webapp, at which time it is used to
      augment the webapp's web.xml.

      If the default-web.xml is modified before the server is restarted, any webapps
      that are restarted will keep the original default-web.xml settings that they
      inherited at the time when they were deployed. In order for any of the restarted
      webapps to pick up the latest default-web.xml settings, they need to be redeployed.

      The feedback received from many users has been that it would be more intuitive
      for the default-web.xml to be treated as a runtime (as opposed to a deploytime)
      option, so that any changes to it would become effective immediately after a
      server restart, without the need to redeploy any of the already deployed webapps.

      1. default-web.xml
        36 kB
        vince kraemer
      2. default-web.xml
        37 kB
        vince kraemer

        Activity

        Hide
        vince kraemer added a comment -

        the second attachment is the modified version of the default-web.xml file...

        The first one is the "default" default-web.xml...

        Show
        vince kraemer added a comment - the second attachment is the modified version of the default-web.xml file... The first one is the "default" default-web.xml...
        Hide
        Tim Quinn added a comment -

        Additional fix checked in (per Harpreet's approval):

        glassfish/appserv-core/src/java/com/sun/enterprise/instance/BaseManager.java

        glassfish/appserv-core/src/java/com/sun/enterprise/instance/WebModulesManager.java

        glassfish/appserv-core/src/java/com/sun/enterprise/deployment/backend/DeploymentUtils.java

        glassfish/appserv-commons/src/java/com/sun/logging/enterprise/system/core/LogStrings.properties

        glassfish/appserv-commons/src/java/com/sun/logging/enterprise/system/tools/deployment/LogStrings.properties

        Show
        Tim Quinn added a comment - Additional fix checked in (per Harpreet's approval): glassfish/appserv-core/src/java/com/sun/enterprise/instance/BaseManager.java glassfish/appserv-core/src/java/com/sun/enterprise/instance/WebModulesManager.java glassfish/appserv-core/src/java/com/sun/enterprise/deployment/backend/DeploymentUtils.java glassfish/appserv-commons/src/java/com/sun/logging/enterprise/system/core/LogStrings.properties glassfish/appserv-commons/src/java/com/sun/logging/enterprise/system/tools/deployment/LogStrings.properties
        Hide
        Tim Quinn added a comment -

        Updating target milestone to reflect which build the fix should be in.

        Show
        Tim Quinn added a comment - Updating target milestone to reflect which build the fix should be in.
        Hide
        vince kraemer added a comment -

        This appears to be working for me now in v2.1 b60a

        is this feature in v3?

        Show
        vince kraemer added a comment - This appears to be working for me now in v2.1 b60a is this feature in v3?
        Hide
        gfuser9999 added a comment -

        Broken as in V3.1.1 and 3.2 too
        See GLASSFISH-11235

        Show
        gfuser9999 added a comment - Broken as in V3.1.1 and 3.2 too See GLASSFISH-11235

          People

          • Assignee:
            Tim Quinn
            Reporter:
            jluehe
          • Votes:
            0 Vote for this issue
            Watchers:
            1 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved: