[jboss-cvs] JBossAS SVN: r59480 - in trunk: server/src/main/org/jboss/deployment and 1 other directories.
jboss-cvs-commits at lists.jboss.org
jboss-cvs-commits at lists.jboss.org
Wed Jan 10 07:33:05 EST 2007
Author: alesj
Date: 2007-01-10 07:32:49 -0500 (Wed, 10 Jan 2007)
New Revision: 59480
Modified:
trunk/ejb3/src/main/org/jboss/ejb3/deployers/AppClientScanningDeployer.java
trunk/ejb3/src/main/org/jboss/ejb3/deployers/Ejb3ClientDeployer.java
trunk/server/src/main/org/jboss/deployment/EARDeployer.java
trunk/server/src/main/org/jboss/deployment/EARStructure.java
trunk/system-jmx/src/main/org/jboss/deployment/DeploymentInfo.java
Log:
finally stream close, introduction of VFSUtils readManifest
Modified: trunk/ejb3/src/main/org/jboss/ejb3/deployers/AppClientScanningDeployer.java
===================================================================
--- trunk/ejb3/src/main/org/jboss/ejb3/deployers/AppClientScanningDeployer.java 2007-01-10 11:41:49 UTC (rev 59479)
+++ trunk/ejb3/src/main/org/jboss/ejb3/deployers/AppClientScanningDeployer.java 2007-01-10 12:32:49 UTC (rev 59480)
@@ -21,18 +21,18 @@
*/
package org.jboss.ejb3.deployers;
+import org.jboss.deployers.plugins.deployers.helpers.AbstractTypedDeployer;
+import org.jboss.deployers.spi.DeploymentException;
+import org.jboss.deployers.spi.deployer.DeploymentUnit;
+import org.jboss.ejb3.metamodel.ApplicationClientDD;
+import org.jboss.virtual.VirtualFile;
+
import java.io.IOException;
import java.io.InputStream;
import java.lang.reflect.Field;
import java.util.jar.Attributes;
import java.util.jar.Manifest;
-import org.jboss.deployers.plugins.deployers.helpers.AbstractTypedDeployer;
-import org.jboss.deployers.spi.DeploymentException;
-import org.jboss.deployers.spi.deployer.DeploymentUnit;
-import org.jboss.ejb3.metamodel.ApplicationClientDD;
-import org.jboss.virtual.VirtualFile;
-
/**
* Scan the main & super classes for annotations.
*
@@ -103,6 +103,7 @@
try
{
+ // TODO - use VFSUtils.readManifest
InputStream is = file.openStream();
Manifest mf;
try
Modified: trunk/ejb3/src/main/org/jboss/ejb3/deployers/Ejb3ClientDeployer.java
===================================================================
--- trunk/ejb3/src/main/org/jboss/ejb3/deployers/Ejb3ClientDeployer.java 2007-01-10 11:41:49 UTC (rev 59479)
+++ trunk/ejb3/src/main/org/jboss/ejb3/deployers/Ejb3ClientDeployer.java 2007-01-10 12:32:49 UTC (rev 59480)
@@ -141,7 +141,7 @@
{
try
{
- // use VFSUtils.readManifest(VirtualFile) .. once VFS lib is updated
+ // TODO - use VFSUtils.readManifest .. once VFS lib is updated
InputStream is = file.openStream();
Manifest mf;
try
Modified: trunk/server/src/main/org/jboss/deployment/EARDeployer.java
===================================================================
--- trunk/server/src/main/org/jboss/deployment/EARDeployer.java 2007-01-10 11:41:49 UTC (rev 59479)
+++ trunk/server/src/main/org/jboss/deployment/EARDeployer.java 2007-01-10 12:32:49 UTC (rev 59480)
@@ -21,36 +21,30 @@
*/
package org.jboss.deployment;
+import org.jboss.metadata.MetaData;
+import org.jboss.metadata.XmlFileLoader;
+import org.jboss.mx.loading.LoaderRepositoryFactory;
+import org.jboss.mx.loading.LoaderRepositoryFactory.LoaderRepositoryConfig;
+import org.jboss.mx.util.MBeanProxyExt;
+import org.jboss.system.ServiceControllerMBean;
+import org.jboss.util.file.JarUtils;
+import org.w3c.dom.Element;
+
+import javax.management.ObjectName;
+import javax.security.jacc.PolicyConfiguration;
+import javax.security.jacc.PolicyConfigurationFactory;
import java.io.File;
+import java.io.FileInputStream;
import java.io.IOException;
import java.io.InputStream;
-import java.io.FileInputStream;
import java.net.MalformedURLException;
import java.net.URL;
import java.security.Policy;
import java.util.Enumeration;
import java.util.HashMap;
import java.util.Iterator;
-import java.util.ArrayList;
-import java.util.jar.JarEntry;
-import java.util.jar.JarFile;
-import java.util.jar.Manifest;
-import java.util.jar.Attributes;
-import java.util.jar.JarInputStream;
+import java.util.jar.*;
-import javax.management.ObjectName;
-import javax.security.jacc.PolicyConfiguration;
-import javax.security.jacc.PolicyConfigurationFactory;
-
-import org.jboss.metadata.MetaData;
-import org.jboss.metadata.XmlFileLoader;
-import org.jboss.mx.loading.LoaderRepositoryFactory;
-import org.jboss.mx.loading.LoaderRepositoryFactory.LoaderRepositoryConfig;
-import org.jboss.mx.util.MBeanProxyExt;
-import org.jboss.system.ServiceControllerMBean;
-import org.jboss.util.file.JarUtils;
-import org.w3c.dom.Element;
-
/**
* Enterprise Archive Deployer.
*
@@ -424,7 +418,7 @@
of the application package, with the .war extension removed.
b. All ear modules with extension of .rar are considered resource adapters.
c. A directory named lib is considered to be the library directory, as
- described in Section EE.8.2.1, Bundled Libraries.
+ described in Section�EE.8.2.1, �Bundled Libraries.�
d. For all ear modules with a filename extension of .jar, but not in the lib
directory, do the following:
i. If the JAR file contains a META-INF/MANIFEST.MF file with a Main-Class
@@ -474,8 +468,15 @@
else if( mfFile.exists() )
{
FileInputStream fis = new FileInputStream(mfFile);
- Manifest mf = new Manifest(fis);
- fis.close();
+ Manifest mf;
+ try
+ {
+ mf = new Manifest(fis);
+ }
+ finally
+ {
+ fis.close();
+ }
Attributes attrs = mf.getMainAttributes();
if( attrs.containsKey(Attributes.Name.MAIN_CLASS) )
{
Modified: trunk/server/src/main/org/jboss/deployment/EARStructure.java
===================================================================
--- trunk/server/src/main/org/jboss/deployment/EARStructure.java 2007-01-10 11:41:49 UTC (rev 59479)
+++ trunk/server/src/main/org/jboss/deployment/EARStructure.java 2007-01-10 12:32:49 UTC (rev 59480)
@@ -21,13 +21,6 @@
*/
package org.jboss.deployment;
-import java.io.IOException;
-import java.io.InputStream;
-import java.util.Iterator;
-import java.util.List;
-import java.util.jar.Attributes;
-import java.util.jar.Manifest;
-
import org.jboss.deployers.plugins.structure.ContextInfoImpl;
import org.jboss.deployers.plugins.structure.vfs.AbstractStructureDeployer;
import org.jboss.deployers.spi.structure.vfs.StructureMetaData;
@@ -38,6 +31,13 @@
import org.jboss.virtual.plugins.vfs.helpers.SuffixMatchFilter;
import org.w3c.dom.Element;
+import java.io.IOException;
+import java.io.InputStream;
+import java.util.Iterator;
+import java.util.List;
+import java.util.jar.Attributes;
+import java.util.jar.Manifest;
+
/**
* Structure deployer for EARs.
*
@@ -98,7 +98,7 @@
public boolean determineStructure(VirtualFile root,
StructureMetaData metaData, StructuredDeployers deployers)
{
- boolean valid = false;
+ boolean valid;
try
{
if( root.isLeaf() == true || root.getName().endsWith(".ear") == false)
@@ -109,26 +109,38 @@
VirtualFile applicationXml = getMetaDataFile(root, "META-INF/application.xml");
VirtualFile jbossAppXml = getMetaDataFile(root, "META-INF/jboss-app.xml");
- VirtualFile lib = null;
+ VirtualFile lib;
J2eeApplicationMetaData j2eeMetaData = new J2eeApplicationMetaData();
boolean scan = true;
if (applicationXml != null)
{
InputStream in = applicationXml.openStream();
- XmlFileLoader xfl = new XmlFileLoader(false);
- Element application = xfl.getDocument(in, "META-INF/application.xml").getDocumentElement();
- j2eeMetaData.importXml(application);
- in.close();
+ try
+ {
+ XmlFileLoader xfl = new XmlFileLoader(false);
+ Element application = xfl.getDocument(in, "META-INF/application.xml").getDocumentElement();
+ j2eeMetaData.importXml(application);
+ }
+ finally
+ {
+ in.close();
+ }
scan = false;
}
if (jbossAppXml != null)
{
InputStream in = jbossAppXml.openStream();
- XmlFileLoader xfl = new XmlFileLoader(false);
- Element jbossApp = xfl.getDocument(in, "META-INF/jboss-app.xml").getDocumentElement();
- j2eeMetaData.importXml(jbossApp);
- in.close();
+ try
+ {
+ XmlFileLoader xfl = new XmlFileLoader(false);
+ Element jbossApp = xfl.getDocument(in, "META-INF/jboss-app.xml").getDocumentElement();
+ j2eeMetaData.importXml(jbossApp);
+ }
+ finally
+ {
+ in.close();
+ }
}
// the adding of the context should happen before the lib directory is processed
@@ -283,9 +295,17 @@
}
else if( mfFile != null )
{
+ // TODO - use VFSUtils.readManifest
InputStream is = mfFile.openStream();
- Manifest mf = new Manifest(is);
- is.close();
+ Manifest mf;
+ try
+ {
+ mf = new Manifest(is);
+ }
+ finally
+ {
+ is.close();
+ }
Attributes attrs = mf.getMainAttributes();
if( attrs.containsKey(Attributes.Name.MAIN_CLASS) )
{
Modified: trunk/system-jmx/src/main/org/jboss/deployment/DeploymentInfo.java
===================================================================
--- trunk/system-jmx/src/main/org/jboss/deployment/DeploymentInfo.java 2007-01-10 11:41:49 UTC (rev 59479)
+++ trunk/system-jmx/src/main/org/jboss/deployment/DeploymentInfo.java 2007-01-10 12:32:49 UTC (rev 59480)
@@ -21,32 +21,25 @@
*/
package org.jboss.deployment;
+import org.jboss.logging.Logger;
+import org.jboss.mx.loading.LoaderRepositoryFactory;
+import org.jboss.mx.loading.LoaderRepositoryFactory.LoaderRepositoryConfig;
+import org.jboss.mx.loading.RepositoryClassLoader;
+import org.jboss.util.collection.ListSet;
+import org.jboss.util.file.Files;
+import org.w3c.dom.Document;
+
+import javax.management.MBeanServer;
+import javax.management.ObjectName;
import java.io.File;
import java.io.FileInputStream;
import java.io.Serializable;
import java.net.URL;
import java.net.URLClassLoader;
-import java.util.ArrayList;
-import java.util.Collection;
-import java.util.Date;
-import java.util.HashMap;
-import java.util.Iterator;
-import java.util.List;
-import java.util.Set;
+import java.util.*;
import java.util.jar.JarFile;
import java.util.jar.Manifest;
-import javax.management.MBeanServer;
-import javax.management.ObjectName;
-
-import org.jboss.logging.Logger;
-import org.jboss.mx.loading.LoaderRepositoryFactory;
-import org.jboss.mx.loading.RepositoryClassLoader;
-import org.jboss.mx.loading.LoaderRepositoryFactory.LoaderRepositoryConfig;
-import org.jboss.util.collection.ListSet;
-import org.jboss.util.file.Files;
-import org.w3c.dom.Document;
-
/**
* Service Deployment Info .
*
@@ -381,8 +374,14 @@
if (file.isDirectory())
{
FileInputStream fis = new FileInputStream(new File(file, "META-INF/MANIFEST.MF"));
- manifest = new Manifest(fis);
- fis.close();
+ try
+ {
+ manifest = new Manifest(fis);
+ }
+ finally
+ {
+ fis.close();
+ }
}
else if(isXML == false)// a jar
manifest = new JarFile(file).getManifest();
More information about the jboss-cvs-commits
mailing list