Scope::set_user does not affect users reported in performance transactions

[antifuchs]

Environment

I'm instrumenting an axum web app, using the default set of tower middleware, tags / user fields set using sentry::configure_scope land on error reporting events, but they are missing from Performance transactions.

Steps to Reproduce

1. Set up the default sentry_tower layers:

       .layer(sentry_tower::NewSentryLayer::new_from_top())
       .layer(sentry_tower::SentryHttpLayer::with_transaction())

2. In a handler (or another layer, it doesn't matter) sets user information or tags on the scope using the (docs-recommended) method, like:

       sentry::configure_scope(|scope| {
           scope.set_tag("lz_transaction", "woo weeee");
           scope.set_user(Some(User {
               id: Some(user.id.id().to_string()),
               username: Some(user.name.to_owned()),
               ..Default::default()
           }));
       });

3. Call up a normally-succeeding handler

4. Put panic!("oops"); in that handler to get an error event.

Expected Result

I'd like the transactions list in Performance to list Users and other tags, like sentry does in error events.

Actual Result

The error event has a user set, but the performance transaction doesn't.

The error event has the user:

perf monitoring list:

[Swatinem]

This should have been implemented a long time ago in https://github.com/getsentry/sentry-rust/pull/596. It might be possible that the transaction is not using the correct Hub/Scope, or the problem might be elsewhere.

[antifuchs]

Oh, I forgot to mention: using scope.set_transaction to name the transaction does make it to the performance list; but the rest of the data associated with the scope doesn't. (I did remove that set_transaction call when writing my tests, thinking it would help; it didn't...)

[ollipa]

I had a similar issue. I was setting the user in a spawned task. When I spawned the task, I was calling .bind_hub(Hub::new_from_top(Hub::current())) on it. After changing it to .bind_hub(Hub::current()) I could see user in performance traces.

I thought that creating a new Hub would also work and using the current Hub would not be required.