Fixed panic when connectionType is not available in request (#901)

PubMatic-OpenWrap · Sep 26, 2024 · 61268f3 · 61268f3
1 parent dae441c
commit 61268f3
Showing 1 changed file with 7 additions and 1 deletion.
diff --git a/modules/pubmatic/openwrap/beforevalidationhook.go b/modules/pubmatic/openwrap/beforevalidationhook.go
@@ -347,7 +347,13 @@ func (m OpenWrap) handleBeforeValidationHook(
 					m.metricEngine.RecordReqImpsWithContentCount(rCtx.PubIDStr, models.ContentTypeSite)
 				}
 			}
-			videoAdUnitCtx = adunitconfig.UpdateVideoObjectWithAdunitConfig(rCtx, imp, div, payload.BidRequest.Device.ConnectionType)
+
+			var connectionType *adcom1.ConnectionType
+			if payload.BidRequest.Device != nil && payload.BidRequest.Device.ConnectionType != nil {
+				connectionType = payload.BidRequest.Device.ConnectionType
+			}
+
+			videoAdUnitCtx = adunitconfig.UpdateVideoObjectWithAdunitConfig(rCtx, imp, div, connectionType)
 			if rCtx.Endpoint == models.EndpointAMP && m.pubFeatures.IsAmpMultiformatEnabled(rCtx.PubID) && isVideoEnabledForAMP(videoAdUnitCtx.AppliedSlotAdUnitConfig) {
 				//Iniitalized local imp.Video object to update macros and get mappings in case of AMP request
 				rCtx.AmpVideoEnabled = true