Refuse a message past the held-message bound with ESME_RTHROTTLED, and answer a full reassembly buffer the same way #29

Merged
lilleman merged 5 commits from held-throttle into main 2026-09-26 15:24:43 +02:00
Owner

A message arriving while the application holds 1000 unanswered or 64 MiB is refused with ESME_RTHROTTLED (ESME_RX_T_APPN on deliver_sm) instead of evicting the oldest. The reassembly buffer's full refusal moves from ESME_RMSGQFUL to the same pair.

A message arriving while the application holds 1000 unanswered or 64 MiB is refused with ESME_RTHROTTLED (ESME_RX_T_APPN on deliver_sm) instead of evicting the oldest. The reassembly buffer's full refusal moves from ESME_RMSGQFUL to the same pair.
lilleman added 1 commit 2026-09-26 13:18:56 +02:00
Refuse a message past the held-message bound with ESME_RTHROTTLED, and answer a full reassembly buffer the same way
Test / lint (pull_request) Successful in 23s
Test / test (18) (pull_request) Successful in 31s
Test / test (20) (pull_request) Successful in 31s
Test / test (22) (pull_request) Successful in 31s
Test / test (24) (pull_request) Successful in 31s
Test / test (26) (pull_request) Successful in 31s
Mirror / push (push) Successful in 4s
855ccbe75f
lilleman added 1 commit 2026-09-26 14:37:21 +02:00
Give the dumbclient peers a namespace owner that outlives them, count window 2000's throttled messages, and correct what the findings blamed on the peer
Mirror / push (push) Successful in 4s
Test / lint (pull_request) Successful in 23s
Test / test (18) (pull_request) Successful in 31s
Test / test (20) (pull_request) Successful in 31s
Test / test (22) (pull_request) Successful in 32s
Test / test (24) (pull_request) Successful in 30s
Test / test (26) (pull_request) Successful in 31s
fb6b125aab
lilleman added 1 commit 2026-09-26 14:41:10 +02:00
Warn once at the held-message bound, tell receivers sendResp() is what releases a message, and pin that a refused segment joins no group
Mirror / push (push) Has been cancelled
Test / test (18) (pull_request) Successful in 31s
Test / test (22) (pull_request) Successful in 31s
Test / test (26) (pull_request) Successful in 32s
Test / lint (pull_request) Successful in 23s
Test / test (20) (pull_request) Successful in 31s
Test / test (24) (pull_request) Successful in 31s
b9167073d4
lilleman added 1 commit 2026-09-26 14:44:01 +02:00
Stop refusing only once the held messages are down to half, so a full window warns once, and name the octets in the warning
Mirror / push (push) Successful in 5s
Test / test (22) (pull_request) Successful in 31s
Test / lint (pull_request) Successful in 22s
Test / test (18) (pull_request) Successful in 30s
Test / test (20) (pull_request) Successful in 30s
Test / test (24) (pull_request) Successful in 31s
Test / test (26) (pull_request) Successful in 31s
f2977638f2
lilleman added 1 commit 2026-09-26 14:53:10 +02:00
Name the retry status by direction, and correct what the held-message decision claims
Test / lint (pull_request) Successful in 28s
Test / test (18) (pull_request) Successful in 31s
Test / test (20) (pull_request) Successful in 30s
Test / test (22) (pull_request) Successful in 31s
Test / test (24) (pull_request) Successful in 30s
Test / test (26) (pull_request) Successful in 37s
Mirror / push (push) Successful in 4s
42b1239f3d
lilleman marked the pull request as ready for review 2026-09-26 15:24:42 +02:00
lilleman merged commit 42b1239f3d into main 2026-09-26 15:24:43 +02:00
lilleman deleted branch held-throttle 2026-09-26 15:24:43 +02:00
Sign in to join this conversation.
No Reviewers
No Label
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: larvit/smpp-js#29