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.

Sign up for GitHub

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

Jump to bottom

Include p2pk into addressindex #1767

Closed
UdjinM6 opened this issue Dec 8, 2017 · 2 comments
Closed

Include p2pk into addressindex #1767

UdjinM6 opened this issue Dec 8, 2017 · 2 comments
Labels
enhancement

Comments

@UdjinM6
Copy link

UdjinM6 commented Dec 8, 2017 •
edited
Loading

See bitpay#17

Example:
https://insight.dash.org/insight/tx/f770f05d2b1c63b71b2650227252da06ef226661982c4ee9b136b64f77bbbd0c
vs
https://insight.dash.org/insight/address/XoLTipv6ryWECYu94vbkmDjntAXqNgouTW
@wozniakk wozniakk mentioned this issue Jan 8, 2018
Merged
UdjinM6 pushed a commit that referenced this issue Jan 9, 2018
@wozniakk @UdjinM6
Include p2pk into addressindex (#1839)
1cffb8a 
* Add simple test for p2pk script

refs #1767

* Add p2pk transaction addresses indexing

Addresses from p2pk transaction are now indexed by transations pubkeys
hashes. Even if it isn't strict behavior and the p2pk are rare
and obsolete, it is transaction that should be able to be looked up.

fixes #1767

* Add p2pk script checking tests

refs #1767
@nmarley
Copy link

nmarley commented Mar 25, 2018

I believe this has been added to the develop branch now. Ok to close this @UdjinM6 @codablock ?

@UdjinM6
Copy link
Author

UdjinM6 commented Mar 25, 2018

Yep, #1839 fixed this I believe. Closing.

@UdjinM6 UdjinM6 closed this as completed Mar 25, 2018
andvgal pushed a commit to energicryptocurrency/gen2-energi that referenced this issue Jan 6, 2019
@wozniakk @andvgal
Include p2pk into addressindex (dashpay#1839)
03067ba 
* Add simple test for p2pk script

refs dashpay#1767

* Add p2pk transaction addresses indexing

Addresses from p2pk transaction are now indexed by transations pubkeys
hashes. Even if it isn't strict behavior and the p2pk are rare
and obsolete, it is transaction that should be able to be looked up.

fixes dashpay#1767

* Add p2pk script checking tests

refs dashpay#1767
CryptoCentric pushed a commit to absolute-community/absolute that referenced this issue Feb 26, 2019
@wozniakk @CryptoCentric
Include p2pk into addressindex (dashpay#1839)
b8c2e88 
* Add simple test for p2pk script

refs dashpay#1767

* Add p2pk transaction addresses indexing

Addresses from p2pk transaction are now indexed by transations pubkeys
hashes. Even if it isn't strict behavior and the p2pk are rare
and obsolete, it is transaction that should be able to be looked up.

fixes dashpay#1767

* Add p2pk script checking tests

refs dashpay#1767
@frndxyz frndxyz mentioned this issue Nov 26, 2019
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
enhancement
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@schinzelh @nmarley @UdjinM6