Skip to content

Commit 863f153

Browse files
author
Michele Nasti
committed
fix for privacyLink in native prebid#10249
1 parent dd4b25e commit 863f153

File tree

3 files changed

+17
-3
lines changed

3 files changed

+17
-3
lines changed

src/constants.json

-1
Original file line numberDiff line numberDiff line change
@@ -163,7 +163,6 @@
163163
"MAIN": 3
164164
},
165165
"NATIVE_KEYS_THAT_ARE_NOT_ASSETS": [
166-
"privacyLink",
167166
"clickUrl",
168167
"sendTargetingKeys",
169168
"adTemplate",

src/native.js

+8
Original file line numberDiff line numberDiff line change
@@ -480,6 +480,11 @@ export function toOrtbNativeRequest(legacyNativeAssets) {
480480
continue;
481481
}
482482

483+
if (key === 'privacyLink') {
484+
ortb.privacy = 1;
485+
continue;
486+
}
487+
483488
const asset = legacyNativeAssets[key];
484489
let required = 0;
485490
if (asset.required && isBoolean(asset.required)) {
@@ -623,6 +628,9 @@ export function fromOrtbNativeRequest(openRTBRequest) {
623628
oldNativeObject[prebidAssetName].len = asset.data.len;
624629
}
625630
}
631+
if (openRTBRequest.privacy) {
632+
oldNativeObject.privacyLink = { required: true };
633+
}
626634
// video was not supported by old prebid assets
627635
}
628636
return oldNativeObject;

test/spec/native_spec.js

+9-2
Original file line numberDiff line numberDiff line change
@@ -860,6 +860,9 @@ describe('validate native', function () {
860860
}]
861861
},
862862
address: {},
863+
privacyLink: {
864+
required: true
865+
}
863866
},
864867
},
865868
};
@@ -915,6 +918,7 @@ describe('validate native', function () {
915918
type: 9,
916919
}
917920
});
921+
expect(ortb.privacy).to.equal(1);
918922
});
919923

920924
['bogusKey', 'clickUrl', 'privacyLink'].forEach(nativeKey => {
@@ -1022,11 +1026,14 @@ describe('validate native', function () {
10221026
expect(oldNativeRequest.sponsoredBy).to.include({
10231027
required: true,
10241028
len: 25
1025-
})
1029+
});
10261030
expect(oldNativeRequest.body).to.include({
10271031
required: true,
10281032
len: 140
1029-
})
1033+
});
1034+
expect(oldNativeRequest.privacyLink).to.include({
1035+
required: true
1036+
});
10301037
});
10311038

10321039
if (FEATURES.NATIVE) {

0 commit comments

Comments
 (0)