Skip to content

Commit

Permalink
Fix secure channel after removing node IDs from rendezvous (#5099)
Browse files Browse the repository at this point in the history
  • Loading branch information
Damian-Nordic authored and pull[bot] committed Mar 4, 2021
1 parent 730d200 commit 1402572
Showing 1 changed file with 3 additions and 1 deletion.
4 changes: 3 additions & 1 deletion src/transport/RendezvousSession.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -179,7 +179,9 @@ void RendezvousSession::OnSessionEstablished()
{
ChipLogError(Ble, "Missing node id in rendezvous parameters. Node ID is required until opcerts are implemented");
}
mPairingSession.PeerConnection().SetPeerNodeId(mParams.GetRemoteNodeId().ValueOr(kUndefinedNodeId));

const auto defaultPeerNodeId = mParams.IsController() ? kTestDeviceNodeId : kTestControllerNodeId;
mPairingSession.PeerConnection().SetPeerNodeId(mParams.GetRemoteNodeId().ValueOr(defaultPeerNodeId));

CHIP_ERROR err = mSecureSessionMgr->NewPairing(
Optional<Transport::PeerAddress>::Value(mPairingSession.PeerConnection().GetPeerAddress()),
Expand Down

0 comments on commit 1402572

Please sign in to comment.