Skip to content
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.

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

Rubicon Bid Adapter: expand fastlane EID protocol and pass p_site.mobile; Support device.ip and device.ipv6. #12471

Conversation

apukh-magnite
Copy link
Contributor

Type of change

  • Feature
  • Refactoring (no functional changes, no api changes)'

Description of change

Update Rubicon PBJS adapter to pass enhanced EID data, site.mobile, and IP information

  • Replaced legacy request.userid with user.ext.eids for user identification.
  • Updated the format of EID data in fastlane.json to include uid, atype, third (empty), inserter, matcher, mm, and rtipartner.
  • Added handling for ortb2.site.mobile and passed it as p_site.mobile to the Fastlane endpoint.
  • Included device IP and IPv6 addresses as 'ip' and 'ipv6' in the request data.

…h source, retain ppuid logic; Add ip and ipv6 from device in bidRequest
@robertrmartinez robertrmartinez self-requested a review November 19, 2024 16:47
@robertrmartinez robertrmartinez self-assigned this Nov 19, 2024
Copy link
Collaborator

@robertrmartinez robertrmartinez left a comment

Choose a reason for hiding this comment

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

one small change

@@ -3,7 +3,6 @@ import { pbsExtensions } from '../libraries/pbsExtensions/pbsExtensions.js';
import { registerBidder } from '../src/adapters/bidderFactory.js';
import { config } from '../src/config.js';
import { BANNER, NATIVE, VIDEO } from '../src/mediaTypes.js';
import { find } from '../src/polyfill.js';
Copy link
Collaborator

Choose a reason for hiding this comment

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

This is not used anywhere? It def should not be!

@@ -542,42 +543,47 @@ export const spec = {
if (bidRequest?.ortb2Imp?.ext?.ae) {
data['o_ae'] = 1;
}
// If the bid request contains a 'mobile' property under 'ortb2.site', add it to 'data' as 'p_site.mobile'.
if (bidRequest?.ortb2?.site?.mobile) {
Copy link
Collaborator

Choose a reason for hiding this comment

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

If defined mobile is either 0 or 1

So, with this current code it would only pass it if it was 1 since 0 is falsy.

So lets add typeof here and pass along.

if (typeof bidRequest?.ortb2?.site?.mobile === 'number') {

Also add a test to confirm p_site.mobile

@robertrmartinez robertrmartinez merged commit 5bc249c into prebid:master Nov 19, 2024
6 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants