Skip to content

Conversation

@pquentin
Copy link
Member

@pquentin pquentin commented Sep 30, 2024

Closes #162

headers=request_headers,
timeout=aiohttp_timeout,
**kwargs,
**kwargs, # type: ignore[arg-type]
Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is not related to Python 3.13, but caused instead by aio-libs/aiohttp#8463. I.. just have no idea how to make mypy happy here, short of maybe adding a class?

exception=err,
)
raise err from None
raise err from e
Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is also unrelated, but it feels wrong to hide parts of exceptions. (And this was needed to debug chain certs code.)

@pquentin pquentin merged commit 6c27e2c into elastic:main Oct 2, 2024
9 checks passed
@pquentin pquentin deleted the python-3.13-support branch October 2, 2024 16:42
github-actions bot pushed a commit that referenced this pull request Oct 2, 2024
@pquentin pquentin self-assigned this Oct 8, 2024
github-actions bot pushed a commit that referenced this pull request Jan 6, 2025
github-actions bot pushed a commit that referenced this pull request Jan 6, 2025
pquentin added a commit that referenced this pull request Jan 6, 2025
(cherry picked from commit 6c27e2c)

Co-authored-by: Quentin Pradet <[email protected]>
pquentin added a commit that referenced this pull request Jan 6, 2025
(cherry picked from commit 6c27e2c)

Co-authored-by: Quentin Pradet <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Certificate chain APIs may change in Python 3.13

2 participants