xmidt-org / wrp-go

go implementation of the Web Routing Protocol
Apache License 2.0
4 stars 7 forks source link

session id header is missing from the delivered event #66

Closed ilawjr closed 2 years ago

ilawjr commented 2 years ago

Several fields from submitted event are missing from the final delivered event. Including: headers, rdr, service_name, url, session_id.

Example using session_id

  1. set up listener (basin in this case)
  2. submit an event to caduceus

{"msg_type":4,"source":"dns:talaria.xmidt.example.com","dest":"event: xmidt-test","transaction_uuid": "33b5224b-877d-43e9-a8ba-de235f844538","content_type": "json","accept": "","status": 504,"rdr": 0,"headers": ["a:b","b:c","d:e"],"metadata": {"/boot-time":"1542834188","/last-reconnect-reason":"spanish inquisition"},"spans": [["Luke","Skywalker","1234", "5678","0"],["Lord","Vader","1234","5678","1"] ],"include_spans": true,"path": "/online","payload": "[redacted]","service_name": "qa","url":"updown","partner_ids":["comcast"],"session_id": "1wksX1ckVakUcgIzXh0S1sSHvK8"}

  1. received the following event from caduceus:

{"Method":"POST","URL":"/api/v1/responder","Body":"[redacted]","Headers":{"Accept-Encoding":["gzip"],"Content-Length":["164"],"Content-Type":["json"],"User-Agent":["Go-http-client/1.1"],"X-Webpa-Device-Id":["dns:talaria.xmidt.example.com"],"X-Webpa-Device-Name":["dns:talaria.xmidt.example.com"],"X-Webpa-Event":["xmidt-test"],"X-Webpa-Signature":["sha1=01c9014ea319ff29a5ffa2c6650a992bc9b1370b"],"X-Webpa-Transaction-Id":["33b5224b-877d-43e9-a8ba-de235f844538"],"X-Xmidt-Include-Spans":["true"],"X-Xmidt-Message-Type":["SimpleEvent"],"X-Xmidt-Metadata":["/boot-time=1542834188","/last-reconnect-reason=spanish inquisition"],"X-Xmidt-Partner-Id":["comcast"],"X-Xmidt-Path":["/online"],"X-Xmidt-Request-Delivery-Response":["0"],"X-Xmidt-Source":["dns:talaria.xmidt.example.com"],"X-Xmidt-Span":["Luke,Skywalker,1234,5678,0","Lord,Vader,1234,5678,1"],"X-Xmidt-Status":["504"],"X-Xmidt-Transaction-Uuid":["33b5224b-877d-43e9-a8ba-de235f844538"]}}

the session_id is missing. We expect to see a header with the value - something similar to: X-Xmidt-Session-Id

ilawjr commented 2 years ago

updated original issue to include other missing fields: headers rdr service_name url session_id