From c668c86f825603a2a4b81b5f504407f17c04b7c2 Mon Sep 17 00:00:00 2001 From: Reinaldy Rafli Date: Sat, 10 Oct 2026 11:00:14 +0700 Subject: [PATCH 1/3] fix(vroom): breaking change on SENTRY_BUCKET_PROFILES Upgraded on https://github.com/getsentry/vroom/pull/721, this causes failing e2e tests Confirmed by reading this: https://pkg.go.dev/gocloud.dev/blob/s3blob?utm_source=godoc#URLOpener, which refers to https://github.com/google/go-cloud/blob/b77c6be2700b2401d3645a7c313b4b848dc54e67/blob/s3blob/s3blob.go#L132-L139 --- docker-compose.yml | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/docker-compose.yml b/docker-compose.yml index 09bfee7a759..de09815f0d9 100644 --- a/docker-compose.yml +++ b/docker-compose.yml @@ -674,7 +674,7 @@ services: environment: SENTRY_KAFKA_BROKERS_PROFILING: "kafka:9092" SENTRY_KAFKA_BROKERS_OCCURRENCES: "kafka:9092" - SENTRY_BUCKET_PROFILES: "s3://profiles?region=us-east-1&endpoint=seaweedfs:8333&s3ForcePathStyle=true&disableSSL=true" + SENTRY_BUCKET_PROFILES: "s3://profiles?region=us-east-1&endpoint=seaweedfs:8333&use_path_style=true&disable_https=true" AWS_ACCESS_KEY: "sentry" AWS_SECRET_KEY: "sentry" SENTRY_SNUBA_HOST: "http://snuba-api:1218" From 763d92176397b9199adb55ddfc67782288ae33b6 Mon Sep 17 00:00:00 2001 From: "sentry-junior[bot]" <264270552+sentry-junior[bot]@users.noreply.github.com> Date: Mon, 12 Oct 2026 01:09:21 +0000 Subject: [PATCH 2/3] fix(tests): Log in through the auth API in integration tests Co-Authored-By: Reinaldy Rafli --- _integration-test/test_01_basics.py | 23 ++++++++++++++--------- 1 file changed, 14 insertions(+), 9 deletions(-) diff --git a/_integration-test/test_01_basics.py b/_integration-test/test_01_basics.py index 7218f857d54..49742bb4444 100644 --- a/_integration-test/test_01_basics.py +++ b/_integration-test/test_01_basics.py @@ -74,16 +74,21 @@ def authenticated_session(): client = httpx.Client() client.get(SENTRY_TEST_HOST, follow_redirects=True) login_csrf_token = client.cookies["sc"] - login_response = client.post( - f"{SENTRY_TEST_HOST}/auth/login/sentry/", - follow_redirects=True, - data={ - "op": "login", - "username": TEST_USER, - "password": TEST_PASS, - "csrfmiddlewaretoken": login_csrf_token, + # Password login goes through the auth API used by the React login page; + # the legacy HTML login form was removed in getsentry/sentry#126712. + api_login_response = client.post( + f"{SENTRY_TEST_HOST}/api/0/auth/login/", + json={"username": TEST_USER, "password": TEST_PASS}, + headers={ + "Referer": f"{SENTRY_TEST_HOST}/auth/login/", + "X-CSRFToken": login_csrf_token, }, - headers={"Referer": f"{SENTRY_TEST_HOST}/auth/login/sentry/"}, + ) + assert api_login_response.status_code == 200, api_login_response.text + login_response = client.get( + SENTRY_TEST_HOST, + follow_redirects=True, + headers={"Referer": SENTRY_TEST_HOST}, ) assert login_response.status_code == 200 parser = BeautifulSoup(login_response.text, "html.parser") From 6c032091040782933d1400e1389a52c245fd6e3e Mon Sep 17 00:00:00 2001 From: "sentry-junior[bot]" <264270552+sentry-junior[bot]@users.noreply.github.com> Date: Mon, 12 Oct 2026 01:20:52 +0000 Subject: [PATCH 3/3] fix(tests): Fetch auth config before API login to set test cookie Co-Authored-By: Reinaldy Rafli --- _integration-test/test_01_basics.py | 7 +++++++ 1 file changed, 7 insertions(+) diff --git a/_integration-test/test_01_basics.py b/_integration-test/test_01_basics.py index 49742bb4444..c72f224fb58 100644 --- a/_integration-test/test_01_basics.py +++ b/_integration-test/test_01_basics.py @@ -76,6 +76,13 @@ def authenticated_session(): login_csrf_token = client.cookies["sc"] # Password login goes through the auth API used by the React login page; # the legacy HTML login form was removed in getsentry/sentry#126712. + # Like the React page, fetch the auth config first: it sets the session + # test cookie that the login endpoint requires. + auth_config_response = client.get( + f"{SENTRY_TEST_HOST}/api/0/auth/config/", + headers={"Referer": f"{SENTRY_TEST_HOST}/auth/login/"}, + ) + assert auth_config_response.status_code == 200, auth_config_response.text api_login_response = client.post( f"{SENTRY_TEST_HOST}/api/0/auth/login/", json={"username": TEST_USER, "password": TEST_PASS},