Skip to content

Commit

Permalink
Fixing change that was introduced on Fix for Find messages commit (#842)
Browse files Browse the repository at this point in the history
Fixing for correct behaviour :
- Find messages received with Unicast Flag set to 0 (multicast) shall be answered 
 with a multicast response.
- Using the same send method as used for cyclic offers to preserve the session id.
Co-authored-by: Duarte Fonseca <[email protected]>
  • Loading branch information
duartenfonseca authored Jan 30, 2025
1 parent 683e145 commit 25bf6ed
Showing 1 changed file with 4 additions and 4 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -1704,9 +1704,7 @@ service_discovery_impl::send_unicast_offer_service(

insert_offer_service(its_messages, _info);

// send message as multicast offer service the same way it is sent
// on the repetition phase to preserve the session id
send(its_messages);
serialize_and_send(its_messages, current_remote_address_);
}

void
Expand All @@ -1718,7 +1716,9 @@ service_discovery_impl::send_multicast_offer_service(

insert_offer_service(its_messages, _info);

serialize_and_send(its_messages, sd_multicast_address_);
// send message as multicast offer service the same way it is sent
// on the repetition phase to preserve the session id
send(its_messages);
}

void
Expand Down

0 comments on commit 25bf6ed

Please sign in to comment.