Remove unnecessary whitelist and listener port

Signed-off-by: Irene Bandera <irenebandera@eprosima.com>
eProsima · Sep 2, 2024 · c43a6f7 · c43a6f7
1 parent dce676b
commit c43a6f7
Show file tree

Hide file tree

Showing 3 changed files with 1 addition and 13 deletions.
diff --git a/ddspipe_participants/src/cpp/participant/dds/CommonParticipant.cpp b/ddspipe_participants/src/cpp/participant/dds/CommonParticipant.cpp
@@ -499,7 +499,7 @@ CommonParticipant::reckon_participant_qos_(
 
 fastdds::dds::DomainParticipant* CommonParticipant::create_dds_participant_()
 {
-    // Set listener mask so reader read its own messages TODO Irene: check if this is needed
+    // Set listener mask so reader read its own messages
     fastdds::dds::StatusMask mask;
     mask << fastdds::dds::StatusMask::publication_matched();
     mask << fastdds::dds::StatusMask::subscription_matched();

diff --git a/ddspipe_participants/src/cpp/participant/dds/DiscoveryServerParticipant.cpp b/ddspipe_participants/src/cpp/participant/dds/DiscoveryServerParticipant.cpp
@@ -254,7 +254,6 @@ DiscoveryServerParticipant::reckon_participant_qos_(
                         auto descriptor_tmp =
                                 create_descriptor<eprosima::fastdds::rtps::UDPv4TransportDescriptor>(
                             participant_configuration->whitelist);
-                        // descriptor_tmp->interfaceWhiteList.push_back(address.ip());
                         descriptor = descriptor_tmp;
                     }
 
@@ -271,7 +270,6 @@ DiscoveryServerParticipant::reckon_participant_qos_(
                         auto descriptor_tmp =
                                 create_descriptor<eprosima::fastdds::rtps::UDPv6TransportDescriptor>(
                             participant_configuration->whitelist);
-                        // descriptor_tmp->interfaceWhiteList.push_back(address.ip());
                         descriptor = descriptor_tmp;
                     }
 
@@ -288,8 +286,6 @@ DiscoveryServerParticipant::reckon_participant_qos_(
                         auto descriptor_tmp =
                                 create_descriptor<eprosima::fastdds::rtps::TCPv4TransportDescriptor>(
                             participant_configuration->whitelist);
-                        descriptor_tmp->add_listener_port(0);
-                        // descriptor_tmp->interfaceWhiteList.push_back(address.ip());
 
                         // Enable TLS
                         if (tls_config.is_active())
@@ -314,8 +310,6 @@ DiscoveryServerParticipant::reckon_participant_qos_(
                         auto descriptor_tmp =
                                 create_descriptor<eprosima::fastdds::rtps::TCPv6TransportDescriptor>(
                             participant_configuration->whitelist);
-                        // descriptor_tmp->add_listener_port(0);
-                        descriptor_tmp->interfaceWhiteList.push_back(address.ip());
 
                         // Enable TLS
                         if (tls_config.is_active())

diff --git a/ddspipe_participants/src/cpp/participant/dds/InitialPeersParticipant.cpp b/ddspipe_participants/src/cpp/participant/dds/InitialPeersParticipant.cpp
@@ -234,7 +234,6 @@ InitialPeersParticipant::reckon_participant_qos_(
                     auto descriptor_tmp =
                             create_descriptor<eprosima::fastdds::rtps::UDPv4TransportDescriptor>(
                         participant_configuration->whitelist);
-                    // descriptor_tmp->interfaceWhiteList.push_back(connection_address.ip());
                     descriptor = descriptor_tmp;
 
                     // To avoid creating a multicast transport in UDP when non listening addresses
@@ -257,7 +256,6 @@ InitialPeersParticipant::reckon_participant_qos_(
                     auto descriptor_tmp =
                             create_descriptor<eprosima::fastdds::rtps::UDPv6TransportDescriptor>(
                         participant_configuration->whitelist);
-                    // descriptor_tmp->interfaceWhiteList.push_back(connection_address.ip());
                     descriptor = descriptor_tmp;
 
                     // To avoid creating a multicast transport in UDP when non listening addresses
@@ -280,8 +278,6 @@ InitialPeersParticipant::reckon_participant_qos_(
                     auto descriptor_tmp =
                             create_descriptor<eprosima::fastdds::rtps::TCPv4TransportDescriptor>(
                         participant_configuration->whitelist);
-                    descriptor_tmp->add_listener_port(0);
-                    // descriptor_tmp->interfaceWhiteList.push_back(address.ip());
 
                     // Enable TLS
                     if (tls_config.is_active())
@@ -306,8 +302,6 @@ InitialPeersParticipant::reckon_participant_qos_(
                     auto descriptor_tmp =
                             create_descriptor<eprosima::fastdds::rtps::TCPv6TransportDescriptor>(
                         participant_configuration->whitelist);
-                    // descriptor_tmp->add_listener_port(0);
-                    descriptor_tmp->interfaceWhiteList.push_back(connection_address.ip());
 
                     // Enable TLS
                     if (tls_config.is_active())