You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@tomee.apache.org by st...@apache.org on 2018/06/18 15:12:34 UTC

[1/2] tomee git commit: TOMEE-2195 fix OSX compile issue in AnnotationDeployer

Repository: tomee
Updated Branches:
  refs/heads/master fe83aa562 -> c381de03b


TOMEE-2195 fix OSX compile issue in AnnotationDeployer

txs to gerdogdu for the patch!


Project: http://git-wip-us.apache.org/repos/asf/tomee/repo
Commit: http://git-wip-us.apache.org/repos/asf/tomee/commit/886decc3
Tree: http://git-wip-us.apache.org/repos/asf/tomee/tree/886decc3
Diff: http://git-wip-us.apache.org/repos/asf/tomee/diff/886decc3

Branch: refs/heads/master
Commit: 886decc3a87f08161d552b2564a558efd637ebca
Parents: fe83aa5
Author: Mark Struberg <st...@apache.org>
Authored: Mon Jun 18 16:58:02 2018 +0200
Committer: Mark Struberg <st...@apache.org>
Committed: Mon Jun 18 16:58:02 2018 +0200

----------------------------------------------------------------------
 .../main/java/org/apache/openejb/config/AnnotationDeployer.java  | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/tomee/blob/886decc3/container/openejb-core/src/main/java/org/apache/openejb/config/AnnotationDeployer.java
----------------------------------------------------------------------
diff --git a/container/openejb-core/src/main/java/org/apache/openejb/config/AnnotationDeployer.java b/container/openejb-core/src/main/java/org/apache/openejb/config/AnnotationDeployer.java
index 9df12fe..893b545 100644
--- a/container/openejb-core/src/main/java/org/apache/openejb/config/AnnotationDeployer.java
+++ b/container/openejb-core/src/main/java/org/apache/openejb/config/AnnotationDeployer.java
@@ -5720,9 +5720,9 @@ public class AnnotationDeployer implements DynamicDeployer {
                 }
             } else if (clazz.isInterface()) {
                 final Class api = clazz;
-                final List<Class> impl = finder.findImplementations(api);
+                final List impl = finder.findImplementations((Class<?>)api);
                 if (impl != null && impl.size() == 1) { // single impl so that's the service
-                    final Class implClass = impl.iterator().next();
+                    final Class implClass = (Class) impl.iterator().next();
                     final String name = implClass.getName();
                     if (!isEJB(implClass)) {
                         classes.add(name);


[2/2] tomee git commit: TOMEE-2196 fix keyStoreFile property

Posted by st...@apache.org.
TOMEE-2196 fix keyStoreFile property

txs to gerdogdu for the patch!


Project: http://git-wip-us.apache.org/repos/asf/tomee/repo
Commit: http://git-wip-us.apache.org/repos/asf/tomee/commit/c381de03
Tree: http://git-wip-us.apache.org/repos/asf/tomee/tree/c381de03
Diff: http://git-wip-us.apache.org/repos/asf/tomee/diff/c381de03

Branch: refs/heads/master
Commit: c381de03bde90e7013efff159e6ebc525c1b20c4
Parents: 886decc
Author: Mark Struberg <st...@apache.org>
Authored: Mon Jun 18 17:11:39 2018 +0200
Committer: Mark Struberg <st...@apache.org>
Committed: Mon Jun 18 17:11:39 2018 +0200

----------------------------------------------------------------------
 .../main/java/org/apache/tomee/embedded/Container.java   |  2 +-
 .../java/org/apache/tomee/embedded/SslTomEETest.java     | 11 +++++++++++
 2 files changed, 12 insertions(+), 1 deletion(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/tomee/blob/c381de03/tomee/tomee-embedded/src/main/java/org/apache/tomee/embedded/Container.java
----------------------------------------------------------------------
diff --git a/tomee/tomee-embedded/src/main/java/org/apache/tomee/embedded/Container.java b/tomee/tomee-embedded/src/main/java/org/apache/tomee/embedded/Container.java
index cb745a3..5458ba7 100644
--- a/tomee/tomee-embedded/src/main/java/org/apache/tomee/embedded/Container.java
+++ b/tomee/tomee-embedded/src/main/java/org/apache/tomee/embedded/Container.java
@@ -629,7 +629,7 @@ public class Container implements AutoCloseable {
             httpsConnector.setProperty("sslProtocol", configuration.getSslProtocol());
 
             if (configuration.getKeystoreFile() != null) {
-                httpsConnector.setAttribute("", configuration.getKeystoreFile());
+                httpsConnector.setAttribute("keystoreFile", configuration.getKeystoreFile());
             }
             if (configuration.getKeystorePass() != null) {
                 httpsConnector.setAttribute("keystorePass", configuration.getKeystorePass());

http://git-wip-us.apache.org/repos/asf/tomee/blob/c381de03/tomee/tomee-embedded/src/test/java/org/apache/tomee/embedded/SslTomEETest.java
----------------------------------------------------------------------
diff --git a/tomee/tomee-embedded/src/test/java/org/apache/tomee/embedded/SslTomEETest.java b/tomee/tomee-embedded/src/test/java/org/apache/tomee/embedded/SslTomEETest.java
index 3f5e711..ae98087 100644
--- a/tomee/tomee-embedded/src/test/java/org/apache/tomee/embedded/SslTomEETest.java
+++ b/tomee/tomee-embedded/src/test/java/org/apache/tomee/embedded/SslTomEETest.java
@@ -16,6 +16,7 @@
  */
 package org.apache.tomee.embedded;
 
+import org.apache.catalina.connector.Connector;
 import org.apache.openejb.loader.Files;
 import org.bouncycastle.asn1.x500.X500Name;
 import org.bouncycastle.cert.X509CertificateHolder;
@@ -43,6 +44,7 @@ import java.util.concurrent.TimeUnit;
 
 import static org.junit.Assert.assertEquals;
 import static org.junit.Assert.assertFalse;
+import static org.junit.Assert.assertNotNull;
 
 public class SslTomEETest {
     @Test
@@ -104,6 +106,15 @@ public class SslTomEETest {
         final Container container = new Container();
         container.setup(configuration);
         container.start();
+        Connector[] connectors = container.getTomcat().getService().findConnectors();
+        for(Connector conn : connectors) {
+        	if(conn.getPort() == 8443) {
+        		Object propertyObject = conn.getProperty("keystoreFile");
+                assertNotNull(propertyObject);
+                assertEquals(keystore.getAbsolutePath(), propertyObject.toString());
+        	}
+        }
+
         try {
             assertEquals(8443, ManagementFactory.getPlatformMBeanServer().getAttribute(new ObjectName("Tomcat:type=ProtocolHandler,port=8443"), "port"));
         } finally {