diff --git a/CHANGELOG.md b/CHANGELOG.md index 217a7626567..a6cfae392e8 100644 --- a/CHANGELOG.md +++ b/CHANGELOG.md @@ -9,6 +9,10 @@ - Add periodic re-authentication with the upstream server, previously there was only one initial authentication. ([#731](https://github.com/getsentry/relay/pull/731)) - The module attribute on stack frames (`$frame.module`) and the (usually serverside-generated) attribute `culprit` can now be scrubbed with advanced data scrubbing. ([#744](https://github.com/getsentry/relay/pull/744)) +**Bug Fixes**: + +- Send requests to the `/envelope/` endpoint instead of the older `/store/` endpoint. This particularly fixes spurious `413 Payload Too Large` errors returned when using Relay with Sentry SaaS. ([#746](https://github.com/getsentry/relay/pull/746)) + **Internal**: - Remove a temporary flag from attachment kafka messages indicating rate limited crash reports to Sentry. This is now enabled by default. ([#718](https://github.com/getsentry/relay/pull/718)) @@ -30,7 +34,6 @@ - Fix hashing of user IP addresses in data scrubbing. Previously, this could create invalid IP addresses which were later rejected by Sentry. Now, the hashed IP address is moved to the `id` field. ([#692](https://github.com/getsentry/relay/pull/692)) - Do not retry authentication with the upstream when a client error is reported (status code 4XX). ([#696](https://github.com/getsentry/relay/pull/696)) - **Internal**: - Extract the event `timestamp` from Minidump files during event normalization. ([#662](https://github.com/getsentry/relay/pull/662)) diff --git a/relay-server/src/actors/events.rs b/relay-server/src/actors/events.rs index 043351d6756..7dbaa88fa87 100644 --- a/relay-server/src/actors/events.rs +++ b/relay-server/src/actors/events.rs @@ -1432,7 +1432,7 @@ impl Handler for EventManager { } log::trace!("sending event to sentry endpoint"); - let request = SendRequest::post(format!("/api/{}/store/", project_id)).build( + let request = SendRequest::post(format!("/api/{}/envelope/", project_id)).build( move |builder| { // Override the `sent_at` timestamp. Since the event went through basic // normalization, all timestamps have been corrected. We propagate the new diff --git a/tests/integration/fixtures/mini_sentry.py b/tests/integration/fixtures/mini_sentry.py index 87bf854c2e4..519942c5894 100644 --- a/tests/integration/fixtures/mini_sentry.py +++ b/tests/integration/fixtures/mini_sentry.py @@ -127,7 +127,7 @@ def store_internal_error_event(): ) return jsonify({"event_id": uuid.uuid4().hex}) - @app.route("/api/42/store/", methods=["POST"]) + @app.route("/api/42/envelope/", methods=["POST"]) def store_event(): if flask_request.headers.get("Content-Encoding", "") == "gzip": data = gzip.decompress(flask_request.data) @@ -144,6 +144,7 @@ def store_event(): return jsonify({"event_id": uuid.uuid4().hex}) @app.route("/api//store/", methods=["POST"]) + @app.route("/api//envelope/", methods=["POST"]) def store_event_catchall(project): raise AssertionError(f"Unknown project: {project}")