Consider implementing blockchain.address.get_first_use

[monsterbitar]

Consider implementing blockchain.address.get_first_use

[monsterbitar]

I commented on the MR with a preference for null instead of error, as I have what I consider valid usage that will not find data (recover from seed without knowing the derivation path, therefor trying multiple paths).

Closing this issue since the feature is implemented, but ideally some coordination with other implementations on the matter would be helpful. if they are willing to update they can, for example, make it so that after some specific version it adops the new behaviour. If not, then it's probably better to stick with the error version for now.

[cculianu]

Update: After discussions on TG it appears people are ok with returning null for unused addresses. I have updated Fulcrum to do just that in the latest commit pushed to master.

[cculianu]

Yes this is not difficult at all to implement since it re-uses existing database data easily enough with zero extra cost. I’ll try and get this in before the next release.

[zander]

this would be useful

[cculianu]

Ok I implemented this in PR #210 . Note that I am copying the API of the Electrs & Rostrum implementation. Of note:

• Electrs/Rostrum return a JSON-RPC error if the address and/or scripthash in question has 0 history (unused). Is this desirable? Can also just return a null result here (but then we would be breaking from Rostrum).
• It's currently impossible to implement the new 2nd optional argument token_filter here in Fulcrum. This is because Fulcrum doesn't store token history (only token utxos). So the API for this in Fulcrum is like the old pre-token Electrs API.

I am in particular curious about the first bullet-point above. Do we want to return null if the address has no history or do we want to throw an RPC error. I am a bit uncomfortable with throwing an error here because the other blockchain.address.* RPCs only throw on invalid arguments, never on missing history or missing utxos for a particular address.

[cculianu]

TBH for the reasons you cited -- it makes more sense to return null here. I will change it and notify in this issue when that's done.

[cculianu]

Actually. I just re-read what you wrote. It seems you want to co-ordinate with Rostrum (the only other impl.). Can you ask @dagurval if he is willing to change it?

If so then I will change it. He may listen to you. :)