nicolaferraro opened a new issue #173: Wrong CloudEvent Headers used
URL: https://github.com/apache/camel-k-runtime/issues/173
 
 
   Since the refactoring, cloudevent headers are not correctly mapped to the 
outgoing http transport.
   
   Knative source -> dumper service
   ```
   POST / HTTP/1.1
   Host: dumper.default.svc.cluster.local
   User-Agent: Vert.x-WebClient/3.8.1
   Content-Length: 12
   Accept-Encoding: gzip
   Firedtime: Thu Oct 24 22:01:22 UTC 2019
   Forwarded: for=172.17.0.26;proto=http, for=172.17.0.10
   Id: ID-camel-timer-flow-source-ttxh8-74d54d878f-4pbzr-1571954479485-0-2
   K-Proxy-Request: activator
   Source: knative://endpoint/sink
   Time: 2019-10-24T22:01:22.462Z
   Type: org.apache.camel.event
   Version: 0.3
   X-B3-Sampled: 1
   X-B3-Spanid: 42fd7e6b20d6a81e
   X-B3-Traceid: deb59b83db4f881842fd7e6b20d6a81e
   X-Envoy-Decorator-Operation: dumper-btstw.default.svc.cluster.local:80/*
   X-Envoy-Expected-Rq-Timeout-Ms: 600000
   X-Envoy-Internal: true
   X-Forwarded-For: 172.17.0.26, 172.17.0.10, 172.17.0.11
   X-Forwarded-Proto: http
   X-Request-Id: fdf24e47-7fba-9572-9e67-2f17fdef00b7
   
   Hello world!
   ```
   
   Notice the lines:
   ```
   Source: knative://endpoint/sink
   Type: org.apache.camel.event
   Version: 0.3
   ```
   
   They are missing the "CE" prefix, and they are also wrong, since 0.3 should 
be contained in the "ce-specversion" header, not "ce-version".
   
   To avoid any issues with libraries preprocessing headers, I've dumped the 
request using this (raw) tool: https://github.com/nicolaferraro/dumper
   
   Apart from the bug, I think the "ce-" prefix should never be trimmed. Wdyt 
@lburgazzoli ?
   
   

----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services

Reply via email to