Skip to content
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 our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix new linting and type issues #125

Merged
merged 1 commit into from
Dec 26, 2023
Merged

Fix new linting and type issues #125

merged 1 commit into from
Dec 26, 2023

Conversation

sethmlarson
Copy link
Owner

No description provided.

@davisagli davisagli merged commit 178fea2 into main Dec 26, 2023
16 checks passed
@davisagli davisagli deleted the fix-lint branch December 26, 2023 18:51
cmin764 added a commit to robocorp/truststore that referenced this pull request Jul 4, 2024
* Mark tests which require internet access

Co-authored-by: Seth Michael Larson <[email protected]>

* Add warnings for packages and libraries to avoid inject_into_ssl() (sethmlarson#122)

* Fix new linting and type issues (sethmlarson#125)

* Add explicit support for Python 3.13

* Make HTTP server per testcase (sethmlarson#130)

* Fix loading of additional certificates for macOS (sethmlarson#134)

* Raise a better exception when peer sends no certificates on Windows (sethmlarson#135)

Co-authored-by: David Glick <[email protected]>

* Release 0.9.0

* Test on Windows and macOS

* Release 0.9.1

* Fix the `aiohttp` examples to clean up resources

The previously present snippets presented a misuse of `aiohttp`'s APIs. While technically they work, they don't release the allocated resources like async CMs would. And so we recommend using the latter.

* Fix MRO with super class `verify_mode` calling

* Attempt `verify_mode` recursion fix

* Solve patching recursion via copy

* Test class override

* Test

* Switch super to direct object (avoid recursion)

* Fix MRO resolution without a super

* Do not re-inject on multiple calls

* Fix typing

---------

Co-authored-by: Stefano Rivera <[email protected]>
Co-authored-by: Seth Michael Larson <[email protected]>
Co-authored-by: Seth Michael Larson <[email protected]>
Co-authored-by: David Glick <[email protected]>
Co-authored-by: Sviatoslav Sydorenko (Святослав Сидоренко) <[email protected]>
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