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

Adapt listpeers code to handle listpeerchannels instead #2

Draft
wants to merge 2 commits into
base: add-trace-log-level
Choose a base branch
from

Conversation

ksedgwic
Copy link
Owner

@ksedgwic ksedgwic commented Apr 29, 2024

Possibly addresses ZmnSCPxj#195

@ksedgwic ksedgwic marked this pull request as draft April 29, 2024 06:52
@ksedgwic ksedgwic changed the title Adapt ccccccfljbrvtdvhgvvnjectlhlugchurgukibecuve`h Adapt listpeers code to handle listpeerchannels instead Apr 29, 2024
@ksedgwic ksedgwic force-pushed the construct-listpeers branch from 8820414 to de01501 Compare April 29, 2024 17:37
A couple `listpeers` uses can remain because they don't need channel
information.  Others use ListpeersAnnouncer and are covered by the
next commit.
Construct a "compatibility struct" to convert `listpeerchannels`
output into legacy `listpeers` format.

Tests written using the legacy listpeers format can use the
`convert_legacy_listpeers` utility to construct a compatibility
struct.

The test_peerjudge_datagatherer malformed test needed to be malformed
differently to achieve the desired effect.
@ksedgwic ksedgwic force-pushed the construct-listpeers branch from de01501 to 1cfd860 Compare April 29, 2024 17:49
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.

1 participant