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

Expose API for PublicAddresses #212

Merged
merged 33 commits into from
Sep 4, 2024
Merged
Show file tree
Hide file tree
Changes from 28 commits
Commits
Show all changes
33 commits
Select commit Hold shift + click to select a range
8502645
identify: Expose API to inject listen addresses
lexnv Aug 20, 2024
e452745
tests: Check identify public addresses are propagated
lexnv Aug 20, 2024
0e42027
listen-addr: Add listen address struct to a separate module
lexnv Aug 28, 2024
890df79
listen-addr/tests: Check listen addr interface functionality
lexnv Aug 28, 2024
59668c9
listen-addr: Use a lock guard for custom iteration over the addresses
lexnv Aug 28, 2024
56ed02a
litep2p: Store shared listenAddresses on Litep2p object
lexnv Aug 28, 2024
a4bbf06
identify: Use the new interface of public addresses
lexnv Aug 28, 2024
f2a1c51
listen-addr: Check listen address contains p2p protocol
lexnv Aug 28, 2024
67b617d
Polish up the API
lexnv Aug 28, 2024
38d8ca3
listen-addr: Register lsiten addresses with adding the local peerID
lexnv Aug 28, 2024
2a45422
Adjust testing to the new interface
lexnv Aug 28, 2024
fc0c700
Use ListenAddresses everywhere
lexnv Aug 28, 2024
7f07e9d
listen-addr: Add better documentation
lexnv Aug 28, 2024
794eba5
listen-addr: Add contains and remove partial methods
lexnv Aug 29, 2024
5e9930a
Merge remote-tracking branch 'origin/master' into lexnv/indentify-con…
lexnv Aug 29, 2024
e11a996
Rename to ExternalAddresses for clarity
lexnv Aug 30, 2024
09b6fee
Refactor and adjust testing
lexnv Sep 2, 2024
c11f468
Merge remote-tracking branch 'origin/master' into lexnv/indentify-con…
lexnv Sep 2, 2024
68c76d2
Apply fmt
lexnv Sep 2, 2024
6161233
Fix cargo doc and clippy
lexnv Sep 2, 2024
dacd665
pub-addr: Rename API methods
lexnv Sep 3, 2024
1194a29
Introduce ListenAddresses object
lexnv Sep 3, 2024
ba6e198
Use listenAddresses added interface
lexnv Sep 3, 2024
e7a7d80
identify: Use user-provided, listen and public addresses
lexnv Sep 3, 2024
9ad762b
manager: Remove transport_manager::register_listen_address
lexnv Sep 3, 2024
d0ebcd2
Adjust testing
lexnv Sep 3, 2024
41c1e23
Fix documentation
lexnv Sep 3, 2024
fd23d15
address: Introduce insertion error for better reporting
lexnv Sep 3, 2024
be8d663
Remove ListenAddresses
lexnv Sep 3, 2024
af8fe60
Adjust testing
lexnv Sep 3, 2024
39153f1
Adjust testing
lexnv Sep 3, 2024
6b7b7b1
identify: Remove public addr from list config
lexnv Sep 4, 2024
a633b68
Merge branch 'master' into lexnv/indentify-confirmed-addresses
lexnv Sep 4, 2024
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
2 changes: 1 addition & 1 deletion examples/custom_executor.rs
Original file line number Diff line number Diff line change
Expand Up @@ -122,7 +122,7 @@ async fn main() {

// dial `litep2p1`
litep2p2
.dial_address(litep2p1.listen_addresses().next().unwrap().clone())
.dial_address(litep2p1.listen_addresses().get_addresses().get(0).unwrap().clone())
.await
.unwrap();

Expand Down
2 changes: 1 addition & 1 deletion examples/custom_protocol.rs
Original file line number Diff line number Diff line change
Expand Up @@ -277,7 +277,7 @@ async fn main() {
let (mut litep2p2, mut handle2) = make_litep2p();

let peer2 = *litep2p2.local_peer_id();
let listen_address = litep2p2.listen_addresses().next().unwrap().clone();
let listen_address = litep2p2.listen_addresses().get_addresses().get(0).unwrap().clone();
litep2p1.add_known_address(peer2, std::iter::once(listen_address));

tokio::spawn(async move {
Expand Down
2 changes: 1 addition & 1 deletion examples/echo_notification.rs
Original file line number Diff line number Diff line change
Expand Up @@ -125,7 +125,7 @@ async fn main() {

// get the first (and only) listen address for the second peer
// and add it as a known address for `litep2p1`
let listen_address = litep2p2.listen_addresses().next().unwrap().clone();
let listen_address = litep2p2.listen_addresses().get_addresses().get(0).unwrap().clone();
let peer = *litep2p2.local_peer_id();

litep2p1.add_known_address(peer, vec![listen_address].into_iter());
Expand Down
2 changes: 1 addition & 1 deletion examples/gossiping.rs
Original file line number Diff line number Diff line change
Expand Up @@ -237,7 +237,7 @@ async fn main() {

// establish connection to litep2p for all other litep2ps
let peer2 = *litep2p2.local_peer_id();
let listen_address = litep2p2.listen_addresses().next().unwrap().clone();
let listen_address = litep2p2.listen_addresses().get_addresses().get(0).unwrap().clone();

litep2p1.add_known_address(peer2, vec![listen_address.clone()].into_iter());
litep2p3.add_known_address(peer2, vec![listen_address.clone()].into_iter());
Expand Down
198 changes: 198 additions & 0 deletions src/addresses.rs
Original file line number Diff line number Diff line change
@@ -0,0 +1,198 @@
// Copyright 2024 litep2p developers
//
// Permission is hereby granted, free of charge, to any person obtaining a
// copy of this software and associated documentation files (the "Software"),
// to deal in the Software without restriction, including without limitation
// the rights to use, copy, modify, merge, publish, distribute, sublicense,
// and/or sell copies of the Software, and to permit persons to whom the
// Software is furnished to do so, subject to the following conditions:
//
// The above copyright notice and this permission notice shall be included in
// all copies or substantial portions of the Software.
//
// THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND, EXPRESS
// OR IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF MERCHANTABILITY,
// FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT. IN NO EVENT SHALL THE
// AUTHORS OR COPYRIGHT HOLDERS BE LIABLE FOR ANY CLAIM, DAMAGES OR OTHER
// LIABILITY, WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING
// FROM, OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER
// DEALINGS IN THE SOFTWARE.

use std::{collections::HashSet, sync::Arc};

use multiaddr::{Multiaddr, Protocol};
use parking_lot::RwLock;

use crate::PeerId;

/// Set of the public addresses of the local node.
///
/// The format of the addresses stored in the set contain the local peer ID.
/// This requirement is enforced by the [`PublicAddresses::add_address`] method,
/// that will add the local peer ID to the address if it is missing.
///
/// # Note
///
/// - The addresses are reported to the identify protocol and are used by other nodes
/// to establish a connection with the local node.
///
/// - Users must ensure that the addresses are reachable from the network.
#[derive(Debug, Clone)]
pub struct PublicAddresses {
pub(crate) inner: Arc<RwLock<HashSet<Multiaddr>>>,
local_peer_id: PeerId,
}

impl PublicAddresses {
/// Creates new [`PublicAddresses`] from the given peer ID.
pub(crate) fn new(local_peer_id: PeerId) -> Self {
Self {
inner: Arc::new(RwLock::new(HashSet::new())),
local_peer_id,
}
}

/// Add a public address to the list of addresses.
///
/// The address must contain the local peer ID, otherwise an error is returned.
/// In case the address does not contain any peer ID, it will be added.
///
/// Returns true if the address was added, false if it was already present.
pub fn add_address(&self, address: Multiaddr) -> Result<bool, InsertionError> {
let address = ensure_local_peer(address, self.local_peer_id)?;
Ok(self.inner.write().insert(address))
}

/// Remove the exact public address.
///
/// The provided address must contain the local peer ID.
pub fn remove_address(&self, address: &Multiaddr) -> bool {
self.inner.write().remove(address)
}

/// Returns a vector of the available listen addresses.
pub fn get_addresses(&self) -> Vec<Multiaddr> {
self.inner.read().iter().cloned().collect()
}
}

/// Set of the addresses the local node listens on.
///
/// The format of the addresses stored in the set contain the local peer ID.
/// This requirement is enforced by the [`ListenAddresses::add_address`] method.
///
/// The listen addresses are populated during the construction of the Litep2p object.
#[derive(Debug, Clone)]
pub struct ListenAddresses {
dmitry-markin marked this conversation as resolved.
Show resolved Hide resolved
pub(crate) inner: Arc<RwLock<HashSet<Multiaddr>>>,
local_peer_id: PeerId,
}

impl ListenAddresses {
/// Creates new [`ListenAddresses`] from the given peer ID.
pub(crate) fn new(local_peer_id: PeerId) -> Self {
Self {
inner: Arc::new(RwLock::new(HashSet::new())),
local_peer_id,
}
}
/// Add a listen address to the list of addresses.
///
/// Returns true if the address was added, false if it was already present.
pub fn add_address(&self, address: Multiaddr) -> Result<bool, InsertionError> {
let address = ensure_local_peer(address, self.local_peer_id)?;
Ok(self.inner.write().insert(address))
}

/// Remove the listen address.
pub fn remove_address(&self, address: &Multiaddr) -> bool {
self.inner.write().remove(address)
}

/// Returns a vector of the available listen addresses.
pub fn get_addresses(&self) -> Vec<Multiaddr> {
self.inner.read().iter().cloned().collect()
}
}

/// Check if the address contains the local peer ID.
///
/// If the address does not contain any peer ID, it will be added.
fn ensure_local_peer(
mut address: Multiaddr,
local_peer_id: PeerId,
) -> Result<Multiaddr, InsertionError> {
if address.is_empty() {
return Err(InsertionError::EmptyAddress);
}

// Verify the peer ID from the address corresponds to the local peer ID.
if let Some(peer_id) = PeerId::try_from_multiaddr(&address) {
if peer_id != local_peer_id {
return Err(InsertionError::DifferentPeerId);
}
} else {
address.push(Protocol::P2p(local_peer_id.into()));
}

Ok(address)
}

/// The error returned when an address cannot be inserted.
#[derive(Debug, Clone, Copy, PartialEq, Eq)]
pub enum InsertionError {
/// The address is empty.
EmptyAddress,
/// The address contains a different peer ID than the local peer ID.
DifferentPeerId,
}

#[cfg(test)]
mod tests {
use super::*;
use std::str::FromStr;

#[test]
fn add_remove_contains() {
let peer_id = PeerId::random();
let addresses = PublicAddresses::new(peer_id);
let address = Multiaddr::from_str("/dns/domain1.com/tcp/30333").unwrap();
let peer_address = Multiaddr::from_str("/dns/domain1.com/tcp/30333")
.unwrap()
.with(Protocol::P2p(peer_id.into()));

assert!(!addresses.get_addresses().contains(&address));

assert!(addresses.add_address(address.clone()).unwrap());
// Adding the address a second time returns Ok(false).
assert!(!addresses.add_address(address.clone()).unwrap());

assert!(!addresses.get_addresses().contains(&address));
assert!(addresses.get_addresses().contains(&peer_address));

addresses.remove_address(&peer_address);
assert!(!addresses.get_addresses().contains(&peer_address));
}

#[test]
fn get_addresses() {
let peer_id = PeerId::random();
let addresses = PublicAddresses::new(peer_id);
let address1 = Multiaddr::from_str("/dns/domain1.com/tcp/30333").unwrap();
let address2 = Multiaddr::from_str("/dns/domain2.com/tcp/30333").unwrap();
// Addresses different than the local peer ID are ignored.
let address3 = Multiaddr::from_str(
"/dns/domain2.com/tcp/30333/p2p/12D3KooWSueCPH3puP2PcvqPJdNaDNF3jMZjtJtDiSy35pWrbt5h",
)
.unwrap();

assert!(addresses.add_address(address1.clone()).unwrap());
assert!(addresses.add_address(address2.clone()).unwrap());
addresses.add_address(address3.clone()).unwrap_err();

let addresses = addresses.get_addresses();
assert_eq!(addresses.len(), 2);
assert!(addresses.contains(&address1.with(Protocol::P2p(peer_id.into()))));
assert!(addresses.contains(&address2.with(Protocol::P2p(peer_id.into()))));
}
}
Loading