Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix(common): fix the direct payment #44

Merged

Conversation

vincenzopalazzo
Copy link
Collaborator

Based on #43

But not fixing #41

Fixing the error output, an example is

```
➜  barq git:(macros/bug-fixing-pay) ✗ clightning --testnet -k barqpay bolt11_invoice=lntb10n1pndpz4usp5dyhj3xzxeq7t4av7h7vd6fe42tp88hlu06ea8aq3egucmf8f9gkqpp5mx5d03qql2e96xzmwu7kv0drwlaam8d0dc3fetk6cpg2nn5rkyesdq5w3jhxarfdenjqcnpwfcsxqyjw5qcqp2rzjqgjmhs48xsve8n2e94ascxzmhrrrt8xpm5fn096ud4qn2nj8qwlkg27skgqqqvqqqcqqqp5eqqqqqqsqqc9qxpqysgq3gx6wj2ngrdwdp6qvg47svm2p7tpnf553lrchttsrz2hmrmmkuchcz9ggpnluugytcczws0q05pd644ng2q4anv4xqnh7yrfhgdrpvcqxhk732
{
   "code": 204,
   "data": {
      "amount_msat": 1000,
      "amount_sent_msat": 1000,
      "created_at": 1724944169,
      "created_index": 1394,
      "destination": "030b686a163aa2bba03cebb8bab7778fac251536498141df0a436d688352d426f6",
      "erring_channel": "8581936x4942974x36632",
      "erring_direction": 0,
      "erring_index": 1,
      "erring_node": "030b686a163aa2bba03cebb8bab7778fac251536498141df0a436d688352d426f6",
      "failcode": 16394,
      "failcodename": "WIRE_UNKNOWN_NEXT_PEER",
      "groupid": 3,
      "id": 1394,
      "payment_hash": "d9a8d7c400fab25d185b773d663da377fbdd9daf6e229caedac050a9ce83b133",
      "raw_message": "400a",
      "status": "pending"
   },
   "message": "failed: WIRE_UNKNOWN_NEXT_PEER (reply from remote)"
}
```

Signed-off-by: Vincenzo Palazzo <[email protected]>
Signed-off-by: Vincenzo Palazzo <[email protected]>
Copy link
Collaborator

@raehat raehat left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Reviewed. PR looks fine to me

@vincenzopalazzo vincenzopalazzo merged commit 83df9c1 into coffee-tools:main Aug 29, 2024
@vincenzopalazzo vincenzopalazzo deleted the macros/bug-fixing-direct branch August 29, 2024 22:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants