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

Output descriptor not included for address index #738

Closed
braydonf opened this issue Mar 26, 2019 · 1 comment · Fixed by #758
Closed

Output descriptor not included for address index #738

braydonf opened this issue Mar 26, 2019 · 1 comment · Fixed by #758
Labels

Comments

@braydonf
Copy link
Contributor

braydonf commented Mar 26, 2019

Versions
v1.0.2 (2b2e53d)

Expected

For p2sh, p2wpkh, p2pkh, p2wsh output types to be indexed separately.

Actual

Only the address hash is used in the index, so there is the output type is not included between the various output types. This has the result that bech32 (segwit) and base58 (non-segwit) addresses that have the same hash share the same results.

@braydonf
Copy link
Contributor Author

braydonf commented Mar 26, 2019

I have a reproducible test case defined in the indexer branch at commit braydonf@4dd0a9c.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant