SwitchYard
  1. SwitchYard
  2. SWITCHYARD-710

Allow direct storage of XML Nodes as Context properties from ContextMappers

    Details

    • Affects:
      Documentation (Ref Guide, User Guide, etc.), Compatibility/Configuration, Release Notes
    • Similar Issues:
      Show 10 results 

      Description

      What started as a discussion regarding the behavior of SOAPContextMapper I believe should be a broader-scoped enhancement to ContextMappers in general.

      Right now, soap header elements are being deconstructed by using their name as the context property key, and their string value as the context property value. This does not take into consideration more complex use cases such as when an entire XML (sub-)tree needs to be stored for things like xml TX/security propagation, as required by SWITCHYARD-703. Thus a configuration property on <contextMapper/> should be added so that the entire Node can be carried along as-is.

      I do believe that the default behavior, however, should be kept as it is now, since for most cases I think just a simple value is sufficient.

      How is this a bigger issue than SOAPContextMappers? Well, we need to go through the other implementations to make sure values won't be assumed as simple Strings all the time. One thing I noticed right away is the HornetQContextMapper, which gets values out using source.getStringProperty instead of getObjectProperty. We need consistency across the ContextMappers.

        Issue Links

          Activity

          Hide
          David Ward
          added a comment - - edited

          will issue new pull requests

          Show
          David Ward
          added a comment - - edited will issue new pull requests
          Show
          David Ward
          added a comment - https://github.com/jboss-switchyard/parent/pull/71 https://github.com/jboss-switchyard/core/pull/380 https://github.com/jboss-switchyard/components/pull/323 https://github.com/jboss-switchyard/quickstarts/pull/126 https://github.com/jboss-switchyard/release/pull/139
          Hide
          Keith Babo
          added a comment -

          pushed

          Show
          Keith Babo
          added a comment - pushed

            People

            • Assignee:
              David Ward
              Reporter:
              David Ward
            • Votes:
              0 Vote for this issue
              Watchers:
              2 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved: