Convert ln to TS (6th commit of PR435)

[Mersho]

No description provided.

[grunch]

After testing I'm seeing a lot of messages like this one

[2024-07-11T11:09:01.224-03:00] error: All promises were rejected AggregateError: All promises were rejected

Any ideas what could generate them?

[Mersho]

Any ideas what could generate them?

Hey @grunch , sorry for the delay. I couldn't reproduce the error, but I did some digging around and I think this error is related to:

Promise.any()

Here are my references and what I found out:

• https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Errors/Promise_any_all_rejected

• https://bittukumar-web.medium.com/promise-and-its-methods-all-race-any-allsettled-0dc677b5aee1

If all the promises are rejected, promise will be rejected with the error “AggregateError: All promises were rejected”

• https://gist.github.com/Taran7885/574bb899ba515438fd6ef456c233a513

This is the only place Promise.any() is called:

https://github.com/lnp2pBot/bot/blob/main/bot/modules/nostr/index.js#L27

[knocte]

@grunch can you test with nostr disabled? this way we would confirm that this problem comes from that part of the code instead of from this PR

[grunch]

@grunch can you test with nostr disabled? this way we would confirm that this problem comes from that part of the code instead of from this PR

That's kinda weird, I can't reproduce the log error again, I'll be testing more today and if it didn't happen again I'll merge this PR

[knocte]

How did the testing go?