GitHub user tadayosi opened a pull request:
https://github.com/apache/camel/pull/1253
CAMEL-10446 - Need to consolidate header mapping logic between Camel and
CXF messages
https://issues.apache.org/jira/browse/CAMEL-10446
My initial attempt for the refactoring: It moves all the header mapping
logic in `DefaultCxfRsBinding` into `CxfHeaderHelper` including the
`camelToCxfHeaderMap` and `cxfToCamelHeaderMap` fields. It also introduces Java
8 API and lambdas so that the code looks clearer. `CxfHeaderHelperTest` is
enhanced as well in order to make sure Camel headers are not tainted with
CXF-specific headers and vice versa.
There is still room for refactoring, however: both `DefaultCxfBinding` and
`DefaultCxfRsBinding` still have a bit of header mapping logic yet! I may
pursue another pull req for the refactoring.
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/tadayosi/camel CAMEL-10446
Alternatively you can review and apply these changes as the patch at:
https://github.com/apache/camel/pull/1253.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 #1253
----
commit 6041d9c897d15ee7e820556c6d020c5d306d87eb
Author: Tadayoshi Sato <[email protected]>
Date: 2016-11-07T13:15:20Z
CAMEL-10446 - Need to consolidate header mapping logic between Camel and
CXF messages
----
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at [email protected] or file a JIRA ticket
with INFRA.
---
