Closed junderw closed 3 years ago
Keen to see this one merged. Can't pay to recent lnd nodes without this
rebased on master
This LGTM from a conceptual standpoint.
However, we need to test this. If you could help by gathering some examples and decoding them so we can see if the decoded data makes sense, then we can create some tests from them using the decoded data and modifying them to trigger all the Errors that are in the logic.
Still need to write tests.
@alexbosworth #30
The BOLT11 description is kind of vague. So the conditions for throwing errors etc. are unclear.