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.

Sign up for GitHub

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

Jump to bottom

Remove fingerprints from socket and track it in state #3640

Open

josevalim wants to merge 2 commits into main from jv-fingerprints-on-state

+186 −181

Member

josevalim commented Jan 17, 2025 •

edited

Loading

fingerprints is a private field that is not actually required to be tracked on the socket, so we moved it to the channel state.

josevalim added 2 commits

January 17, 2025 18:26


          Remove fingerprints from socket and track it in state

312fcbd


          No need to pass socket to traverse

dd4a2c2

josevalim mentioned this pull request

Request for feedback: Format rendering override public API #3631

Open

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet