Add /txs/package endpoint to submit tx packages

[stevenroose]

I'm not sure if those stability tests etc are ran in CI. I'll let CI run and please let me know if manual testing is advised. It won't work with versions before 28.0-rc anyway.

Is there a common practice here for what to do when an api endpoint requires a certain daemon version? I guess now the node will throw an error and it will just be propagated upwards to the user.

[tnull]

I highly appreciate the effort to add submitpackage support to the Esplora protocol, as we Lightning devs want to lean on it being available soon. Given that there are multiple forks, should there be some coordination across projects to make sure the endpoints remain interoperable and usable by the same clients, e.g., rust-esplora-client?

See for example mempool/electrs#105 which seems to use another endpoint (txs/package) that would be incompatible to the one chosen here, IIUC?

[stevenroose]

Hey, I agree with your proposed endpoints in mempool/electrs#105 and will happily adapt mine to that :) I might even adopt your implementation :)

[wiz]

based on the comments by @stevenroose to adopt mempool/electrs#105 we will go ahead and merge that cc @moneyball

[tnull]

Hey, I agree with your proposed endpoints in mempool/electrs#105 and will happily adapt mine to that :) I might even adopt your implementation :)

Sounds like we're aligned on an API then. Thanks for resolving this! :)

[stevenroose]

Hi all, I (finally, my apologies) adapted this PR to resemble the same REST endpoint and response as what @wiz added for mempool.

@shesek is it ok if the internal mempool handling is done in a separate PR? It's seems to require a little bit of a non-trivial refactor and I don't really have the time. I would prefer that to not block having this endpoint added.

Also rebased to resolve some conflict.

[stevenroose]

@shesek any update on this??

[RobinLinus]

Any updates on this?

[tnull]

Any update on this?

[RCasatta]

ACK 9a4175d

added test in #159

[RCasatta]

merged via #159