relay icon indicating copy to clipboard operation
relay copied to clipboard

Form Data events are dropped

Open Dav1dde opened this issue 1 year ago • 1 comments

Form Data events are dropped in the store.

https://github.com/getsentry/relay/blob/3bd026610b483e3c1120238b7e1aa5dd0feecc4d/relay-server/src/services/store.rs#L283-L302

S4S Issue

Dav1dde avatar Mar 28 '24 15:03 Dav1dde

I added some more metadata to the error, but I don't see a pattern. We might need to attach the entire incoming envelope to the error to see if this is a bug or just custom SDK setups sending invalid envelopes.

jjbayer avatar Apr 24 '24 08:04 jjbayer

Option: Just drop it in the store.

Dav1dde avatar Dec 11 '24 13:12 Dav1dde

This test replicates the error:

def test_attachments_with_form_data(
    mini_sentry, relay_with_processing, attachments_consumer, outcomes_consumer
):
    project_id = 42
    event_id = "515539018c9b4260a6f999572f1661ee"

    mini_sentry.add_full_project_config(project_id)
    relay = relay_with_processing()
    attachments_consumer = attachments_consumer()
    outcomes_consumer = outcomes_consumer()

    # Create attachments with both file content and form data
    attachments = [
        ("att_1", "foo.txt", b"file content"),
        ("form_key", None, b"form value")  # Form data entry
    ]

    relay.send_attachments(project_id, event_id, attachments)

iambriccardo avatar Dec 20 '24 13:12 iambriccardo

The problem was found, we are discussing offline what is the expected handling of form data. If it should be converted into its own event or produced as attachment.

iambriccardo avatar Dec 30 '24 07:12 iambriccardo