-
Notifications
You must be signed in to change notification settings - Fork 267
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
Filter out non-standard channels from channelInfo
API
#2107
Conversation
# Conflicts: # eclair-core/src/main/scala/fr/acinq/eclair/channel/Channel.scala
…e.scala Co-authored-by: Pierre-Marie Padiou <pm47@users.noreply.github.com>
# Conflicts: # eclair-core/src/test/scala/fr/acinq/eclair/integration/ChannelIntegrationSpec.scala # eclair-core/src/test/scala/fr/acinq/eclair/integration/PaymentIntegrationSpec.scala # eclair-core/src/test/scala/fr/acinq/eclair/payment/PaymentInitiatorSpec.scala # eclair-core/src/test/scala/fr/acinq/eclair/payment/PaymentLifecycleSpec.scala # eclair-node/src/main/scala/fr/acinq/eclair/api/handlers/Invoice.scala
# Conflicts: # eclair-core/src/test/scala/fr/acinq/eclair/payment/PaymentInitiatorSpec.scala # eclair-core/src/test/scala/fr/acinq/eclair/payment/PaymentLifecycleSpec.scala
Codecov Report
@@ Coverage Diff @@
## master #2107 +/- ##
==========================================
- Coverage 87.73% 87.63% -0.11%
==========================================
Files 159 165 +6
Lines 12455 12758 +303
Branches 522 535 +13
==========================================
+ Hits 10927 11180 +253
- Misses 1528 1578 +50
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This needs a test (and probably a squash/rebase with a clean commit message), otherwise LGTM.
Sigh, looks like I've done something wrong again with squashing, maybe we can do without it? |
Sure, no worries. It's really worth a bit of time to learn to master git, it's not that hard and really helps productivity, I can only highly recommend it! |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
A couple of nits
eclair-core/src/test/scala/fr/acinq/eclair/EclairImplSpec.scala
Outdated
Show resolved
Hide resolved
eclair-core/src/test/scala/fr/acinq/eclair/EclairImplSpec.scala
Outdated
Show resolved
Hide resolved
Co-authored-by: Bastien Teinturier <31281497+t-bast@users.noreply.github.com>
This change makes it possible for a non-standard channel to reply with
CommandFailure
which in turn will be filtered out inchannelsInfo
method.The reason is that currently hosted channels have to return something to make the whole API call succeed and whatever they return would break external tools which only expect standard channel formats.