Skip to content

Update cookie sync to use bidder name instead of cookie family - #4464

Merged
And1sS merged 11 commits into
masterfrom
update-cookie-sync-to-use-bidder-name-instead-of-cookie-family
Jul 9, 2026
Merged

Update cookie sync to use bidder name instead of cookie family#4464
And1sS merged 11 commits into
masterfrom
update-cookie-sync-to-use-bidder-name-instead-of-cookie-family

Conversation

@Lightwood13

Copy link
Copy Markdown
Collaborator

Port of https://github.com/prebid/prebid-server/pull/2948/changes

  1. Updated /cookie_sync endpoint to return bidder name instead of cookie family name in the redirect url.
  2. Updated /setuid endpoint to accept bidder name instead of cookie family name.

@Lightwood13
Lightwood13 requested review from And1sS and CTMBNara April 16, 2026 12:37
@Lightwood13
Lightwood13 force-pushed the update-cookie-sync-to-use-bidder-name-instead-of-cookie-family branch from ab59133 to 522d1c8 Compare April 20, 2026 12:57
…ead-of-cookie-family

# Conflicts:
#	src/main/java/org/prebid/server/spring/config/bidder/TaboolaConfiguration.java
Comment thread src/main/java/org/prebid/server/handler/SetuidHandler.java Fixed
Comment thread src/main/java/org/prebid/server/handler/SetuidHandler.java Fixed
Comment thread src/main/java/org/prebid/server/handler/SetuidHandler.java Fixed
Comment thread src/main/java/org/prebid/server/handler/SetuidHandler.java Fixed
@Lightwood13
Lightwood13 force-pushed the update-cookie-sync-to-use-bidder-name-instead-of-cookie-family branch from df57b22 to 75acdfd Compare June 11, 2026 13:12
Comment thread src/main/java/org/prebid/server/bidder/UsersyncInfoFactory.java
Comment thread src/main/java/org/prebid/server/bidder/UsersyncInfoFactory.java Outdated
Comment thread src/main/java/org/prebid/server/cookie/CookieSyncService.java Outdated
Comment thread src/main/java/org/prebid/server/handler/SetuidHandler.java Outdated
Comment thread src/main/java/org/prebid/server/spring/config/ServiceConfiguration.java Outdated
Comment thread src/test/java/org/prebid/server/cookie/CookieSyncServiceTest.java Outdated
Comment thread src/test/java/org/prebid/server/cookie/CookieSyncServiceTest.java
Comment thread src/test/java/org/prebid/server/cookie/CookieSyncServiceTest.java
Comment thread src/test/java/org/prebid/server/cookie/CookieSyncServiceTest.java
Comment thread src/test/java/org/prebid/server/cookie/CookieSyncServiceTest.java
Comment thread src/main/java/org/prebid/server/handler/SetuidHandler.java Dismissed
Comment thread src/main/java/org/prebid/server/handler/SetuidHandler.java Dismissed
Comment thread src/main/java/org/prebid/server/handler/SetuidHandler.java Dismissed
Comment thread src/main/java/org/prebid/server/handler/SetuidHandler.java Dismissed
@Lightwood13
Lightwood13 requested a review from CTMBNara June 18, 2026 13:08
Comment thread src/main/java/org/prebid/server/bidder/UsersyncInfoFactory.java Outdated
CTMBNara
CTMBNara previously approved these changes Jul 9, 2026
@And1sS
And1sS merged commit 2880782 into master Jul 9, 2026
9 of 10 checks passed
@And1sS
And1sS deleted the update-cookie-sync-to-use-bidder-name-instead-of-cookie-family branch July 9, 2026 14:59
floxis-admin added a commit to floxis-admin/prebid-server-java that referenced this pull request Aug 8, 2026
Merge master and switch the endpoint from manual {{Macro}} substitution to org.prebid.server.util.Uri, dropping our HttpUtil.encodeUrl calls so dynamic parts are encoded once by the template instead of twice. Update the usersync redirect URL to the single-brace macro form.

Compare imp exts with equals (Lombok @value supplies it) rather than field by field. Move ObjectNode to imports. Drop the seat query param from the integration-test endpoint and its wiremock stub; seat routing is covered by unit tests, plus a new one pinning that a macro-less endpoint resolves unchanged.

Master removed UsersyncerCreator (prebid#4464); usersync is now assembled from bidder config, so drop the wiring.
@floxis-admin floxis-admin mentioned this pull request Aug 8, 2026
11 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants