Repository navigation
Restore the previous request name after temporary grouping - #3583
Merged
Merged
Conversation
Collaborator
|
Nice! |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Change
Restore the previous
request_namewhenrename_request()exits. With nested groupings, requests after the inner context currently lose the outer name and are grouped under their URLs. A temporarily overridden explicit name is also lost. Preserve it on normal exit and exceptions inHttpSession,FastHttpSessionand the experimentalOpenAIClient.This extends the inherited reset-to-None behavior retained by #3532. It adds no public API or dependencies. Documentation now explains restoration and nesting.
Validation
index.html/report.htmlassets. Telemetry tests pass on both after installing the declared test extras.The full suite and longer runner runs exceeded the execution limit; complete full-suite coverage and a green full-suite result are not claimed. Local evidence applies to
2564e6don0d61b12.Prepared with AI assistance; the final diff and paired validation evidence were reviewed.