Petals SE Camel

Mixup with Exchange Properties between Petals Consumers and Providers

Details

  • Type: Bug Bug
  • Status: Resolved Resolved
  • Priority: Major Major
  • Resolution: Fixed
  • Affects Version/s: 0.5.0
  • Fix Version/s: 0.5.1, 0.5.2, 1.0.1
  • Component/s: None
  • Security Level: Public
  • Description:
    Hide

    When a Petals exchange arrives on the Camel SE, its properties (not the message properties, that are business-specific but the exchange's) are copied to the Camel exchange.
    When a Camel exchange is sent to Petals, its properties (exchange's again) are copied to the newly sent Petals Exchange.

    There is thus a mixup with Flow Attributes, or CDK async property, etc.

    It is not possible to simply not forward them, because we actually don't know if some properties are not added explicitly by the route (for Camel exchanges sent to Petals), or if the properties set by another Petals service is needed in the route (for Petals exchanges sent received by Camel).

    It is not yet clear how to fix this.

    Show
    When a Petals exchange arrives on the Camel SE, its properties (not the message properties, that are business-specific but the exchange's) are copied to the Camel exchange. When a Camel exchange is sent to Petals, its properties (exchange's again) are copied to the newly sent Petals Exchange. There is thus a mixup with Flow Attributes, or CDK async property, etc. It is not possible to simply not forward them, because we actually don't know if some properties are not added explicitly by the route (for Camel exchanges sent to Petals), or if the properties set by another Petals service is needed in the route (for Petals exchanges sent received by Camel). It is not yet clear how to fix this.
  • Environment:
    -

Activity

Hide
Victor NOËL added a comment - Fri, 11 Sep 2015 - 09:33:37 +0200

For now, we do not transfer properties, see https://github.com/petalslink/petals-se-camel/commit/1e3436960eb28f9990f69aae84044aacbc242739

But we should find a better solution...

Show
Victor NOËL added a comment - Fri, 11 Sep 2015 - 09:33:37 +0200 For now, we do not transfer properties, see https://github.com/petalslink/petals-se-camel/commit/1e3436960eb28f9990f69aae84044aacbc242739 But we should find a better solution...
Victor NOËL made changes - Fri, 11 Sep 2015 - 09:33:37 +0200
Field Original Value New Value
Status New [ 10000 ] Open [ 10002 ]
Priority Major [ 3 ]
Victor NOËL made changes - Fri, 11 Sep 2015 - 09:33:41 +0200
Status Open [ 10002 ] In Progress [ 10003 ]
Hide
Victor NOËL added a comment - Fri, 11 Sep 2015 - 10:03:13 +0200
Show
Victor NOËL added a comment - Fri, 11 Sep 2015 - 10:03:13 +0200 This temporary fix was backported for Petals 4.3.x (see https://github.com/petalslink/petals-se-camel/commit/fb181b9db3c04d39001c20b3cd8d371a76e11c70)
Victor NOËL made changes - Fri, 11 Sep 2015 - 10:03:27 +0200
Fix Version/s 0.5.1 [ 10559 ]
Fix Version/s 1.0.0 [ 10543 ]
Victor NOËL made changes - Wed, 30 Sep 2015 - 16:10:20 +0200
Status In Progress [ 10003 ] Open [ 10002 ]
Victor NOËL made changes - Thu, 1 Oct 2015 - 09:29:27 +0200
Status Open [ 10002 ] In Progress [ 10003 ]
Hide
Victor NOËL added a comment - Thu, 1 Oct 2015 - 09:29:53 +0200

A preliminary fix is in v0.5.1 (and thus will be in 1.0.0 at least).

Show
Victor NOËL added a comment - Thu, 1 Oct 2015 - 09:29:53 +0200 A preliminary fix is in v0.5.1 (and thus will be in 1.0.0 at least).
Victor NOËL made changes - Thu, 1 Oct 2015 - 09:29:53 +0200
Status In Progress [ 10003 ] Resolved [ 10004 ]
Fix Version/s 1.0.0 [ 10543 ]
Resolution Fixed [ 1 ]
Hide
Victor NOËL added a comment - Thu, 1 Oct 2015 - 09:30:28 +0200

Reopening because there is more work to do.

Show
Victor NOËL added a comment - Thu, 1 Oct 2015 - 09:30:28 +0200 Reopening because there is more work to do.
Victor NOËL made changes - Thu, 1 Oct 2015 - 09:30:28 +0200
Status Resolved [ 10004 ] Open [ 10002 ]
Resolution Fixed [ 1 ]
Christophe DENEUX made changes - Thu, 1 Oct 2015 - 11:12:34 +0200
Fix Version/s 1.0.1 [ 10577 ]
Christophe DENEUX made changes - Mon, 23 May 2016 - 12:50:44 +0200
Fix Version/s 1.0.2 [ 10656 ]
Fix Version/s 1.0.1 [ 10577 ]
Victor NOËL made changes - Mon, 12 Sep 2016 - 11:03:43 +0200
Status Open [ 10002 ] In Progress [ 10003 ]
Hide
Victor NOËL added a comment - Mon, 12 Sep 2016 - 11:04:34 +0200 - edited
Show
Victor NOËL added a comment - Mon, 12 Sep 2016 - 11:04:34 +0200 - edited This was actually fixed with PETALSSECAMEL-16, PETALSSECAMEL-17, PETALSSECAMEL-22 and PETALSDISTRIB-282
Victor NOËL made changes - Mon, 12 Sep 2016 - 11:04:34 +0200
Status In Progress [ 10003 ] Resolved [ 10004 ]
Fix Version/s 0.5.2 [ 10576 ]
Fix Version/s 1.0.1 [ 10577 ]
Fix Version/s 1.0.2 [ 10656 ]
Resolution Fixed [ 1 ]
Transition Status Change Time Execution Times Last Executer Last Execution Date
New New Open Open
1m 31s
1
Victor NOËL
Fri, 11 Sep 2015 - 09:33:37 +0200
Open Open In Progress In Progress
4s
1
Victor NOËL
Fri, 11 Sep 2015 - 09:33:41 +0200
In Progress In Progress Open Open
19d 6h 36m
1
Victor NOËL
Wed, 30 Sep 2015 - 16:10:20 +0200
Open Open In Progress In Progress
17h 19m
1
Victor NOËL
Thu, 1 Oct 2015 - 09:29:27 +0200
In Progress In Progress Resolved Resolved
26s
1
Victor NOËL
Thu, 1 Oct 2015 - 09:29:53 +0200
Resolved Resolved Open Open
35s
1
Victor NOËL
Thu, 1 Oct 2015 - 09:30:28 +0200
Open Open In Progress In Progress
347d 1h 33m
1
Victor NOËL
Mon, 12 Sep 2016 - 11:03:43 +0200
In Progress In Progress Resolved Resolved
51s
1
Victor NOËL
Mon, 12 Sep 2016 - 11:04:34 +0200

People

Dates

  • Created:
    Fri, 11 Sep 2015 - 09:32:06 +0200
    Updated:
    Mon, 12 Sep 2016 - 11:05:08 +0200
    Resolved:
    Mon, 12 Sep 2016 - 11:04:34 +0200