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

patch rpc-bitcoin to be compatible with bitcoin core 28.0 #66

Merged
merged 1 commit into from
Oct 31, 2024

Conversation

remcoros
Copy link
Contributor

@remcoros remcoros commented Oct 2, 2024

rpc-bitcoin is not compatible with Bitcoin Core 28.0, see vansergen/rpc-bitcoin#65

This PR is a (temporary?) solution while there is no new upstream package version available.

This uses the patch-package npm package.
The postinstall script in package.json is automatically executed when npm i runs and this patches rpc-bitcoin.

@d6n13l0l1v3r
Copy link

@benjamin-wilson do you plant to implement it or merge this branch in order to have support to core 28

@owenashurst
Copy link

Just tested locally and seems to fix the issue.

@d6n13l0l1v3r
Copy link

d6n13l0l1v3r commented Oct 21, 2024

Just tested locally and seems to fix the issue.

my too I test locally with bitcoin 28 without any problem, including commit 2912dd5 because @remcoros don't sync yet

@remcoros
Copy link
Contributor Author

Just tested locally and seems to fix the issue.

my too I test locally with bitcoin 28 without any problem, including commit 2912dd5 because @remcoros don't sync yet

there are no merge conflicts, so you can merge this PR to master and be up to date.

@nmfretz
Copy link

nmfretz commented Oct 31, 2024

Thanks for this workaround @remcoros. I just went down a rabbithole after updating Core to v28 and tracked the issue down to the rpc-bitcoin library which seems to be a dead project at this point.

@benjamin-wilson do you have a timeline for testing+merging this and pushing an updated docker image? Public Pool is in the Umbrel app store and it would be great to update it with official docker images that maintain compatibility with Bitcoin Core v28. But if this is still a ways away I can just build an image to temporarily get Umbrel users back up and running until a new official image is released.

@benjamin-wilson benjamin-wilson merged commit f9e49a8 into benjamin-wilson:master Oct 31, 2024
@benjamin-wilson
Copy link
Owner

The docker image on umbrel is maintained by someone else. I've contacted them and asked them to update.
https://github.com/smolgrrr

@nmfretz
Copy link

nmfretz commented Oct 31, 2024

Wonderful. Thanks @benjamin-wilson, really appreciate it!

I know @smolgrrr well. He may be unavailable to update the public-pool-ui image, so I can handle that.
Edit: looks like his public-pool-ui image is up-to-date

Otherwise, he was using @sethforprivacy's public-pool image. Do you know if there are any major differences between your image and seth's?

@benjamin-wilson
Copy link
Owner

I didn't create or maintain the docker images, tbh I don't know

@nmfretz
Copy link

nmfretz commented Oct 31, 2024

Alrighty, thanks again @benjamin-wilson

@smolgrrr
Copy link

smolgrrr commented Nov 1, 2024

sorry @benjamin-wilson @nmfretz was looking into this the other week, but got caught up with work. ive updated the ui image, but it doesn't yet work with the umbrel app-store framework (i was using a proxy for the ui before, but need to change that).
will jump into it again shortly.

@benjamin-wilson
Copy link
Owner

sorry @benjamin-wilson @nmfretz was looking into this the other week, but got caught up with work. ive updated the ui image, but it doesn't yet work with the umbrel app-store framework (i was using a proxy for the ui before, but need to change that). will jump into it again shortly.

I think the umbrel image has already been patched and merged yesterday! Thanks for looking into this though.

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.

6 participants