Movatterモバイル変換


[0]ホーム

URL:


Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Sign up
Appearance settings

feat: Add support for asynchronousAuthorizedSession api#1577

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to ourterms of service andprivacy statement. We’ll occasionally send you account related emails.

Already on GitHub?Sign in to your account

Merged
ohmayr merged 13 commits intomainfromadd-support-for-async-authorized-session-api
Sep 16, 2024

Conversation

ohmayr
Copy link
Contributor

@ohmayrohmayr commentedAug 9, 2024
edited
Loading

This PR implements the asynchronousAuthorizedSession API to support async requests.

* chore: initial setup for async auth sessions api* fix whitespace* add init file* update file names to aiohttp* update import statement
* feat: implement async timeout guard* add docstring* clean whitespace* update import file name* add missing return statement* update test cases* update test cases* include underlying timeout exception in trace* avoid the cost of actual time
…1575)* feat: implement asynchronous response class for AuthorizedSessions API* check if aiohttp is installed and avoid tests dependency* update content to be async* update docstring to be specific to aiohttp* add type checking and avoid leaking underlying API responses* add test case for iterating chunks* add read method to response interface* address PR comments* fix lint issues
@ohmayrohmayr added the do not mergeIndicates a pull request not ready for merge, due to either quality or timing. labelAug 13, 2024
@ohmayrohmayr changed the titlefeat: Add Support for Asynchronous AuthorizedSession APIfeat: Add support for asynchronousAuthorizedSession apiAug 13, 2024
…1579)* feat: implement request class for asynchoronous AuthorizedSession API* add type checking and address TODOs* remove default values from interface methods* aiohttp reponse close method must not be awaited* cleanup* update Request class docstring
* feat: Implement Asynchronous AuthorizedSession class* add comment for implementing locks within refresh* move timeout guard to sessions* add unit tests and code cleanup* implement async exponential backoff iterator* cleanup* add testing for http methods and cleanup* update number of retries to 3* refactor test cases* fix linter and mypy issues* fix pytest code coverage
@ohmayrohmayr marked this pull request as ready for reviewAugust 15, 2024 17:26
@ohmayrohmayr requested review froma team ascode ownersAugust 15, 2024 17:26
arithmetic1728and others added2 commitsAugust 23, 2024 14:08
* chore: Add aiohttp requirements test constraint. (#1566)See#1565for more information.* chore(main): release 2.33.0 (#1560)* chore(main): release 2.33.0* fix: retry token request on retryable status code (#1563)* fix: retry token request on retryable status code* feat(auth): Update get_client_ssl_credentials to support X.509 workload certs (#1558)* feat(auth): Update get_client_ssl_credentials to support X.509 workload certs* feat(auth): Update has_default_client_cert_source* feat(auth): Fix formatting* feat(auth): Fix test__mtls_helper.py* feat(auth): Fix function name in tests* chore: Refresh system test creds.* feat(auth): Fix style* feat(auth): Fix casing* feat(auth): Fix linter issue* feat(auth): Fix coverage issue---------Co-authored-by: Carl Lundin <clundin@google.com>Co-authored-by: Carl Lundin <108372512+clundin25@users.noreply.github.com>* chore: Update ECP deps. (#1583)* chore(main): release 2.34.0 (#1574)* cleanup: minor code cleanup* fix lint issues---------Co-authored-by: Carl Lundin <108372512+clundin25@users.noreply.github.com>Co-authored-by: release-please[bot] <55107282+release-please[bot]@users.noreply.github.com>Co-authored-by: Tom Milligan <tom.milligan@uipath.com>Co-authored-by: Andy Zhao <andyzhao@google.com>Co-authored-by: Carl Lundin <clundin@google.com>
@ohmayrohmayr removed the do not mergeIndicates a pull request not ready for merge, due to either quality or timing. labelSep 16, 2024
@ohmayrohmayr merged commit2910b6b intomainSep 16, 2024
16 checks passed
@ohmayrohmayr deleted the add-support-for-async-authorized-session-api branchSeptember 16, 2024 21:44
Sign up for freeto join this conversation on GitHub. Already have an account?Sign in to comment
Reviewers

@arithmetic1728arithmetic1728arithmetic1728 approved these changes

@clundin25clundin25clundin25 approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

3 participants
@ohmayr@arithmetic1728@clundin25

[8]ページ先頭

©2009-2025 Movatter.jp