Skip to content

fix: sdk-5295 preserve request options and api results - #122

Draft
dciccale wants to merge 2 commits into
masterfrom
fix/sdk-5295-request-options
Draft

fix: sdk-5295 preserve request options and api results#122
dciccale wants to merge 2 commits into
masterfrom
fix/sdk-5295-request-options

Conversation

@dciccale

Copy link
Copy Markdown

What changed

  • return underlying client results from module-level API calls
  • pass configured proxy and timeout options to requests.Session.post
  • use time.monotonic() and remove the external monotonic dependency
  • add focused regression tests

Why

The module proxy discarded client return values. The request layer built proxy and timeout kwargs but did not pass them to the session.

Validation

  • 7 focused consumer timing and batching tests passed
  • 2 new return-value and request-option tests passed
  • git diff --check passed
  • the full suite requires data-plane environment credentials and was not used as the local gate

References

Use a valid Requests proxy mapping and mock transport calls. Fill batching queues before starting workers so the tests verify exact batch boundaries without thread scheduling races.
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.

1 participant