In the following snippet, shouldn't the current context be the POST context?

async fn request_quote(count: u32) -> Result<f64, Box<dyn std::error::Error>> {
    // TODO: better testing here and default quote_service_addr
    let quote_service_addr: String = format!(
        "{}{}",
        env::var("QUOTE_SERVICE_ADDR").expect("$QUOTE_SERVICE_ADDR is not set"),
        "/getquote"
    );

    let mut reqbody = HashMap::new();
    reqbody.insert("numberOfItems", count);

    let client = ClientBuilder::new(reqwest::Client::new())
        .with(TracingMiddleware::<SpanBackendWithUrl>::new())
        .build();

    let req = client.request(Method::POST, quote_service_addr);

    let mut headers = HeaderMap::new();

    let cx = Context::current();
    global::get_text_map_propagator(|propagator| {
        propagator.inject_context(&cx, &mut HeaderInjector(&mut headers))
    });

    let resp = req
        .json(&reqbody)
        .headers(headers)
        .send()
        .await?
        .text_with_charset("utf-8")
        .await?;

    debug!("{:?}", resp);

    match resp.parse::<f64>() {
        Ok(f) => Ok(f),
        Err(error) => Err(Box::new(error)),
    }
}

https://github.com/open-telemetry/opentelemetry-demo/blob/main/src/shippingservice/src/shipping_service/quote.rs#L64-L67

@TommyCpp do you spot where we are doing the context propagation wrong here?

Feel free to assign this to me. I can take a look this week

Thank you @TommyCpp!

I think the issue lie with reqwest-tracing create, which priovides a middle layer to create a child span for HTTP calls.

    let client = ClientBuilder::new(reqwest::Client::new())
        .with(TracingMiddleware::<SpanBackendWithUrl>::new())
        .build();

But that happens after we setup the header

    let cx = Context::current();
    global::get_text_map_propagator(|propagator| {
        propagator.inject_context(&cx, &mut HeaderInjector(&mut headers))
    });

and somewhere before the request is sent.

The quick patch would be disable the .with(TracingMiddleware::<SpanBackendWithUrl>::new()) but that means we will delete the POST span from shippingservice. If this can wait I think it's better to fix stuff from reqwest-tracing side

By default the propgation on the POST span should be included automatically by the middleware(as we didn't use DisableOtelPropagation extension here)

Thank you @TommyCpp!

We can definitely wait for the better fix.
This is in the demo for some time already, so it won't be an issue to keep it as is for a bit longer.
Should I create an issue in the reqwest-tracing?
Is that this repo: https://github.com/TrueLayer/reqwest-middleware?

Yep

I can open the issue on their side. I also want to construct some examples between rust client and rust server to help investigation

@TommyCpp did you happen to have opened a ticket on their side?
If not, I could do it.

Do I need to provide any extra info other than what you've detailed here: #1318 (comment)?