-
Notifications
You must be signed in to change notification settings - Fork 4
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.
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
feat: storing multiple ip
, host
and port
entries per NodeId
#482
Comments
This will be needed for MatrixAI/js-mdns#1. |
From #485:
|
This is related to #537. |
This is done. It's already available in the |
Specification
With the realisation that a
NodeId
->address
is a many to many relationship. We need to expand theNodeGraph
functionality to enable the ability to track multiple addresses per node. This can be acheved by expanding the level structure frombucket/NodeId
->nodeAddress
tobucket/nodeId/[ip4|ip6|host]/port
->null
. This should allow us to track multiplenodeId
->Ip4|ip6|host:port
mappings in a set like fashion.The utility of this needs to be reviewed. A nodeId refers to a single running node. There should only be one valid
IP:PORT
address for that node at any given time. So we only really need to track the latest address information for any node. A host could resolve to multip;e 'A' or 'AAAA' records and we'd have to check each one but we'd only store the hostname for that.Additional context
testnet.polykey.com
Polykey-CLI#71Tasks
NodeGraph
level structure to bebucket/nodeId/host|ip/port -> null
to enable tracking of multiple addresses.The text was updated successfully, but these errors were encountered: