Skip to content

fix(python-sdk): REST client should not close after stream request#491

Merged
evansims merged 1 commit intomainfrom
fix/python/rest-client-should-not-close-after-stream
Feb 14, 2025
Merged

fix(python-sdk): REST client should not close after stream request#491
evansims merged 1 commit intomainfrom
fix/python/rest-client-should-not-close-after-stream

Conversation

@evansims
Copy link
Copy Markdown
Contributor

@evansims evansims commented Feb 14, 2025

Description

This is a minor bug fix to the new stream method added to the Python SDK's REST client in 0.9.1: the pool shouldn't close after the request concludes, as the same pool can continue to be used for future requests.

References

Generates openfga/python-sdk#172

Review Checklist

  • I have clicked on "allow edits by maintainers".
  • I have added documentation for new/changed functionality in this PR or in a PR to openfga.dev [Provide a link to any relevant PRs in the references section above]
  • The correct base branch is being used, if not main
  • I have added tests to validate that the change in functionality is working as expected

@evansims evansims added bug Something isn't working python-sdk Affects the Python SDK labels Feb 14, 2025
@evansims evansims force-pushed the fix/python/rest-client-should-not-close-after-stream branch from 9d75be2 to b8365f2 Compare February 14, 2025 19:13
@evansims evansims force-pushed the fix/python/rest-client-should-not-close-after-stream branch from b8365f2 to cc3bb6e Compare February 14, 2025 19:17
@evansims evansims marked this pull request as ready for review February 14, 2025 19:17
@evansims evansims requested a review from a team as a code owner February 14, 2025 19:17
@evansims evansims enabled auto-merge February 14, 2025 19:19
@evansims evansims added this pull request to the merge queue Feb 14, 2025
Merged via the queue into main with commit fe6c28c Feb 14, 2025
14 checks passed
@evansims evansims deleted the fix/python/rest-client-should-not-close-after-stream branch February 14, 2025 20:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working python-sdk Affects the Python SDK

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants