From da2e6d4649e72ddb5039fc8618243fe64f66b956 Mon Sep 17 00:00:00 2001
From: Anaswara Nambiar <159991511+anaswarac-dac@users.noreply.github.com>
Date: Wed, 14 Jan 2026 12:38:10 +0530
Subject: [PATCH] Refactor bandwidth logic and add debug logging

Refactor bandwidth checks to simplify logic for Downlink and Uplink MBR updates. Added logging for cases where both max bandwidth values are zero.
---
 producer/policyauthorization.go | 13 +++++++------
 1 file changed, 7 insertions(+), 6 deletions(-)

diff --git a/producer/policyauthorization.go b/producer/policyauthorization.go
index 15f1ed4a..acfaaaa4 100644
--- a/producer/policyauthorization.go
+++ b/producer/policyauthorization.go
@@ -1847,15 +1847,11 @@ func updateQosInMedSubComp(qosData *models.QosData, comp *models.MediaComponent,
 	}
 
 	// update Downlink MBR
-	if maxBwDl == 0.0 {
-		updatedQosData.MaxbrDl = comp.MarBwDl
-	} else {
+	if maxBwDl != 0.0 {
 		updatedQosData.MaxbrDl = pcf_context.ConvertBitRateToString(maxBwDl)
 	}
 	// update Uplink MBR
-	if maxBwUl == 0.0 {
-		updatedQosData.MaxbrUl = comp.MarBwUl
-	} else {
+	if maxBwUl != 0.0 {
 		updatedQosData.MaxbrUl = pcf_context.ConvertBitRateToString(maxBwUl)
 	}
 	// if gbr == 0 then assign gbr = mbr
@@ -1867,6 +1863,11 @@ func updateQosInMedSubComp(qosData *models.QosData, comp *models.MediaComponent,
 	if minBwUl != 0.0 {
 		updatedQosData.GbrUl = pcf_context.ConvertBitRateToString(minBwUl)
 	}
+
+	if maxBwUl == 0 && maxBwDl == 0 {
+		logger.PolicyAuthorizationlog.Debugf(
+			"MediaSubComp produced zero bandwidth, keeping existing QoS unchanged")
+	}
 	return updatedQosData, ulExist, dlExist
 }