Fix typo and improve logging.

pkg/sip/client.go

@@ -20,6 +20,7 @@ import (
 	"sync"
 
 	"github.com/emiago/sipgo"
+	"github.com/livekit/protocol/logger"
 	"github.com/livekit/protocol/rpc"
 	"golang.org/x/exp/maps"
 
@@ -79,12 +80,17 @@ func (c *Client) Stop() error {
 
 func (c *Client) UpdateSIPParticipant(ctx context.Context, req *rpc.InternalUpdateSIPParticipantRequest) (*rpc.InternalUpdateSIPParticipantResponse, error) {
 	if req.CallTo == "" {
+		logger.Infow("Disconnect SIP participant",
+			"roomName", req.RoomName, "participant", req.ParticipantId)
 		// Disconnect participant
 		if call := c.getCall(req.ParticipantId); call != nil {
 			call.Close()
 		}
 		return &rpc.InternalUpdateSIPParticipantResponse{}, nil
 	}
+	logger.Infow("Updating SIP participant",
+		"roomName", req.RoomName, "participant", req.ParticipantId,
+		"from", req.Number, "to", req.CallTo, "address", req.Address)
 	err := c.getOrCreateCall(req.ParticipantId).Update(ctx, sipOutboundConfig{
 		address: req.Address,
 		from:    req.Number,

pkg/sip/outbound.go

@@ -125,24 +125,28 @@ func (c *outboundCall) Update(ctx context.Context, sipNew sipOutboundConfig, lkN
 	if c.sipCur == sipNew && c.lkCur == lkNew {
 		return nil
 	}
-	if c.sipCur.address == "" || c.sipCur.to == "" {
+	if sipNew.address == "" || sipNew.to == "" {
+		logger.Infow("Shutdown of outbound SIP call",
+			"roomName", lkNew.roomName, "from", sipNew.from, "to", sipNew.to, "address", sipNew.address)
 		// shutdown the call
 		c.close()
 		return nil
 	}
 	if err := c.startMedia(); err != nil {
 		c.close()
-		return err
+		return fmt.Errorf("start media failed: %w", err)
 	}
 	if err := c.updateRoom(lkNew); err != nil {
 		c.close()
-		return err
+		return fmt.Errorf("update room failed: %w", err)
 	}
 	if err := c.updateSIP(sipNew); err != nil {
 		c.close()
-		return err
+		return fmt.Errorf("update SIP failed: %w", err)
 	}
 	c.relinkMedia()
+	logger.Infow("Outbound SIP update complete",
+		"roomName", lkNew.roomName, "from", sipNew.from, "to", sipNew.to, "address", sipNew.address)
 	return nil
 }