refactor: acquireAsync will dispatch a job, not be dispatched

[ximinez]

High Level Overview of Change

Does a few different things:

1. Adds a new RAII class CanProcess to use where needed for job deduplication.
2. Adds and improves logging across a variety of classes. Uses the new to_short_string(base_uint) function to help disambiguate some messages about peers and ledgers.
3. Exits InboundLedger::onTimer without sending new requests if the complete ledger is available. (Duh.)
4. Changes the interface and usage of InboundLedgers::acquireAsync so that it dispatches a job, and is not called from a dispatched job. This will reduce the number of jobs spun up only to immediately exit.

Context of Change

This is the safe subset of changes from #5301, which has problems, plus the item to dispatch from acquireAsync.

Type of Change

• Refactor (non-breaking change that only restructures code)
• Performance (increase or change in throughput and/or latency)
• Tests (you added tests for code that already exists, or your new feature included in this PR)

API Impact

None

[codecov]

Codecov Report

❌ Patch coverage is 58.58586% with 41 lines in your changes missing coverage. Please review.
✅ Project coverage is 79.5%. Comparing base (92281a4) to head (c77a048).

Files with missing lines	Patch %	Lines
src/xrpld/app/misc/NetworkOPs.cpp	21.4%	22 Missing ⚠️
src/xrpld/app/ledger/detail/InboundLedgers.cpp	54.2%	11 Missing ⚠️
src/xrpld/app/ledger/detail/InboundLedger.cpp	25.0%	3 Missing ⚠️
src/xrpld/app/ledger/detail/LedgerMaster.cpp	0.0%	3 Missing ⚠️
src/xrpld/app/consensus/RCLConsensus.cpp	0.0%	2 Missing ⚠️

Additional details and impacted files

@@           Coverage Diff           @@
##           develop   #5327   +/-   ##
=======================================
  Coverage     79.5%   79.5%           
=======================================
  Files          816     817    +1     
  Lines        72180   72209   +29     
  Branches      8295    8273   -22     
=======================================
+ Hits         57364   57394   +30     
+ Misses       14816   14815    -1     

Files with missing lines	Coverage Δ
include/xrpl/basics/CanProcess.h	100.0% <100.0%> (ø)
include/xrpl/protocol/LedgerHeader.h	100.0% <ø> (ø)
src/xrpld/app/consensus/RCLValidations.cpp	72.9% <100.0%> (-1.6%)	⬇️
src/xrpld/app/ledger/InboundLedgers.h	100.0% <ø> (ø)
src/xrpld/app/ledger/detail/TimeoutCounter.cpp	88.4% <100.0%> (-0.3%)	⬇️
src/xrpld/app/ledger/detail/TimeoutCounter.h	100.0% <ø> (ø)
src/xrpld/app/misc/HashRouter.h	100.0% <ø> (ø)
src/xrpld/app/misc/NetworkOPs.h	100.0% <ø> (ø)
src/xrpld/app/consensus/RCLConsensus.cpp	62.1% <0.0%> (+0.7%)	⬆️
src/xrpld/app/ledger/detail/InboundLedger.cpp	36.7% <25.0%> (-0.2%)	⬇️
... and 3 more

... and 2 files with indirect coverage changes

🚀 New features to boost your workflow:

• ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.