Skip to content

Fix extra preplay parameter when no ping is configured#71

Merged
MattiasBuelens merged 3 commits intomasterfrom
bugfix/THEOSD-16012-uplynk-preplay
Oct 8, 2025
Merged

Fix extra preplay parameter when no ping is configured#71
MattiasBuelens merged 3 commits intomasterfrom
bugfix/THEOSD-16012-uplynk-preplay

Conversation

@ceyhun-o
Copy link
Contributor

@ceyhun-o ceyhun-o commented Oct 8, 2025

This PR fixes an issue where the player adds a &ad.pingc=0 to preplay parameters when there is no ping configuration, which causes a 403 since the requested preplay URL no longer matches the signature due to the extra &ad.pingc=0. This change aligns the behavior to the Web SDK implementation.

There is also a small correction for the parameter name.

@MattiasBuelens MattiasBuelens added bug Something isn't working 🔌 connector: uplynk Affects the Uplynk connector labels Oct 8, 2025
@MattiasBuelens MattiasBuelens self-assigned this Oct 8, 2025
@MattiasBuelens MattiasBuelens merged commit d00d6ef into master Oct 8, 2025
1 check failed
@MattiasBuelens MattiasBuelens deleted the bugfix/THEOSD-16012-uplynk-preplay branch October 8, 2025 12:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working 🔌 connector: uplynk Affects the Uplynk connector

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants