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

    • Type: Bug Bug
    • Status: Closed Closed (View Workflow)
    • Priority: Major Major
    • Resolution: Done
    • Affects Version/s: None
    • Fix Version/s: 6.0.0.M1
    • Component/s: Web (Tomcat) service
    • Security Level: Public (Everyone can see)
    • Labels:
      None
    • 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.

        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: