fix: Check ledger range in every handler

[PeterChen13579]

fixes #1565

[codecov]

Codecov Report

Attention: Patch coverage is 88.23529% with 4 lines in your changes missing coverage. Please review.

Project coverage is 71.69%. Comparing base (63c80f2) to head (e05c5bc).
Report is 5 commits behind head on develop.

Files with missing lines	Patch %	Lines
src/rpc/common/impl/HandlerProvider.cpp	0.00%	1 Missing ⚠️
src/rpc/handlers/ServerInfo.hpp	0.00%	0 Missing and 1 partial ⚠️
src/rpc/handlers/Subscribe.cpp	50.00%	0 Missing and 1 partial ⚠️
src/rpc/handlers/Tx.hpp	0.00%	0 Missing and 1 partial ⚠️

Additional details and impacted files

@@             Coverage Diff             @@
##           develop    #1755      +/-   ##
===========================================
+ Coverage    71.65%   71.69%   +0.03%     
===========================================
  Files          295      295              
  Lines        12649    12679      +30     
  Branches      6471     6500      +29     
===========================================
+ Hits          9064     9090      +26     
  Misses        1872     1872              
- Partials      1713     1717       +4     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[cindyyan317]

It's not every handler. I did not see subscribe unsubscribe, they are immune?