Implement Hash for ConfirmationTarget

[benthecarman]

In the ldk sample ConfirmationTarget is remade so it can be used in a HashMap. This should allow for a simplification in the sample node and likely other projects.

[codecov-commenter]

Codecov Report

Base: 90.78% // Head: 90.76% // Decreases project coverage by -0.02% ⚠️

Coverage data is based on head (83dcd39) compared to base (e55e0d5).
Patch coverage: 0.00% of modified lines in pull request are covered.

Additional details and impacted files

@@            Coverage Diff             @@
##             main    #1831      +/-   ##
==========================================
- Coverage   90.78%   90.76%   -0.03%     
==========================================
  Files          87       87              
  Lines       47604    47604              
  Branches    47604    47604              
==========================================
- Hits        43218    43207      -11     
- Misses       4386     4397      +11     

Impacted Files	Coverage Δ
lightning/src/chain/chaininterface.rs	95.65% <0.00%> (ø)
lightning/src/chain/onchaintx.rs	94.57% <0.00%> (-0.44%)	⬇️
lightning/src/ln/functional_tests.rs	96.94% <0.00%> (-0.12%)	⬇️
lightning/src/ln/channelmanager.rs	85.38% <0.00%> (-0.03%)	⬇️

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

☔ View full report at Codecov.
📢 Do you have feedback about the report comment? Let us know in this issue.