Skip to content
This repository has been archived by the owner on Dec 16, 2021. It is now read-only.

feat: expose rawchainlocksig and zmqpubrawtxlocksig from Core #221

Merged
merged 3 commits into from
Dec 25, 2020

Conversation

shumkov
Copy link
Member

@shumkov shumkov commented Dec 24, 2020

Issue being fixed or feature implemented

Drive requires rawchainlocksig zmq socket instead of rawchainlock. DAPI requires zmqpubrawtxlocksig

What was done?

Expose rawchainlocksig and zmqpubrawtxlocksig in dashd config instead of rawchainlock

How Has This Been Tested?

Run setup for local development command

Breaking Changes

None

Checklist:

  • I have performed a self-review of my own code
  • I have commented my code, particularly in hard-to-understand areas
  • I have added or updated relevant unit/integration/functional/e2e tests
  • I have made corresponding changes to the documentation

For repository code-owners and collaborators only

  • I have assigned this pull request to a milestone

@shumkov shumkov added this to the v0.17 milestone Dec 24, 2020
zmqpubrawchainlocksig=tcp://0.0.0.0:29998

{{? it.network === 'testnet'}}testnet=1
[test]{{?}}
{{? it.network === 'local'}}
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Use elif syntax like {{?? it.network === 'local'}}

Copy link
Member Author

@shumkov shumkov Dec 25, 2020

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Good point!

jawid-h
jawid-h previously approved these changes Dec 25, 2020
shuplenkov
shuplenkov previously approved these changes Dec 25, 2020
@shumkov shumkov dismissed stale reviews from shuplenkov and jawid-h via f91cdb4 December 25, 2020 08:14
@shumkov shumkov changed the title feat: expose rawchainlocksig from Core feat: expose rawchainlocksig and zmqpubrawtxlocksig from Core Dec 25, 2020
@shumkov shumkov merged commit af100c8 into v0.17-dev Dec 25, 2020
@shumkov shumkov deleted the update-zmq-sockets branch December 25, 2020 08:56
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Development

Successfully merging this pull request may close these issues.

4 participants