You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@camel.apache.org by da...@apache.org on 2018/08/18 06:08:26 UTC

[camel] branch master updated: [CAMEL-12605] Refactored name and package of AS2 utilities (#2482)

This is an automated email from the ASF dual-hosted git repository.

davsclaus pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/camel.git


The following commit(s) were added to refs/heads/master by this push:
     new 389aa81  [CAMEL-12605] Refactored name and package of AS2 utilities (#2482)
389aa81 is described below

commit 389aa81d788997cdff4534ec8cdbe884805d18bd
Author: William Collins <wc...@redhat.com>
AuthorDate: Sat Aug 18 02:08:19 2018 -0400

    [CAMEL-12605] Refactored name and package of AS2 utilities (#2482)
---
 .../apache/camel/component/as2/api/protocol/RequestAS2.java    |  8 ++++----
 .../component/as2/api/protocol/RequestAsynchronousMDN.java     |  4 ++--
 .../apache/camel/component/as2/api/protocol/ResponseMDN.java   | 10 +++++-----
 .../camel/component/as2/api/{Util.java => util/AS2Utils.java}  |  7 ++++---
 .../org/apache/camel/component/as2/api/AS2MessageTest.java     |  7 ++++---
 .../camel/component/as2/AS2ClientManagerIntegrationTest.java   |  4 ++--
 6 files changed, 21 insertions(+), 19 deletions(-)

diff --git a/components/camel-as2/camel-as2-api/src/main/java/org/apache/camel/component/as2/api/protocol/RequestAS2.java b/components/camel-as2/camel-as2-api/src/main/java/org/apache/camel/component/as2/api/protocol/RequestAS2.java
index be1ee6a..852067c 100644
--- a/components/camel-as2/camel-as2-api/src/main/java/org/apache/camel/component/as2/api/protocol/RequestAS2.java
+++ b/components/camel-as2/camel-as2-api/src/main/java/org/apache/camel/component/as2/api/protocol/RequestAS2.java
@@ -22,7 +22,7 @@ import org.apache.camel.component.as2.api.AS2ClientManager;
 import org.apache.camel.component.as2.api.AS2Constants;
 import org.apache.camel.component.as2.api.AS2Header;
 import org.apache.camel.component.as2.api.InvalidAS2NameException;
-import org.apache.camel.component.as2.api.Util;
+import org.apache.camel.component.as2.api.util.AS2Utils;
 import org.apache.http.HttpException;
 import org.apache.http.HttpRequest;
 import org.apache.http.HttpRequestInterceptor;
@@ -61,7 +61,7 @@ public class RequestAS2 implements HttpRequestInterceptor {
         /* AS2-From header */
         String as2From = coreContext.getAttribute(AS2ClientManager.AS2_FROM, String.class);
         try {
-            Util.validateAS2Name(as2From);
+            AS2Utils.validateAS2Name(as2From);
         } catch (InvalidAS2NameException e) {
             throw new HttpException("Invalid AS-From name", e);
         }
@@ -70,7 +70,7 @@ public class RequestAS2 implements HttpRequestInterceptor {
         /* AS2-To header */
         String as2To = coreContext.getAttribute(AS2ClientManager.AS2_TO, String.class);
         try {
-            Util.validateAS2Name(as2To);
+            AS2Utils.validateAS2Name(as2To);
         } catch (InvalidAS2NameException e) {
             throw new HttpException("Invalid AS-To name", e);
         }
@@ -78,7 +78,7 @@ public class RequestAS2 implements HttpRequestInterceptor {
 
         /* Message-Id header*/
         // SHOULD be set to aid in message reconciliation
-        request.addHeader(AS2Header.MESSAGE_ID, Util.createMessageId(clientFQDN));
+        request.addHeader(AS2Header.MESSAGE_ID, AS2Utils.createMessageId(clientFQDN));
     }
 
 }
diff --git a/components/camel-as2/camel-as2-api/src/main/java/org/apache/camel/component/as2/api/protocol/RequestAsynchronousMDN.java b/components/camel-as2/camel-as2-api/src/main/java/org/apache/camel/component/as2/api/protocol/RequestAsynchronousMDN.java
index c427628..ed03fbb 100644
--- a/components/camel-as2/camel-as2-api/src/main/java/org/apache/camel/component/as2/api/protocol/RequestAsynchronousMDN.java
+++ b/components/camel-as2/camel-as2-api/src/main/java/org/apache/camel/component/as2/api/protocol/RequestAsynchronousMDN.java
@@ -21,7 +21,7 @@ import java.io.IOException;
 import org.apache.camel.component.as2.api.AS2AsynchronousMDNManager;
 import org.apache.camel.component.as2.api.AS2Constants;
 import org.apache.camel.component.as2.api.AS2Header;
-import org.apache.camel.component.as2.api.Util;
+import org.apache.camel.component.as2.api.util.AS2Utils;
 import org.apache.http.HttpException;
 import org.apache.http.HttpRequest;
 import org.apache.http.HttpRequestInterceptor;
@@ -51,7 +51,7 @@ public class RequestAsynchronousMDN implements HttpRequestInterceptor {
 
         /* Message-Id header */
         // SHOULD be set to aid in message reconciliation
-        request.addHeader(AS2Header.MESSAGE_ID, Util.createMessageId(senderFQDN));
+        request.addHeader(AS2Header.MESSAGE_ID, AS2Utils.createMessageId(senderFQDN));
 
         /* Recipient-Address header */
         String recipientAddress = coreContext.getAttribute(AS2AsynchronousMDNManager.RECIPIENT_ADDRESS, String.class);
diff --git a/components/camel-as2/camel-as2-api/src/main/java/org/apache/camel/component/as2/api/protocol/ResponseMDN.java b/components/camel-as2/camel-as2-api/src/main/java/org/apache/camel/component/as2/api/protocol/ResponseMDN.java
index 98a72fa..e0aecbd 100644
--- a/components/camel-as2/camel-as2-api/src/main/java/org/apache/camel/component/as2/api/protocol/ResponseMDN.java
+++ b/components/camel-as2/camel-as2-api/src/main/java/org/apache/camel/component/as2/api/protocol/ResponseMDN.java
@@ -28,13 +28,13 @@ import org.apache.camel.component.as2.api.AS2ServerManager;
 import org.apache.camel.component.as2.api.AS2SignedDataGenerator;
 import org.apache.camel.component.as2.api.AS2TransferEncoding;
 import org.apache.camel.component.as2.api.InvalidAS2NameException;
-import org.apache.camel.component.as2.api.Util;
 import org.apache.camel.component.as2.api.entity.AS2DispositionType;
 import org.apache.camel.component.as2.api.entity.DispositionMode;
 import org.apache.camel.component.as2.api.entity.DispositionNotificationMultipartReportEntity;
 import org.apache.camel.component.as2.api.entity.DispositionNotificationOptions;
 import org.apache.camel.component.as2.api.entity.DispositionNotificationOptionsParser;
 import org.apache.camel.component.as2.api.entity.MultipartSignedEntity;
+import org.apache.camel.component.as2.api.util.AS2Utils;
 import org.apache.camel.component.as2.api.util.EntityUtils;
 import org.apache.camel.component.as2.api.util.HttpMessageUtils;
 import org.apache.camel.component.as2.api.util.SigningUtils;
@@ -139,7 +139,7 @@ public class ResponseMDN implements HttpResponseInterceptor {
             /* AS2-From header */
             String as2From = HttpMessageUtils.getHeaderValue(request, AS2Header.AS2_TO);
             try {
-                Util.validateAS2Name(as2From);
+                AS2Utils.validateAS2Name(as2From);
             } catch (InvalidAS2NameException e) {
                 throw new HttpException("Invalid AS-From name", e);
             }
@@ -148,7 +148,7 @@ public class ResponseMDN implements HttpResponseInterceptor {
             /* AS2-To header */
             String as2To = HttpMessageUtils.getHeaderValue(request, AS2Header.AS2_FROM);
             try {
-                Util.validateAS2Name(as2To);
+                AS2Utils.validateAS2Name(as2To);
             } catch (InvalidAS2NameException e) {
                 throw new HttpException("Invalid AS-To name", e);
             }
@@ -156,7 +156,7 @@ public class ResponseMDN implements HttpResponseInterceptor {
 
             /* Message-Id header*/
             // RFC4130 - 7.3 -  A Message-ID header is added to support message reconciliation
-            response.addHeader(AS2Header.MESSAGE_ID, Util.createMessageId(serverFQDN));
+            response.addHeader(AS2Header.MESSAGE_ID, AS2Utils.createMessageId(serverFQDN));
 
             AS2SignedDataGenerator gen = null;
             if (dispositionNotificationOptions.getSignedReceiptProtocol() != null && signingCertificateChain != null
@@ -182,7 +182,7 @@ public class ResponseMDN implements HttpResponseInterceptor {
             }
         }
 
-        LOG.debug(Util.printMessage(response));
+        LOG.debug(AS2Utils.printMessage(response));
     }
 
 }
diff --git a/components/camel-as2/camel-as2-api/src/main/java/org/apache/camel/component/as2/api/Util.java b/components/camel-as2/camel-as2-api/src/main/java/org/apache/camel/component/as2/api/util/AS2Utils.java
similarity index 97%
rename from components/camel-as2/camel-as2-api/src/main/java/org/apache/camel/component/as2/api/Util.java
rename to components/camel-as2/camel-as2-api/src/main/java/org/apache/camel/component/as2/api/util/AS2Utils.java
index 3d8f483..d4c3161 100644
--- a/components/camel-as2/camel-as2-api/src/main/java/org/apache/camel/component/as2/api/Util.java
+++ b/components/camel-as2/camel-as2-api/src/main/java/org/apache/camel/component/as2/api/util/AS2Utils.java
@@ -14,7 +14,7 @@
  * See the License for the specific language governing permissions and
  * limitations under the License.
  */
-package org.apache.camel.component.as2.api;
+package org.apache.camel.component.as2.api.util;
 
 import java.awt.event.KeyEvent;
 import java.io.ByteArrayOutputStream;
@@ -25,6 +25,7 @@ import java.util.Random;
 import java.util.regex.Matcher;
 import java.util.regex.Pattern;
 
+import org.apache.camel.component.as2.api.InvalidAS2NameException;
 import org.apache.http.Header;
 import org.apache.http.HeaderIterator;
 import org.apache.http.HttpEntity;
@@ -38,7 +39,7 @@ import org.apache.http.StatusLine;
 /**
  * Utility Methods used in AS2 Component
  */
-public final class Util {
+public final class AS2Utils {
 
     public static final String DQUOTE = "\"";
     public static final String BACKSLASH = "\\\\";
@@ -54,7 +55,7 @@ public final class Util {
 
     private static Random generator = new Random();
 
-    private Util() {
+    private AS2Utils() {
     }
 
     /**
diff --git a/components/camel-as2/camel-as2-api/src/test/java/org/apache/camel/component/as2/api/AS2MessageTest.java b/components/camel-as2/camel-as2-api/src/test/java/org/apache/camel/component/as2/api/AS2MessageTest.java
index 9d84709..f17e7c4 100644
--- a/components/camel-as2/camel-as2-api/src/test/java/org/apache/camel/component/as2/api/AS2MessageTest.java
+++ b/components/camel-as2/camel-as2-api/src/test/java/org/apache/camel/component/as2/api/AS2MessageTest.java
@@ -39,6 +39,7 @@ import org.apache.camel.component.as2.api.entity.DispositionNotificationMultipar
 import org.apache.camel.component.as2.api.entity.MimeEntity;
 import org.apache.camel.component.as2.api.entity.MultipartSignedEntity;
 import org.apache.camel.component.as2.api.entity.TextPlainEntity;
+import org.apache.camel.component.as2.api.util.AS2Utils;
 import org.apache.camel.component.as2.api.util.EntityUtils;
 import org.apache.camel.component.as2.api.util.HttpMessageUtils;
 import org.apache.camel.component.as2.api.util.MicUtils;
@@ -207,7 +208,7 @@ public class AS2MessageTest {
                     org.apache.camel.component.as2.api.entity.EntityParser.parseAS2MessageEntity(request);
                     context.setAttribute(AS2ServerManager.SUBJECT, SUBJECT);
                     context.setAttribute(AS2ServerManager.FROM, AS2_NAME);
-                    LOG.debug(Util.printMessage(request));
+                    LOG.debug(AS2Utils.printMessage(request));
                 } catch (Exception e) {
                     throw new HttpException("Failed to parse AS2 Message Entity", e);
                 }
@@ -447,7 +448,7 @@ public class AS2MessageTest {
         HttpMessageUtils.setHeaderValue(request, AS2Header.DATE, httpdate);
         HttpMessageUtils.setHeaderValue(request, AS2Header.AS2_TO, AS2_NAME);
         HttpMessageUtils.setHeaderValue(request, AS2Header.AS2_FROM, AS2_NAME);
-        String originalMessageId = Util.createMessageId(SERVER_FQDN);
+        String originalMessageId = AS2Utils.createMessageId(SERVER_FQDN);
         HttpMessageUtils.setHeaderValue(request, AS2Header.MESSAGE_ID, originalMessageId);
         HttpMessageUtils.setHeaderValue(request, AS2Header.DISPOSITION_NOTIFICATION_OPTIONS,
                 DISPOSITION_NOTIFICATION_OPTIONS);
@@ -500,7 +501,7 @@ public class AS2MessageTest {
         ReceivedContentMic mdnMic = mdnEntity.getReceivedContentMic();
         assertEquals("Unexpected value for Recieved Content Mic", expectedMic.getEncodedMessageDigest(),
                 mdnMic.getEncodedMessageDigest());
-        LOG.debug("\r\n" + Util.printMessage(mndRequest));
+        LOG.debug("\r\n" + AS2Utils.printMessage(mndRequest));
     }
 
 }
diff --git a/components/camel-as2/camel-as2-component/src/test/java/org/apache/camel/component/as2/AS2ClientManagerIntegrationTest.java b/components/camel-as2/camel-as2-component/src/test/java/org/apache/camel/component/as2/AS2ClientManagerIntegrationTest.java
index df55e92..899399f 100644
--- a/components/camel-as2/camel-as2-component/src/test/java/org/apache/camel/component/as2/AS2ClientManagerIntegrationTest.java
+++ b/components/camel-as2/camel-as2-component/src/test/java/org/apache/camel/component/as2/AS2ClientManagerIntegrationTest.java
@@ -40,7 +40,6 @@ import org.apache.camel.component.as2.api.AS2MimeType;
 import org.apache.camel.component.as2.api.AS2ServerConnection;
 import org.apache.camel.component.as2.api.AS2ServerManager;
 import org.apache.camel.component.as2.api.AS2SignedDataGenerator;
-import org.apache.camel.component.as2.api.Util;
 import org.apache.camel.component.as2.api.entity.AS2DispositionModifier;
 import org.apache.camel.component.as2.api.entity.AS2DispositionType;
 import org.apache.camel.component.as2.api.entity.AS2MessageDispositionNotificationEntity;
@@ -50,6 +49,7 @@ import org.apache.camel.component.as2.api.entity.DispositionMode;
 import org.apache.camel.component.as2.api.entity.DispositionNotificationMultipartReportEntity;
 import org.apache.camel.component.as2.api.entity.MimeEntity;
 import org.apache.camel.component.as2.api.entity.MultipartSignedEntity;
+import org.apache.camel.component.as2.api.util.AS2Utils;
 import org.apache.camel.component.as2.api.util.EntityUtils;
 import org.apache.camel.component.as2.api.util.HttpMessageUtils;
 import org.apache.camel.component.as2.api.util.MicUtils;
@@ -378,7 +378,7 @@ public class AS2ClientManagerIntegrationTest extends AbstractAS2TestSupport {
         HttpMessageUtils.setHeaderValue(request, AS2Header.DATE, httpdate);
         HttpMessageUtils.setHeaderValue(request, AS2Header.AS2_TO, AS2_NAME);
         HttpMessageUtils.setHeaderValue(request, AS2Header.AS2_FROM, AS2_NAME);
-        String originalMessageId = Util.createMessageId(SERVER_FQDN);
+        String originalMessageId = AS2Utils.createMessageId(SERVER_FQDN);
         HttpMessageUtils.setHeaderValue(request, AS2Header.MESSAGE_ID, originalMessageId);
         HttpMessageUtils.setHeaderValue(request, AS2Header.DISPOSITION_NOTIFICATION_OPTIONS,
                 DISPOSITION_NOTIFICATION_OPTIONS);