Skip to content

Fix QUIC issues#2164

Merged
EmelyanenkoK merged 1 commit intotestnetfrom
master
Feb 24, 2026
Merged

Fix QUIC issues#2164
EmelyanenkoK merged 1 commit intotestnetfrom
master

Conversation

@EmelyanenkoK
Copy link
Member

No description provided.

@EmelyanenkoK EmelyanenkoK merged commit bc14daa into testnet Feb 24, 2026
60 of 78 checks passed
@github-actions
Copy link

  1. quic/quic-sender.cpp:109-116DEFAULT_STREAM_SIZE_LIMIT is computed but never enforced when options_.max_size is unset because the guard still checks options_.max_size.has_value(). This leaves streams unbounded by default (potential memory/DoS risk). If the default is intended, drop the has_value() guard or set options_.max_size to the default at configuration time.

  2. validator/impl/liteserver-cache.hpp:81-88 — The duplicate sendMessage guard is now an LRU cache. Under high churn, entries can be evicted while the original message is still in flight, allowing duplicates to be accepted again. If duplicate suppression needs to hold until completion (or at least for the 60s window), consider “touching” on duplicates and/or using an explicit TTL/in-flight tracking rather than LRU eviction.

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