Skip to content

TRUSTX Bid Adapter: initial release #14218

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
patmmccann merged 8 commits into from
Dec 9, 2025
Merged

TRUSTX Bid Adapter: initial release #14218

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
8 commits
Select commit Hold shift + click to select a range
867fc1f
TRUSTX Bidder Adapter
trustXops Nov 26, 2025
6e0dc89
Remove trustx alias in favor of standalone adapter
trustXops Nov 26, 2025
b2fbe2d
Update default ttl and usync delete url
trustXops Nov 26, 2025
8a04538
Align docs/tests with param type
trustXops Nov 26, 2025
1ee7765
Refactor duplicate code patterns
trustXops Nov 26, 2025
b224963
refactor & simplify adapter
trustXops Dec 5, 2025
8e0c80c
Fix linter error
trustXops Dec 5, 2025
cb60a03
Merge branch 'master' into trustx-bidder-adapter
patmmccann Dec 9, 2025
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
9 changes: 1 addition & 8 deletions modules/gridBidAdapter.js
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -47,13 +47,6 @@ const LOG_ERROR_MESS = {
};

const ALIAS_CONFIG = {
'trustx': {
endpoint: 'https://grid.bidswitch.net/hbjson?sp=trustx',
syncurl: 'https://x.bidswitch.net/sync?ssp=themediagrid',
bidResponseExternal: {
netRevenue: false
}
},
'gridNM': {
defaultParams: {
multiRequest: true
Expand All @@ -66,7 +59,7 @@ let hasSynced = false;
export const spec = {
code: BIDDER_CODE,
gvlid: GVLID,
aliases: ['playwire', 'adlivetech', 'gridNM', { code: 'trustx', skipPbsAliasing: true }],
aliases: ['playwire', 'adlivetech', 'gridNM'],
Copy link
Collaborator

@patmmccann patmmccann Dec 2, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

given that your new file seems largely identical to the grid adapter, could you explain why you are proposing this change?

Copy link
Contributor Author

@trustXops trustXops Dec 4, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@patmmccann TRUSTX is moving away from Criteo’s Grid to its own ad server, which requires a dedicated standalone adapter.
Any similarity to the existing Grid adapter is intentional, ensuring a smooth transition for current TRUSTX publishers while preserving performance. The new adapter remains fully compatible with existing Grid parameters—uid/secid, bidFloor, keywords, and others—so publishers can migrate without any code changes. This preserves seamless adoption and minimizes integration friction
The PR also removes the trustx alias from the Grid adapter to prevent conflicts. The new standalone adapter becomes the official and only integration point for TRUSTX.

Copy link
Collaborator

@patmmccann patmmccann Dec 4, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

do you own the endpoint? Are you really building out the deletion request support and the first party id that is redundant with shardid. They are some extremely unique things grid is doing that you are copying here that makes it seem like you are still grid

supportedMediaTypes: [ BANNER, VIDEO ],
/**
* Determines whether or not the given bid request is valid.
Expand Down
Loading
Loading