Comments (5)
Thanks, @issmall. You're absolutely right; the comparison method currently used is incorrect and needs correction.
That said, unless the Buffers were duplicated, their references should have remained consistent. Could you elaborate on how you came across this error?
I plan to review the PR shortly. At an initial look, there could be issues related to it passing the linter tests. Also, it's noteworthy that the witnesses
type is undefined | Buffer
, which isn't suitable as arguments for Buffer.compare
.
Did the provided fix resolve your specific problem? While your identification of this issue is indeed valid and needs fixing, I want to ensure that we're also addressing any underlying problem you might be facing.
from descriptors.
Hi @issmall , can take this a look?
from descriptors.
It works.
I'm currently working on developing a self-custody platform based on this project. A few months ago, when I was researching Miniscript, I discovered this project and tried to write a demo. At that time, I could successfully create a wallet, sign transactions, and send them to the blockchain. However, last week, while testing, I encountered persistent errors. Initially, I thought it was a problem with my own calls, but later, I traced the issue to this specific location.
This project is fantastic!
from descriptors.
It works.
Do you mean specifically the PR I just submitted? #23
This project is fantastic!
Thank you for your kind words, @issmall. It means the world to me!
from descriptors.
Yes, I tested the fix in my project, it works well
from descriptors.
Related Issues (20)
- Remove bitcoinCoreFixtures.js from git; it's generated automatically HOT 1
- support multiple wildcards, and pass an array of indices
- Take into account Resource Limitations HOT 1
- Run createcreateFixtures.js before tests are run
- Handle BIP 32 path element overflow
- Return key origin HOT 2
- add compatibility with Ledger HOT 1
- Add new methods HOT 6
- Refactor DescriptorsLibrary HOT 1
- WIF or xprv keyExpressions should automatically sign the Psbt
- `updatePsbt` throws an Error when attempting to add another UTXO belonging to the same timelocked descriptor/address HOT 4
- `sortedmulti` nestedn in `wsh`? HOT 1
- Forced to invoke the construtor and instantiate a new object every time I want to increment an address index HOT 1
- Lookbehind in regexp not supported on iOS HOT 4
- Support for `@scure/btc-signer` HOT 1
- Output descriptor with both receiving and change paths HOT 4
- Master key prefix m is added to non-master xpub's path HOT 5
- TS types are not available HOT 4
- new Output with descriptor: addr(TAPROOT) HOT 6
Recommend Projects
-
React
A declarative, efficient, and flexible JavaScript library for building user interfaces.
-
Vue.js
🖖 Vue.js is a progressive, incrementally-adoptable JavaScript framework for building UI on the web.
-
Typescript
TypeScript is a superset of JavaScript that compiles to clean JavaScript output.
-
TensorFlow
An Open Source Machine Learning Framework for Everyone
-
Django
The Web framework for perfectionists with deadlines.
-
Laravel
A PHP framework for web artisans
-
D3
Bring data to life with SVG, Canvas and HTML. 📊📈🎉
-
Recommend Topics
-
javascript
JavaScript (JS) is a lightweight interpreted programming language with first-class functions.
-
web
Some thing interesting about web. New door for the world.
-
server
A server is a program made to process requests and deliver data to clients.
-
Machine learning
Machine learning is a way of modeling and interpreting data that allows a piece of software to respond intelligently.
-
Visualization
Some thing interesting about visualization, use data art
-
Game
Some thing interesting about game, make everyone happy.
Recommend Org
-
Facebook
We are working to build community through open source technology. NB: members must have two-factor auth.
-
Microsoft
Open source projects and samples from Microsoft.
-
Google
Google ❤️ Open Source for everyone.
-
Alibaba
Alibaba Open Source for everyone
-
D3
Data-Driven Documents codes.
-
Tencent
China tencent open source team.
from descriptors.