Fix https://github.com/LibreQoE/LibreQoS/issues/255

src/LibreQoS.py

@@ -537,6 +537,24 @@ def refreshShapers():
 		minDownload, minUpload = findBandwidthMins(network, 0)
 		logging.info("Found the bandwidth minimums for each node")
 		
+		
+		# Child nodes inherit bandwidth maximums of parents. We apply this here to avoid bugs when compression is applied with flattenA().
+		def inheritBandwidthMaxes(data, parentMaxDL, parentMaxUL):
+			for node in data:
+				if isinstance(node, str):
+					if (isinstance(data[node], dict)) and (node != 'children'):
+						# Cap based on this node's max bandwidth, or parent node's max bandwidth, whichever is lower
+						data[node]['downloadBandwidthMbps'] = min(data[node]['downloadBandwidthMbps'],parentMaxDL)
+						data[node]['uploadBandwidthMbps'] = min(data[node]['uploadBandwidthMbps'],parentMaxUL)
+						# Recursive call this function for children nodes attached to this node
+						if 'children' in data[node]:
+							# We need to keep tabs on the minor counter, because we can't have repeating class IDs. Here, we bring back the minor counter from the recursive function
+							inheritBandwidthMaxes(data[node]['children'], data[node]['downloadBandwidthMbps'], data[node]['uploadBandwidthMbps'])
+			#return data
+		# Here is the actual call to the recursive function
+		inheritBandwidthMaxes(network, parentMaxDL=upstreamBandwidthCapacityDownloadMbps, parentMaxUL=upstreamBandwidthCapacityUploadMbps)
+
+		
 		# Compress network.json. HTB only supports 8 levels of HTB depth. Compress to 8 layers if beyond 8.
 		def flattenB(data):
 			newDict = {}