-
Notifications
You must be signed in to change notification settings - Fork 2.1k
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
Adriver Bid and Id Modules: buyerid bug fix #8768
Conversation
@m-oranskaya I will follow-up on the download page to add adriver |
We think that error in the CircleCI tests is not related to our changes. Shall we do something else to get our changes approved? |
Hi, why do you send id's from other modules ? (the CI error is not related to your changes, I confirm) |
modules/adriverIdSystem.js
Outdated
} | ||
let sharedID = storage.getDataFromLocalStorage('_pubcid') || storage.getCookie('pubcid'); | ||
let unifiedID = storage.getCookie('__uid2_advertising_token'); | ||
let newUrl = url + '&custom=' + (universalVal ? '221=' + encodeURIComponent(universalVal) + ';' : '') + (sharedID ? '222=' + encodeURIComponent(sharedID) + ';' : '') + (unifiedID ? '223=' + encodeURIComponent(unifiedID) : '') + '&cid=' + (storage.getDataFromLocalStorage('adrcid') || storage.getCookie('adrcid')); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
you can't do this
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
please see id module rules at https://docs.prebid.org/dev-docs/module-rules.html
We have made the necessary changes, is there anything else we need to do to get the Pull Request approved? |
modules/adriverBidAdapter.js
Outdated
@@ -99,22 +97,17 @@ export const spec = { | |||
}); | |||
}); | |||
|
|||
let userid = validBidRequests[0].userId; | |||
let adrcidCookie = storage.getDataFromLocalStorage('adrcid') || validBidRequests[0].userId.adrcid; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
During my tests, I saw the fallback throw an error during the first auction, because the userId is not defined.
Please just update to something like : || validBidRequests[0].userId?.adrcid;
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
Could you add our userId module to download page? |
|
* Readme: remove dead dev dependency and maintenance badges (#8849) * Readme: remove dead dev dependency badge and issue * Remove maintainability and add back issues * Prebid 7.11.0 release * Increment version to 7.12.0-pre * Adriver Bid and Id Modules: buyerid bug fix (#8768) * initial commit * adriver id submodule add * add id system tests, fix adriver bid adapter tests * adriver: fix buyerid * remarks fixing * removal of excess * delete custom parameter * bug fixes * handle native response privacy link (#8838) Co-authored-by: Zicong Zhou <zicong.zhou@indexexchange.com> * Criteo Bid Adapter - Add support for banner+native multiformat ad unit (#8842) Previously, the use of a native adunit was exclusive with the banner type. * Ras Bid Adapter: support for SlotSequence parameter (#8792) * add rasbidadapter pos param * Read pos off the adunit * rename conflicting pos parameter for clarity * Improve Digital adapter: refactor code to align with latest RAZR creative tags (#8827) * IX Bid Adapter: Native OpenRTB Request Support (#8853) * fix native click trackers to only fire on click * fix unit tests for ix * remove version for native requests * remove unnecessary request conversion Co-authored-by: Zicong Zhou <zicong.zhou@indexexchange.com> * Update Sonobi adapter with GVLID (#8860) * dgkeyword RTD provider: fix tests causing ID5 test failures (#8862) * Id5 id configurable fetch flow (#8784) Co-authored-by: Paweł Kowalski <pkowalski@id5.io> * NextMillenium Bid Adapter: Remove ortb2 referrerInfo (#8868) * remove ortb2, get device/site manually * updated tests * remove fallbacks * no need to craete variable if there is no fallback (return in place) * removed one test case Co-authored-by: Yakov Klein <yakovklein@Yakovs-MacBook-Air.local> * adserver.js : remove unused code (#8855) * Displayio Bid Adapter: custom render; fix eids payload (#8847) * Custom render; call pubmatic get user id function is removed * use refererInfo; remove call createEidsArray * VidCrunch LLC bidder (#8872) Co-authored-by: Chris Huie <phoenixtechnerd@gmail.com> Co-authored-by: Prebid.js automated release <prebidjs-release@prebid.org> Co-authored-by: m-oranskaya <99481039+m-oranskaya@users.noreply.github.com> Co-authored-by: Love Sharma <sharmak.love@gmail.com> Co-authored-by: Zicong Zhou <zicong.zhou@indexexchange.com> Co-authored-by: Léonard Labat <le.labat@criteo.com> Co-authored-by: wsusrasp <106743463+wsusrasp@users.noreply.github.com> Co-authored-by: Catalin Ciocov <catalin.ciocov@gmail.com> Co-authored-by: Mike Miller <mike@solitaired.com> Co-authored-by: Demetrio Girardi <dgirardi@prebid.org> Co-authored-by: Scott Menzer <scott@id5.io> Co-authored-by: Paweł Kowalski <pkowalski@id5.io> Co-authored-by: JacobKlein26 <42449375+JacobKlein26@users.noreply.github.com> Co-authored-by: Yakov Klein <yakovklein@Yakovs-MacBook-Air.local> Co-authored-by: caseywhitmire <60086994+caseywhitmire@users.noreply.github.com> Co-authored-by: philan15 <37775368+philan15@users.noreply.github.com> Co-authored-by: Gena <wertixvost@gmail.com>
Please tell the approximate time when you will be able to add our module on https://docs.prebid.org/download.html in the "User ID Modules" section. It is very important for us. Several clients that use prebid are looking forward to being able to generate code. |
* Create codeql-analysis.yml * Update codeql-analysis.yml * Update instream.html * Update emx_digitalBidAdapter_spec.js * Update index.js * Update index.js * Update index.js * Merge master (#8873) * Readme: remove dead dev dependency and maintenance badges (#8849) * Readme: remove dead dev dependency badge and issue * Remove maintainability and add back issues * Prebid 7.11.0 release * Increment version to 7.12.0-pre * Adriver Bid and Id Modules: buyerid bug fix (#8768) * initial commit * adriver id submodule add * add id system tests, fix adriver bid adapter tests * adriver: fix buyerid * remarks fixing * removal of excess * delete custom parameter * bug fixes * handle native response privacy link (#8838) Co-authored-by: Zicong Zhou <zicong.zhou@indexexchange.com> * Criteo Bid Adapter - Add support for banner+native multiformat ad unit (#8842) Previously, the use of a native adunit was exclusive with the banner type. * Ras Bid Adapter: support for SlotSequence parameter (#8792) * add rasbidadapter pos param * Read pos off the adunit * rename conflicting pos parameter for clarity * Improve Digital adapter: refactor code to align with latest RAZR creative tags (#8827) * IX Bid Adapter: Native OpenRTB Request Support (#8853) * fix native click trackers to only fire on click * fix unit tests for ix * remove version for native requests * remove unnecessary request conversion Co-authored-by: Zicong Zhou <zicong.zhou@indexexchange.com> * Update Sonobi adapter with GVLID (#8860) * dgkeyword RTD provider: fix tests causing ID5 test failures (#8862) * Id5 id configurable fetch flow (#8784) Co-authored-by: Paweł Kowalski <pkowalski@id5.io> * NextMillenium Bid Adapter: Remove ortb2 referrerInfo (#8868) * remove ortb2, get device/site manually * updated tests * remove fallbacks * no need to craete variable if there is no fallback (return in place) * removed one test case Co-authored-by: Yakov Klein <yakovklein@Yakovs-MacBook-Air.local> * adserver.js : remove unused code (#8855) * Displayio Bid Adapter: custom render; fix eids payload (#8847) * Custom render; call pubmatic get user id function is removed * use refererInfo; remove call createEidsArray * VidCrunch LLC bidder (#8872) Co-authored-by: Chris Huie <phoenixtechnerd@gmail.com> Co-authored-by: Prebid.js automated release <prebidjs-release@prebid.org> Co-authored-by: m-oranskaya <99481039+m-oranskaya@users.noreply.github.com> Co-authored-by: Love Sharma <sharmak.love@gmail.com> Co-authored-by: Zicong Zhou <zicong.zhou@indexexchange.com> Co-authored-by: Léonard Labat <le.labat@criteo.com> Co-authored-by: wsusrasp <106743463+wsusrasp@users.noreply.github.com> Co-authored-by: Catalin Ciocov <catalin.ciocov@gmail.com> Co-authored-by: Mike Miller <mike@solitaired.com> Co-authored-by: Demetrio Girardi <dgirardi@prebid.org> Co-authored-by: Scott Menzer <scott@id5.io> Co-authored-by: Paweł Kowalski <pkowalski@id5.io> Co-authored-by: JacobKlein26 <42449375+JacobKlein26@users.noreply.github.com> Co-authored-by: Yakov Klein <yakovklein@Yakovs-MacBook-Air.local> Co-authored-by: caseywhitmire <60086994+caseywhitmire@users.noreply.github.com> Co-authored-by: philan15 <37775368+philan15@users.noreply.github.com> Co-authored-by: Gena <wertixvost@gmail.com> * Revert "Merge master (#8873)" (#8911) This reverts commit 07c5369. * Update code scan config * Update viewability * Sanitize JSON for adgeneration Co-authored-by: Patrick McCann <patmmccann@gmail.com> Co-authored-by: Chris Huie <phoenixtechnerd@gmail.com> Co-authored-by: Prebid.js automated release <prebidjs-release@prebid.org> Co-authored-by: m-oranskaya <99481039+m-oranskaya@users.noreply.github.com> Co-authored-by: Love Sharma <sharmak.love@gmail.com> Co-authored-by: Zicong Zhou <zicong.zhou@indexexchange.com> Co-authored-by: Léonard Labat <le.labat@criteo.com> Co-authored-by: wsusrasp <106743463+wsusrasp@users.noreply.github.com> Co-authored-by: Catalin Ciocov <catalin.ciocov@gmail.com> Co-authored-by: Mike Miller <mike@solitaired.com> Co-authored-by: Scott Menzer <scott@id5.io> Co-authored-by: Paweł Kowalski <pkowalski@id5.io> Co-authored-by: JacobKlein26 <42449375+JacobKlein26@users.noreply.github.com> Co-authored-by: Yakov Klein <yakovklein@Yakovs-MacBook-Air.local> Co-authored-by: caseywhitmire <60086994+caseywhitmire@users.noreply.github.com> Co-authored-by: philan15 <37775368+philan15@users.noreply.github.com> Co-authored-by: Gena <wertixvost@gmail.com>
Fix Adriver ID system test failures (https://app.circleci.com/pipelines/github/prebid/Prebid.js/13137/workflows/5e686ed9-e57e-47b3-9c4e-8beea10be321/jobs/24200). This was already done in prebid#8489, but prebid#8768 undid it.
Fix Adriver ID system test failures (https://app.circleci.com/pipelines/github/prebid/Prebid.js/13137/workflows/5e686ed9-e57e-47b3-9c4e-8beea10be321/jobs/24200). This was already done in #8489, but #8768 undid it.
Hello, @osazos, @ChrisHuie, @patmmccann ! |
Hello, @osazos, @ChrisHuie, @patmmccann ! |
Actually you only have your bidAdapter documented. You have to create a new PR in https://github.com/prebid/prebid.github.io to add your user id module.
Take a look at others modules files as example. Sorry for the response delay. |
Hi @osazos ! Thank you for the information. |
* initial commit * adriver id submodule add * add id system tests, fix adriver bid adapter tests * adriver: fix buyerid * remarks fixing * removal of excess * delete custom parameter * bug fixes
* Create codeql-analysis.yml * Update codeql-analysis.yml * Update instream.html * Update emx_digitalBidAdapter_spec.js * Update index.js * Update index.js * Update index.js * Merge master (prebid#8873) * Readme: remove dead dev dependency and maintenance badges (prebid#8849) * Readme: remove dead dev dependency badge and issue * Remove maintainability and add back issues * Prebid 7.11.0 release * Increment version to 7.12.0-pre * Adriver Bid and Id Modules: buyerid bug fix (prebid#8768) * initial commit * adriver id submodule add * add id system tests, fix adriver bid adapter tests * adriver: fix buyerid * remarks fixing * removal of excess * delete custom parameter * bug fixes * handle native response privacy link (prebid#8838) Co-authored-by: Zicong Zhou <zicong.zhou@indexexchange.com> * Criteo Bid Adapter - Add support for banner+native multiformat ad unit (prebid#8842) Previously, the use of a native adunit was exclusive with the banner type. * Ras Bid Adapter: support for SlotSequence parameter (prebid#8792) * add rasbidadapter pos param * Read pos off the adunit * rename conflicting pos parameter for clarity * Improve Digital adapter: refactor code to align with latest RAZR creative tags (prebid#8827) * IX Bid Adapter: Native OpenRTB Request Support (prebid#8853) * fix native click trackers to only fire on click * fix unit tests for ix * remove version for native requests * remove unnecessary request conversion Co-authored-by: Zicong Zhou <zicong.zhou@indexexchange.com> * Update Sonobi adapter with GVLID (prebid#8860) * dgkeyword RTD provider: fix tests causing ID5 test failures (prebid#8862) * Id5 id configurable fetch flow (prebid#8784) Co-authored-by: Paweł Kowalski <pkowalski@id5.io> * NextMillenium Bid Adapter: Remove ortb2 referrerInfo (prebid#8868) * remove ortb2, get device/site manually * updated tests * remove fallbacks * no need to craete variable if there is no fallback (return in place) * removed one test case Co-authored-by: Yakov Klein <yakovklein@Yakovs-MacBook-Air.local> * adserver.js : remove unused code (prebid#8855) * Displayio Bid Adapter: custom render; fix eids payload (prebid#8847) * Custom render; call pubmatic get user id function is removed * use refererInfo; remove call createEidsArray * VidCrunch LLC bidder (prebid#8872) Co-authored-by: Chris Huie <phoenixtechnerd@gmail.com> Co-authored-by: Prebid.js automated release <prebidjs-release@prebid.org> Co-authored-by: m-oranskaya <99481039+m-oranskaya@users.noreply.github.com> Co-authored-by: Love Sharma <sharmak.love@gmail.com> Co-authored-by: Zicong Zhou <zicong.zhou@indexexchange.com> Co-authored-by: Léonard Labat <le.labat@criteo.com> Co-authored-by: wsusrasp <106743463+wsusrasp@users.noreply.github.com> Co-authored-by: Catalin Ciocov <catalin.ciocov@gmail.com> Co-authored-by: Mike Miller <mike@solitaired.com> Co-authored-by: Demetrio Girardi <dgirardi@prebid.org> Co-authored-by: Scott Menzer <scott@id5.io> Co-authored-by: Paweł Kowalski <pkowalski@id5.io> Co-authored-by: JacobKlein26 <42449375+JacobKlein26@users.noreply.github.com> Co-authored-by: Yakov Klein <yakovklein@Yakovs-MacBook-Air.local> Co-authored-by: caseywhitmire <60086994+caseywhitmire@users.noreply.github.com> Co-authored-by: philan15 <37775368+philan15@users.noreply.github.com> Co-authored-by: Gena <wertixvost@gmail.com> * Revert "Merge master (prebid#8873)" (prebid#8911) This reverts commit 07c5369. * Update code scan config * Update viewability * Sanitize JSON for adgeneration Co-authored-by: Patrick McCann <patmmccann@gmail.com> Co-authored-by: Chris Huie <phoenixtechnerd@gmail.com> Co-authored-by: Prebid.js automated release <prebidjs-release@prebid.org> Co-authored-by: m-oranskaya <99481039+m-oranskaya@users.noreply.github.com> Co-authored-by: Love Sharma <sharmak.love@gmail.com> Co-authored-by: Zicong Zhou <zicong.zhou@indexexchange.com> Co-authored-by: Léonard Labat <le.labat@criteo.com> Co-authored-by: wsusrasp <106743463+wsusrasp@users.noreply.github.com> Co-authored-by: Catalin Ciocov <catalin.ciocov@gmail.com> Co-authored-by: Mike Miller <mike@solitaired.com> Co-authored-by: Scott Menzer <scott@id5.io> Co-authored-by: Paweł Kowalski <pkowalski@id5.io> Co-authored-by: JacobKlein26 <42449375+JacobKlein26@users.noreply.github.com> Co-authored-by: Yakov Klein <yakovklein@Yakovs-MacBook-Air.local> Co-authored-by: caseywhitmire <60086994+caseywhitmire@users.noreply.github.com> Co-authored-by: philan15 <37775368+philan15@users.noreply.github.com> Co-authored-by: Gena <wertixvost@gmail.com>
Fix Adriver ID system test failures (https://app.circleci.com/pipelines/github/prebid/Prebid.js/13137/workflows/5e686ed9-e57e-47b3-9c4e-8beea10be321/jobs/24200). This was already done in prebid#8489, but prebid#8768 undid it.
* initial commit * adriver id submodule add * add id system tests, fix adriver bid adapter tests * adriver: fix buyerid * remarks fixing * removal of excess * delete custom parameter * bug fixes
* Create codeql-analysis.yml * Update codeql-analysis.yml * Update instream.html * Update emx_digitalBidAdapter_spec.js * Update index.js * Update index.js * Update index.js * Merge master (prebid#8873) * Readme: remove dead dev dependency and maintenance badges (prebid#8849) * Readme: remove dead dev dependency badge and issue * Remove maintainability and add back issues * Prebid 7.11.0 release * Increment version to 7.12.0-pre * Adriver Bid and Id Modules: buyerid bug fix (prebid#8768) * initial commit * adriver id submodule add * add id system tests, fix adriver bid adapter tests * adriver: fix buyerid * remarks fixing * removal of excess * delete custom parameter * bug fixes * handle native response privacy link (prebid#8838) Co-authored-by: Zicong Zhou <zicong.zhou@indexexchange.com> * Criteo Bid Adapter - Add support for banner+native multiformat ad unit (prebid#8842) Previously, the use of a native adunit was exclusive with the banner type. * Ras Bid Adapter: support for SlotSequence parameter (prebid#8792) * add rasbidadapter pos param * Read pos off the adunit * rename conflicting pos parameter for clarity * Improve Digital adapter: refactor code to align with latest RAZR creative tags (prebid#8827) * IX Bid Adapter: Native OpenRTB Request Support (prebid#8853) * fix native click trackers to only fire on click * fix unit tests for ix * remove version for native requests * remove unnecessary request conversion Co-authored-by: Zicong Zhou <zicong.zhou@indexexchange.com> * Update Sonobi adapter with GVLID (prebid#8860) * dgkeyword RTD provider: fix tests causing ID5 test failures (prebid#8862) * Id5 id configurable fetch flow (prebid#8784) Co-authored-by: Paweł Kowalski <pkowalski@id5.io> * NextMillenium Bid Adapter: Remove ortb2 referrerInfo (prebid#8868) * remove ortb2, get device/site manually * updated tests * remove fallbacks * no need to craete variable if there is no fallback (return in place) * removed one test case Co-authored-by: Yakov Klein <yakovklein@Yakovs-MacBook-Air.local> * adserver.js : remove unused code (prebid#8855) * Displayio Bid Adapter: custom render; fix eids payload (prebid#8847) * Custom render; call pubmatic get user id function is removed * use refererInfo; remove call createEidsArray * VidCrunch LLC bidder (prebid#8872) Co-authored-by: Chris Huie <phoenixtechnerd@gmail.com> Co-authored-by: Prebid.js automated release <prebidjs-release@prebid.org> Co-authored-by: m-oranskaya <99481039+m-oranskaya@users.noreply.github.com> Co-authored-by: Love Sharma <sharmak.love@gmail.com> Co-authored-by: Zicong Zhou <zicong.zhou@indexexchange.com> Co-authored-by: Léonard Labat <le.labat@criteo.com> Co-authored-by: wsusrasp <106743463+wsusrasp@users.noreply.github.com> Co-authored-by: Catalin Ciocov <catalin.ciocov@gmail.com> Co-authored-by: Mike Miller <mike@solitaired.com> Co-authored-by: Demetrio Girardi <dgirardi@prebid.org> Co-authored-by: Scott Menzer <scott@id5.io> Co-authored-by: Paweł Kowalski <pkowalski@id5.io> Co-authored-by: JacobKlein26 <42449375+JacobKlein26@users.noreply.github.com> Co-authored-by: Yakov Klein <yakovklein@Yakovs-MacBook-Air.local> Co-authored-by: caseywhitmire <60086994+caseywhitmire@users.noreply.github.com> Co-authored-by: philan15 <37775368+philan15@users.noreply.github.com> Co-authored-by: Gena <wertixvost@gmail.com> * Revert "Merge master (prebid#8873)" (prebid#8911) This reverts commit 07c5369. * Update code scan config * Update viewability * Sanitize JSON for adgeneration Co-authored-by: Patrick McCann <patmmccann@gmail.com> Co-authored-by: Chris Huie <phoenixtechnerd@gmail.com> Co-authored-by: Prebid.js automated release <prebidjs-release@prebid.org> Co-authored-by: m-oranskaya <99481039+m-oranskaya@users.noreply.github.com> Co-authored-by: Love Sharma <sharmak.love@gmail.com> Co-authored-by: Zicong Zhou <zicong.zhou@indexexchange.com> Co-authored-by: Léonard Labat <le.labat@criteo.com> Co-authored-by: wsusrasp <106743463+wsusrasp@users.noreply.github.com> Co-authored-by: Catalin Ciocov <catalin.ciocov@gmail.com> Co-authored-by: Mike Miller <mike@solitaired.com> Co-authored-by: Scott Menzer <scott@id5.io> Co-authored-by: Paweł Kowalski <pkowalski@id5.io> Co-authored-by: JacobKlein26 <42449375+JacobKlein26@users.noreply.github.com> Co-authored-by: Yakov Klein <yakovklein@Yakovs-MacBook-Air.local> Co-authored-by: caseywhitmire <60086994+caseywhitmire@users.noreply.github.com> Co-authored-by: philan15 <37775368+philan15@users.noreply.github.com> Co-authored-by: Gena <wertixvost@gmail.com>
Fix Adriver ID system test failures (https://app.circleci.com/pipelines/github/prebid/Prebid.js/13137/workflows/5e686ed9-e57e-47b3-9c4e-8beea10be321/jobs/24200). This was already done in prebid#8489, but prebid#8768 undid it.
Type of change
Description of change
Fixed bug with getting buyerid and fixing tests
Maintainer email: m.oranskaya@adriver.ru
Other information
Please add adriver to the generation page https://docs.prebid.org/download.html in the "User ID Modules" section.
Because during the last update, the adriverIdSystem module was added, but it was not added on the generation page