camel-issues mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "ASF GitHub Bot (JIRA)" <j...@apache.org>
Subject [jira] [Commented] (CAMEL-10510) HL7AcknowledgementGenerator should set exchange property and not message header
Date Tue, 22 Nov 2016 19:05:58 GMT

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

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

GitHub user hqstevenson opened a pull request:

    https://github.com/apache/camel/pull/1296

    CAMEL-10510: Change HL7AcknowledgementGenerator to set CamelMllpAckno…

    …wledgement exchange property instead of message header

You can merge this pull request into a Git repository by running:

    $ git pull https://github.com/hqstevenson/camel CAMEL-10510

Alternatively you can review and apply these changes as the patch at:

    https://github.com/apache/camel/pull/1296.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

    This closes #1296
    
----
commit 8b399ce5f2096e081798466eb11152cd07d1786b
Author: Quinn Stevenson <quinn@pronoia-solutions.com>
Date:   2016-11-22T19:04:04Z

    CAMEL-10510: Change HL7AcknowledgementGenerator to set CamelMllpAcknowledgement exchange
property instead of message header

----


> HL7AcknowledgementGenerator should set exchange property and not message header
> -------------------------------------------------------------------------------
>
>                 Key: CAMEL-10510
>                 URL: https://issues.apache.org/jira/browse/CAMEL-10510
>             Project: Camel
>          Issue Type: Bug
>          Components: camel-mllp
>    Affects Versions: 2.17.0
>            Reporter: Quinn Stevenson
>            Assignee: Quinn Stevenson
>            Priority: Minor
>
> The HL7AcknowledgmentGenerator is setting the CamelMllpAcknowledgement message header,
but this should be an exchange property.
> This is "hangover" from when this class was an internal/implementation class.  Since
I moved it and made it generally available for use, it should be setting the exchange property
so that the acknowledgement will be delivered.
> As currently written, the user will have to copy the message header to the exchange property
for it to be used.  If this isn't done, the component will generate a new acknowledgement.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

Mime
View raw message