fix(okhttp): keep the wrapped EventListener per Call by markushi · Pull Request #6003 · getsentry/sentry-java · GitHub

📜 Description

SentryOkHttpEventListener held the wrapped EventListener in a single mutable field that
callStart overwrote for each call. It is now kept in a per-Call map, the same pattern the class
already uses for eventMap. No public API change.

💡 Motivation and Context

OkHttp uses one listener instance for all calls, thus concurrent calls were all delegated to the
listener made for the call that started last. This breaks the EventListener.Factory contract and
loses the terminal callEnd/callFailed of every overlapping call.

resolves: SentryOkHttpEventListener breaks EventListener.Factory contract #5967
resolves: JAVA-695

💚 How did you test it?

Added unit tests.

📝 Checklist

I added GH Issue ID & Linear ID
I added tests to verify the changes.
No new PII added or SDK only sends newly added PII if sendDefaultPII is enabled.
I updated the docs if needed.
I updated the wizard if needed.
Review from the native team if needed.
No breaking change or entry added to the changelog.
No breaking change for hybrid SDKs or communicated to hybrid SDKs.
Public API changes reviewed by another Mobile SDK team member or implemented according to the develop docs spec.

🔮 Next steps

Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>

JAVA-695

Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>

📲 Install Builds

Android

🔗 App Name	App ID	Version	Configuration
SDK Size	io.sentry.tests.size	8.57.0 (1)	release

⚙️ sentry-android Build Distribution Settings

Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>

Move the okhttp changelog entry into a new Unreleased section, as 8.54.0 was released on main. Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>

Thanks for this 💯 !

One comment worth addressing; otherwise looking good.

Keep both Unreleased changelog entries. Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>

Excellent! One tweak more to satisfy the EventListener.Factory contract, and I think we'll be there 🥇

Cursor Bugbot has reviewed your changes and found 2 potential issues.

❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.

Want reviews to match your repository better? Bugbot Learning can learn team-specific rules from PR activity. A team admin can enable Learning in the Cursor dashboard.

Reviewed by Cursor Bugbot for commit 728129d. Configure here.

…r for them

Lurking - hi @0xadam-brown! Crazy seeing you around, hope you're doing well!

Lurking - hi @0xadam-brown! Crazy seeing you around, hope you're doing well!

@victorlai-stripe – so good to see your name show up here! Definitely doing well (and enjoying Sentry). Hope Stripe's been an excellent change-up on your end too 💯 !

A few quick comments, but no blockers.

…ion tests

fix(okhttp): keep the wrapped EventListener per Call

105211a

Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>

docs: add changelog entry

14f6b50

Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>

refactor(okhttp): look up the wrapped EventListener once per callback

11786ec

Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>

markushi added the sanity-check PR needs a lightweight review for obvious issues label Aug 26, 2026

markushi marked this pull request as ready for review August 26, 2026 10:00

markushi requested review from 0xadam-brown, adinauer, romtsn and runningcode as code owners August 26, 2026 10:00

chore: merge origin/main into fix/okhttp-event-listener-per-call

3506dcc

Move the okhttp changelog entry into a new Unreleased section, as 8.54.0 was released on main. Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>

0xadam-brown reviewed Aug 27, 2026

View reviewed changes

Comment thread sentry-okhttp/src/main/java/io/sentry/okhttp/SentryOkHttpEventListener.kt Outdated

chore: merge origin/main into fix/okhttp-event-listener-per-call

247ee38

Keep both Unreleased changelog entries. Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>

sentry Bot reviewed Aug 28, 2026

View reviewed changes

Comment thread sentry-okhttp/src/main/java/io/sentry/okhttp/SentryOkHttpEventListener.kt

fix(okhttp): delegate canceled() outside of the call window

b508070

markushi force-pushed the fix/okhttp-event-listener-per-call branch from bb23afd to b508070 Compare August 28, 2026 06:58

markushi requested a review from 0xadam-brown August 28, 2026 07:10

0xadam-brown reviewed Aug 28, 2026

View reviewed changes

Comment thread sentry-okhttp/src/main/java/io/sentry/okhttp/SentryOkHttpEventListener.kt Outdated

Comment thread sentry-okhttp/src/test/java/io/sentry/okhttp/SentryOkHttpEventListenerDelegationTest.kt Outdated

markushi added 2 commits August 28, 2026 10:07

chore: merge origin/main into fix/okhttp-event-listener-per-call

552a731

fix(okhttp): bind out-of-window cancels to the listener of their call

728129d

sentry Bot reviewed Aug 31, 2026

View reviewed changes

Comment thread sentry-okhttp/src/main/java/io/sentry/okhttp/SentryOkHttpEventListener.kt Outdated

cursor Bot reviewed Aug 31, 2026

View reviewed changes

Comment thread sentry-okhttp/src/main/java/io/sentry/okhttp/SentryOkHttpEventListener.kt Outdated

Comment thread sentry-okhttp/src/main/java/io/sentry/okhttp/SentryOkHttpEventListener.kt

fix(okhttp): drop out-of-window cancels instead of creating a listene…

7b7f12e

…r for them

markushi added 3 commits September 15, 2026 16:48

fix(okhttp): avoid computeIfAbsent to keep Android API 21 compatibility

0600e6c

Merge branch 'main' into fix/okhttp-event-listener-per-call

841d1f5

Fix Changelog

af59e38

0xadam-brown reviewed Sep 16, 2026

View reviewed changes

Comment thread sentry-okhttp/src/main/java/io/sentry/okhttp/SentryOkHttpEventListener.kt

Comment thread sentry-okhttp/src/test/java/io/sentry/okhttp/SentryOkHttpEventListenerDelegationTest.kt

0xadam-brown approved these changes Sep 17, 2026

View reviewed changes

markushi added 3 commits September 18, 2026 12:37

docs(okhttp): document that canceled() is not always delegated

9a88456

Merge branch 'main' into fix/okhttp-event-listener-per-call

c53a1ef

docs(okhttp): note the deliberate canceled() deviation in the delegat…

f3ddd8c

…ion tests

markushi merged commit 8f0dc10 into main Sep 18, 2026
70 of 71 checks passed

markushi deleted the fix/okhttp-event-listener-per-call branch September 18, 2026 11:36

fix(okhttp): keep the wrapped EventListener per Call by markushi · Pull Request #6003 · getsentry/sentry-java · GitHub

Uh oh!

fix(okhttp): keep the wrapped EventListener per Call - #6003

fix(okhttp): keep the wrapped EventListener per Call#6003markushi merged 15 commits intomaingetsentry/sentry-java:mainfrom fix/okhttp-event-listener-per-callgetsentry/sentry-java:fix/okhttp-event-listener-per-callCopy head branch name to clipboard

Conversation

markushi commented Aug 26, 2026

📜 Description

💡 Motivation and Context

💚 How did you test it?

📝 Checklist

🔮 Next steps

Uh oh!

linear-code Bot commented Aug 26, 2026

Uh oh!

sentry Bot commented Aug 26, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

📲 Install Builds

Android

Uh oh!

0xadam-brown left a comment

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Uh oh!

0xadam-brown left a comment

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Uh oh!

Uh oh!

cursor Bot left a comment

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Uh oh!

victorlai-stripe commented Sep 3, 2026

Uh oh!

0xadam-brown commented Sep 6, 2026

Uh oh!

0xadam-brown left a comment

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

3 participants

fix(okhttp): keep the wrapped EventListener per Call#6003
markushi merged 15 commits into
mainfrom
fix/okhttp-event-listener-per-call

sentry Bot commented Aug 26, 2026 •
edited
Loading