[ 
https://issues.apache.org/jira/browse/CAMEL-12605?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16584660#comment-16584660
 ] 

ASF GitHub Bot commented on CAMEL-12605:
----------------------------------------

davsclaus closed pull request #2482: [CAMEL-12605] Refactored name and package 
of AS2 utilities
URL: https://github.com/apache/camel/pull/2482
 
 
   

This is a PR merged from a forked repository.
As GitHub hides the original diff on merge, it is displayed below for
the sake of provenance:

As this is a foreign pull request (from a fork), the diff is supplied
below (as it won't show otherwise due to GitHub magic):

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 be1ee6a4bb6..852067c7710 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.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 void process(HttpRequest request, HttpContext context) 
throws HttpExcepti
         /* 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 void process(HttpRequest request, HttpContext context) 
throws HttpExcepti
         /* 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 void process(HttpRequest request, HttpContext context) 
throws HttpExcepti
 
         /* 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 c4276288675..ed03fbbf820 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 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 void process(HttpRequest request, HttpContext context) 
throws HttpExcepti
 
         /* 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 98a72fa0400..e0aecbdf81d 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.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 void process(HttpResponse response, HttpContext 
context) throws HttpExcep
             /* 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 void process(HttpResponse response, HttpContext 
context) throws HttpExcep
             /* 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 void process(HttpResponse response, HttpContext 
context) throws HttpExcep
 
             /* 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 void process(HttpResponse response, HttpContext 
context) throws HttpExcep
             }
         }
 
-        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 3d8f4835f7c..d4c31616a6d 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.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 @@
 /**
  * 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 @@
 
     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 9d84709770f..f17e7c45826 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.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 void handle(HttpRequest request, HttpResponse 
response, HttpContext conte
                     
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 void asynchronousMdnMessageTest() throws Exception {
         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 void asynchronousMdnMessageTest() throws Exception {
         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 df55e9215f3..899399f66b9 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.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.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 void asyncMDNTest() throws Exception {
         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);


 

----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


> Enhance the AS2 Component to send and receive encrypted AS2 messages
> --------------------------------------------------------------------
>
>                 Key: CAMEL-12605
>                 URL: https://issues.apache.org/jira/browse/CAMEL-12605
>             Project: Camel
>          Issue Type: Improvement
>    Affects Versions: 2.23.0
>            Reporter: William Collins
>            Assignee: William Collins
>            Priority: Major
>
> Enhance the AS2 Component to support encrypted AS2 messages per RFC4130



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)

Reply via email to