Consider including MessageId tag in RabbitMQActivitySource during BasicPublish

[YayBurritos]

Is your feature request related to a problem? Please describe.

The Deliver method of RabbitMQActivitySource accepts IReadOnlyBasicProperties as input. This allows the MessageId, if provided, to be added as a tag (span attribute) in the PopulateMessagingTags method:

            if (!string.IsNullOrEmpty(readOnlyBasicProperties.MessageId))
            {
                activity.SetTag(MessageId, readOnlyBasicProperties.MessageId);
            }

However, the BasicPublish method currently doesn't accept IReadOnlyBasicProperties as input, so MessageId can't be included.

Having MessageId as a tag in both the "publish" and "deliver" trace spans will allow users to be able to find traces - both from the publisher and consumer perspective - if they know the MessageId, which could theoretically be obtained from application logs, an audit table, etc.

Describe the solution you'd like

I'd like to update the RabbitMQActivitySource.BasicPublish method to allow for IReadOnlyBasicProperties to be passed in as an additional parameter. That would allow the IReadOnlyBasicProperties to then be passed along to the PopulateMessagingTags method, which would add the MessageId as a tag.

Describe alternatives you've considered

No response

Additional context

No response

[YayBurritos]

I'm happy to open a PR for this. It seems to be a pretty easy update to the RabbitMQActivitySource class and possibly some test updates to projects/Test/SequentialIntegration/TestActivitySource.cs.

@lukebakken : Are you responsible for assigning issues?

[lukebakken]

Generally, with free software projects, maintainers LOVE it when people open PRs on their own, without asking. Especially if they include a good rationale for it (like you have), and if the PRs include tests.

In other words, have at it 😸

[lukebakken]

Implemented in #1852