feat: Added AddAttachment to the Hint

[bitsandfoxes]

Resolves: #2549

Taken from the API we have on the scope to add an attachment. So instead of

options.SetBeforeSend((@event, hint) =>
{
    hint.Attachments.Add(new Attachment(
        AttachmentType.Default,
        new FileAttachmentContent("<filepath>", true),
        Path.GetFileName("<filepath>"),
        null));
    return @event;
});

we can do

options.SetBeforeSend((@event, hint) =>
{
    hint.AddAttachment("<filepath>");
    return @event;
});

[github-actions]

	Messages
📖	Do not forget to update Sentry-docs with your feature once the pull request gets approved.

Generated by 🚫 dangerJS against 89d49a0

[jamescrosswell]

@bitsandfoxes do hints get sent to Sentry? I thought they were just there for context prior to capturing an event (so useful in the BeforeSend callback but not afterwards)...

[bitsandfoxes]

@bitsandfoxes do hints get sent to Sentry? I thought they were just there for context prior to capturing an event (so useful in the BeforeSend callback but not afterwards)...

Yes they do. Hints have a bit of a mixed responsibility but the attachment part is supposed to be the answer to "how do I add attachments in the BeforeSend callback?".

[jamescrosswell]

Looks like we have good basic tests to ensure attachments can be added to the Hints.

Maybe worth adding a test to the Hub to ensure Hint.Attachments make their way through to Sentry when capturing an event?

[bitsandfoxes]

Good catch! Thanks for the callout!