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

update_cookies in client_reqrep.py has the same performance issue that was solved for the jar itself in #9204 #9520

Closed
bdraco opened this issue Oct 21, 2024 · 4 comments
Assignees
Labels
enhancement waiting-for-upstream We are waiting for an upstream change

Comments

@bdraco
Copy link
Member

bdraco commented Oct 21, 2024

The cookies get recreated every time. I need to dig into the cookie code some more to see if we can avoid this

if isinstance(value, Morsel):

#9204

Screenshot 2024-10-21 at 1 44 47 PM
@bdraco bdraco self-assigned this Oct 21, 2024
@bdraco
Copy link
Member Author

bdraco commented Oct 21, 2024

Screenshot 2024-10-17 at 9 55 31 PM

bdraco added a commit that referenced this issue Oct 24, 2024
This will get the ball rolling so we know
when we have solved #9520
@github-staff github-staff deleted a comment Oct 29, 2024
@bdraco
Copy link
Member Author

bdraco commented Nov 1, 2024

I think we have to wait for python/cpython#126157

@bdraco bdraco added the waiting-for-upstream We are waiting for an upstream change label Nov 1, 2024
@bdraco
Copy link
Member Author

bdraco commented Nov 1, 2024

If we don't find another solution by the time Python 3.14 is released, we can close this one

@bdraco
Copy link
Member Author

bdraco commented Nov 30, 2024

I think its unlikely we are going to be able to improve the performance without moving away from stdlib http.cookies. As the change to improve Morsel creation time is already merged upstream, I think there is nothing left to do but wait so I'm going to close this issue out.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement waiting-for-upstream We are waiting for an upstream change
Development

No branches or pull requests

2 participants