-
Notifications
You must be signed in to change notification settings - Fork 64
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
patch rpc-bitcoin to be compatible with bitcoin core 28.0 #66
Conversation
@benjamin-wilson do you plant to implement it or merge this branch in order to have support to core 28 |
Just tested locally and seems to fix the issue. |
Thanks for this workaround @remcoros. I just went down a rabbithole after updating Core to v28 and tracked the issue down to the @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. |
The docker image on umbrel is maintained by someone else. I've contacted them and asked them to update. |
Wonderful. Thanks @benjamin-wilson, really appreciate it! I know @smolgrrr well. He may be unavailable to update the Otherwise, he was using @sethforprivacy's |
I didn't create or maintain the docker images, tbh I don't know |
Alrighty, thanks again @benjamin-wilson |
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). |
I think the umbrel image has already been patched and merged yesterday! Thanks for looking into this though. |
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 whennpm i
runs and this patches rpc-bitcoin.