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

Claus Ibsen commented on CAMEL-26550:
-------------------------------------

PR: https://github.com/apache/camel/pull/29277

Rejecting the combination at start would break the documented server-streaming 
example (routeControlledStreamObserver=true with the default PROPAGATION) and 
two tests, so the PR supports it instead: with 
routeControlledStreamObserver=true the PROPAGATION observer passes the 
route-controlled exchange properties and leaves the replies and the end of the 
call to the route, as DELEGATION does, while still routing the events one at a 
time, in order.

_Claude Code on behalf of Claus Ibsen_

> camel-grpc - with consumerStrategy=PROPAGATION and 
> routeControlledStreamObserver=true the route cannot control the reply
> ------------------------------------------------------------------------------------------------------------------------
>
>                 Key: CAMEL-26550
>                 URL: https://issues.apache.org/jira/browse/CAMEL-26550
>             Project: Camel
>          Issue Type: Bug
>            Reporter: Claus Ibsen
>            Assignee: Claus Ibsen
>            Priority: Minor
>             Fix For: 4.24.0
>
>
> Found while fixing CAMEL-26515 (PR #29112).
> {{GrpcConsumer}} accepts {{consumerStrategy=PROPAGATION}} together with 
> {{routeControlledStreamObserver=true}}. For client and bidi streaming calls, 
> {{GrpcRequestPropagationStreamObserver}} (lines 41 and 61) never sets the 
> {{grpcResponseObserver}} exchange property, and still sends the route body as 
> the reply itself. So a route that uses the 
> {{toRouteControlledStreamObserver}} producer fails (no response observer), 
> and the reply is not controlled by the route.
> The docs say to use {{DELEGATION}} for a route-controlled stream observer.
> Options:
> * reject the combination when the consumer starts, with a clear message 
> pointing to DELEGATION (recommended), or
> * support it: set the property and do not send the body in the propagation 
> observer.
> _Claude Code on behalf of Claus Ibsen (davsclaus)_



--
This message was sent by Atlassian Jira
(v10.3.26#10030026)

Reply via email to