1
89%
master: 89%

Ran 05 Dec 2025 11:24AM UTC

Files 623

Run time 15s

Badge

Embed ▾

Committed 05 Dec 2025 11:11AM UTC coverage: 88.639% (-0.008%) from 88.647%

Job # #20105.1

Build Type

push

github

Committed by

web-flow

Commit Message

okhttp: Fix race condition overwriting MAX_CONCURRENT_STREAMS (#12548)

### What this PR does

This PR fixes a race condition in `OkHttpClientTransport` where
`MAX_CONCURRENT_STREAMS` sent by the server could be incorrectly
overwritten by the client's default initialization.

The fix simply reorders the initialization to happen **before** starting
the reader thread, ensuring that any updates from the server are
preserved.

### Note on Testing

I attempted to add a deterministic reproduction test, but reliably
simulating this specific race condition proved difficult without
intrusive changes.
I request reviewers to primarily verify the logical correctness of the
reordering. I am open to collaborating with the team to develop a
suitable test case if required.


### Future Work

This PR covers **Step 1** (Fixing the race condition) of the plan
discussed in #11985.
I plan to follow up with **Step 2** (Adding assertions to verify no
pending streams exist) in a separate PR.

Part of #11985

Run Details

35173 of 39681 relevant lines covered (88.64%)

0.89 hits per line

grpc / grpc-java / #20105 / 1
89%
master: 89%

README BADGES
x

Markdown

Textile

RDoc

HTML

Rst

Source Files on job #20105.1

grpc / grpc-java / #20105 / 1 89% master: 89%

README BADGES x

Markdown

Textile

RDoc

HTML

Rst

Source Files on job #20105.1

grpc / grpc-java / #20105 / 1
89%
master: 89%

README BADGES
x