Skip to content

Fix misc#188

Merged
anxolin merged 2 commits intobalance-ssefrom
fix-misc
Jan 16, 2026
Merged

Fix misc#188
anxolin merged 2 commits intobalance-ssefrom
fix-misc

Conversation

@anxolin
Copy link
Copy Markdown
Contributor

@anxolin anxolin commented Jan 16, 2026

Fixes/improve several minot issues

Main one, to handle better when clients disconnect.

You can test to abruptelly disconnect. The client will notice and stop tracking the user

I used this command to subscribe, then I kill -9 the pid:

curl -X GET "http://localhost:3001/1/accounts/0x87A04752E516548B0d5d4DF97384C0b22B649179/balances/sse?tokens=0x4d5f47fa6a74757f35c14fd3a6ef8e3c9bc514e8%2C0x6b175474e89094c44da98b954eedeac495271d0f%2C0xa0b86991c6218b36c1d19d4a2e9eb0ce3606eb48" -H "accept: text/event-stream"

See:

image

@coderabbitai
Copy link
Copy Markdown
Contributor

coderabbitai bot commented Jan 16, 2026

Important

Review skipped

Auto reviews are disabled on base/target branches other than the default branch.

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.


Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@socket-security
Copy link
Copy Markdown

Review the following changes in direct dependencies. Learn more about Socket for GitHub.

Diff Package Supply Chain
Security
Vulnerability Quality Maintenance License
Addedpg@​8.13.1991009893100

View full report

@anxolin anxolin merged commit 6998f92 into balance-sse Jan 16, 2026
7 checks passed
@anxolin anxolin deleted the fix-misc branch January 16, 2026 17:13
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.

1 participant