Skip to content

6996 Fix Unable to generate P2TR address via gRPC#7146

Closed
rox1338 wants to merge 1 commit intoElementsProject:masterfrom
rox1338:6996-grpc-taproot
Closed

6996 Fix Unable to generate P2TR address via gRPC#7146
rox1338 wants to merge 1 commit intoElementsProject:masterfrom
rox1338:6996-grpc-taproot

Conversation

@rox1338
Copy link

@rox1338 rox1338 commented Mar 12, 2024

See #6996 for more info on the bug.

Disclaimer:
I did this change directly in github and didn't run any tests.
So see this as a suggestion only.
Someone with knowledge of the code needs to have a thorough look.

@rox1338 rox1338 requested a review from cdecker as a code owner March 12, 2024 10:43
@ErikDeSmedt
Copy link
Contributor

I think you're on the right track to diagnose this bug.

The issue is a little bit deeper though. The cln-rpc/src/model.rs-file is auto-generated by msggen.
To fix the bug we need to ensure msggen generates the correct code.

Otherwise, next time msggen is run it will override your changes.

@rox1338
Copy link
Author

rox1338 commented Mar 25, 2024

@ErikDeSmedt You are right.
Would appreciate if someone who knows what he is doing fixes this.

@s373nZ
Copy link
Contributor

s373nZ commented Oct 15, 2024

@rox1338 It looks like this was addressed in PR #7217 by @daywalker90. Have you managed to re-test against a more recent release? Can this PR could be closed?

@rox1338
Copy link
Author

rox1338 commented Oct 16, 2024

Yep, works now!
Thank you!

@rox1338 rox1338 closed this Oct 16, 2024
@rox1338 rox1338 deleted the 6996-grpc-taproot branch October 16, 2024 08:52
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.

3 participants