Skip to content
This repository has been archived by the owner on Aug 23, 2020. It is now read-only.

ZMQ documentation is not up to date... #1597

Closed
GalRogozinski opened this issue Sep 4, 2019 · 3 comments · Fixed by #1665
Closed

ZMQ documentation is not up to date... #1597

GalRogozinski opened this issue Sep 4, 2019 · 3 comments · Fixed by #1665
Assignees
@GalRogozinski
Labels
C-ZMQ E-Documentation Epic - Documentation of IRI T-Bug

Comments

@GalRogozinski
Copy link
Contributor

GalRogozinski commented Sep 4, 2019 •
edited
Loading

Description

While closing #1072 we accidentally stopped publishing the following ZMQ topics:

  • rts information about walk terminations
  • dnscv neighbor DNS validations
  • dnscc neighbor DNS confirmations
  • dnscu neighbor DNS updates
  • hmr for the hit to miss ratio
  • antn for added non-tethered neighbors ( testnet only )
  • rntn for refused non-tethered neighbors
  • rstat for information about the tips requester
  • rtl for transactions randomly removed from the request list

Requirements

We need to decide between:

  1. Restoring them
  2. Deleting from documentation because lack of complaints means that no one uses them
@GalRogozinski GalRogozinski added T-Bug C-ZMQ E-Documentation Epic - Documentation of IRI labels Sep 4, 2019
@muXxer
Copy link
Contributor

muXxer commented Nov 13, 2019

Better delete that. :)
We didn't implement them in HORNET.
I think they are not used.

@GalRogozinski
Copy link
Contributor Author

Will delete

@GalRogozinski
Copy link
Contributor Author

Need to make sure that https://docs.iota.org/docs/node-software/0.1/iri/references/zmq-events is up to date as well

@GalRogozinski GalRogozinski mentioned this issue Nov 18, 2019
Merged
1 task
@GalRogozinski GalRogozinski self-assigned this Nov 18, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees

@GalRogozinski GalRogozinski

Labels
C-ZMQ E-Documentation Epic - Documentation of IRI T-Bug
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Fix: Zmq documentation iotaledger/iri
2 participants
@GalRogozinski @muXxer