Skip to content
This repository has been archived by the owner on Jan 29, 2023. It is now read-only.

Add ElectrumX Client #905

Draft
wants to merge 1 commit into
base: bixin_dev
Choose a base branch
from
Draft

Conversation

huazhouwang
Copy link

What does this implement/fix? Explain your changes.

支持 ElectrumX
目前发现 ElectrumX 提供的接口有部分无法满足我们的需求:

  1. 交易无法获取区块高度(electrum 原来实现先调用 get_history 获取交易ID及其区块高度列表,然后再调用 transaction.get 获取交易)
  2. 搜索交易无法分页,数据量过大(electrum 原来实现是先调用 get_history 获取所有的交易 ID 缓存后再分别获取交易信息保存进 DB)。即使搜索不考虑分页的话,因为每笔交易还要分别获取它输入的前置交易,虽然有 batch call 但请求量还是很大的
  3. 无法获取 UTXO 是否来自 coinbase,所以无法知道 coinbase 是否已经成熟了(有 100 个确认),但这个影响不大

Does this close any currently open issues?

If it fixes a bug or resolves a feature request, be sure to link to that issue.

Pull request type

Put an x in the boxes that apply

  • Bugfix
  • Feature
  • Code style update (formatting, renaming)
  • Refactoring (no functional changes, no api changes)
  • Build related changes
  • Documentation content changes
  • Other (please describe):

Where has this been tested?

Any other comments?

@huazhouwang
Copy link
Author

这个 PR 前置依赖 #894
目前看 ElectrumX 不适合获取交易历史
@taimanhui @shuaiLiWang

@taimanhui
Copy link

@huazhouwang 不要考虑交易历史,i.e. 不要和区块浏览器去卷,卷不过的

@huazhouwang huazhouwang changed the title [WIP] Add ElectrumX Client Add ElectrumX Client Sep 17, 2021
@huazhouwang
Copy link
Author

这个 PR 也可以看了

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

Successfully merging this pull request may close these issues.

2 participants