Application Server 3  4  5 and 6
  1. Application Server 3 4 5 and 6
  2. JBAS-7100

Exploding vfsmemory:// contents is empty for web deployments

    Details

    • Similar Issues:
      Show 10 results 

      Description

      The following patch addresses this:

      [alrubinger@localhost Branch_5_x]$ svn di server/
      When using an in-memory archive backed by VFS, we're using an "unjar"; this leaves out any RAM-backed resources. Instead explode the contents, regardless of what's behind the virtual abstraction:

      Index: server/src/main/org/jboss/web/deployers/AbstractWarDeployer.java
      ===================================================================
      — server/src/main/org/jboss/web/deployers/AbstractWarDeployer.java (revision 91204)
      +++ server/src/main/org/jboss/web/deployers/AbstractWarDeployer.java (working copy)
      -357,7 +357,7 @@

      VFSDeploymentUnit vfsUnit = (VFSDeploymentUnit)unit;
      VirtualFile root = vfsUnit.getRoot();

      • VirtualFile unjared = VFSUtils.unjar(root);
        + VirtualFile unjared = VFSUtils.explode(root);
        // file was not unjared yet
        if (root != unjared)
        {

      This was exposed by prototyping in EMB-32.

        Gliffy Diagrams

          Issue Links

            Activity

            Hide
            Andrew Rubinger added a comment -

            No longer needed due to Embedded using ShrinkWrap, not VFS. Leads to JBAS-7148.

            Show
            Andrew Rubinger added a comment - No longer needed due to Embedded using ShrinkWrap, not VFS. Leads to JBAS-7148 .
            Hide
            Dimitris Andreadis added a comment -

            Andrew, are you going to revert it?

            Show
            Dimitris Andreadis added a comment - Andrew, are you going to revert it?
            Hide
            Andrew Rubinger added a comment -

            Yes; was planning on doing so after this next big patch I'm working on it complete. If needed sooner ping me and I'll do it in a new workspace.

            Show
            Andrew Rubinger added a comment - Yes; was planning on doing so after this next big patch I'm working on it complete. If needed sooner ping me and I'll do it in a new workspace.
            Hide
            Andrew Rubinger added a comment -

            Reverted

            Show
            Andrew Rubinger added a comment - Reverted

              People

              • Assignee:
                Andrew Rubinger
                Reporter:
                Andrew Rubinger
              • Votes:
                0 Vote for this issue
                Watchers:
                1 Start watching this issue

                Dates

                • Created:
                  Updated:
                  Resolved:

                  Development