Skip to content
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

misc: use common topics #389

Open
dariusc93 opened this issue Dec 13, 2023 · 0 comments
Open

misc: use common topics #389

dariusc93 opened this issue Dec 13, 2023 · 0 comments
Assignees
Labels
improvement Improve on existing feature P-High High Priority

Comments

@dariusc93
Copy link
Contributor

In the current implementation, we maintain separate pubsub topics for peers to allow separating data out for specific peers through a "mailbox" of sorts. This would help prevent needless data from being propagated to other peers of the same topic due to specific requests and events being encrypted for a specific peer. Example would be if we send or receive a friend request, the payload itself is encrypted between those two peers. In raygun, conversation topics are shared among the members of the conversation so messages can easily propagate or fanout to other peers in that mesh while still being intended for those set of peers in that topic. However, due to the separate topics as of #87, we would be required to be connected to those peers to be able to publish to those topics unless we are connected to a peer who is subscribed to that topic (in which in our current implementation, that would not be possible although #336 would allow that to be possible). As a result, data may not propagate and fanout.

In 7754827, a placeholder was added as a start of allowing the identity to be announced to a targeted topic when after adding a friend, or initializing the identity profile with a set of friends who may be online to build a discovery path (which is disabled by default). Specific peers topics (under the current implementation) will continue to publish encrypted messages while targeted topics may only publish signed data if the published message does not contain any sensitive information, otherwise it would be encrypted and marked for the intended recipients of the data.

Notes:

  • Although discovery is disabled by default, the placeholder functionality would improve identity discovery through a "friend-of-friend" style of trust as it only broadcast the identity out after accepting a request or after the initialization of the identity profile with existing friends (unless we are connected to a peer under the same topic).
  • This change should not impact the current functionality as it does intend to maintain a level of security and privacy for specific data that should not be broadcasted through a topic.
    • This might change in the far future were we would send data directly instead to reduce the redundancy for specific messages while connected to those peers.
@dariusc93 dariusc93 added improvement Improve on existing feature P-Low Low Priority labels Dec 13, 2023
@dariusc93 dariusc93 self-assigned this Dec 13, 2023
@dariusc93 dariusc93 added P-High High Priority and removed P-Low Low Priority labels Jun 10, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
improvement Improve on existing feature P-High High Priority
Projects
None yet
Development

No branches or pull requests

1 participant