Skip to content

Update modules submodule to ee127dd - #1538

Merged
bmehta001 merged 2 commits into
mainfrom
user/khrechykhin/fix-crash-on-response-deadlock
Sep 21, 2026
Merged

bmehta001 merged 2 commits into
mainfrom
user/khrechykhin/fix-crash-on-response-deadlock

Conversation

@kyrylo-hrechykhin-msft

@kyrylo-hrechykhin-msft Kyrylo Hrechykhin (kyrylo-hrechykhin-msft) commented Sep 21, 2026 •

Copy link
Copy Markdown
Contributor

Summary

Updates the lib/modules submodule from 7bd8b516e2d93d1704834e0895733ae7bc2d1f43 to ee127dd18d0c702b55dd662306d09c781c271f3b.

Included modules changes

EXP fix

The EXP change prevents ExpCommon::OnHttpResponse() from re-entering the non-recursive client state mutex when an HTTP transport invokes the response callback synchronously from SendRequestAsync().

Related bug: 5737445

Scope

This PR changes only the lib/modules submodule pointer. The implementation and tests were reviewed and merged independently in the modules repository.

Kyrylo Hrechykhin added 2 commits September 21, 2026 18:02
Advance lib/modules to merged cpp_client_telemetry_modules PR #359, which defers synchronous HTTP response processing to the EXP message worker.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

Copilot-Session: 31ae858c-f4c3-4ab4-8191-a6f7d65ae25b
@kyrylo-hrechykhin-msft Kyrylo Hrechykhin (kyrylo-hrechykhin-msft) changed the title Update modules for EXP HTTP callback deadlock fix Update modules submodule to ee127dd Sep 21, 2026
@bmehta001
bmehta001 enabled auto-merge (squash) September 21, 2026 17:30
@bmehta001
bmehta001 merged commit 186a59c into main Sep 21, 2026
43 of 47 checks passed
@bmehta001
bmehta001 deleted the user/khrechykhin/fix-crash-on-response-deadlock branch September 21, 2026 17:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants