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

Oxxion Rtd Module: add bid filtering #10100

Merged
merged 6 commits into from
Jun 27, 2023

Conversation

matthieularere-msq
Copy link
Contributor

Type of change

  • Feature

Description of change

Add a bidder filtering functionnality.

@@ -23,14 +23,16 @@ Then add the oxxion Rtd module to your prebid configuration :
pbjs.setConfig(
...
realTimeData: {
auctionDelay: 200,
auctionDelay: 300,
Copy link
Collaborator

Choose a reason for hiding this comment

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

typo on line 15 above

Copy link
Contributor Author

Choose a reason for hiding this comment

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

You meant to add the new use case in the description ?

Copy link
Collaborator

@patmmccann patmmccann Jun 14, 2023

Choose a reason for hiding this comment

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

you write rtbModule instead of rtdModule

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Thanks, I hadn't seen it. It's fixed.

@@ -1,34 +1,71 @@
import { submodule } from '../src/hook.js'
import { deepAccess, logInfo } from '../src/utils.js'
import { deepAccess, logInfo, logError } from '../src/utils.js'
import MD5 from 'crypto-js/md5.js';
Copy link
Collaborator

Choose a reason for hiding this comment

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

why not use something supported already?

Copy link
Collaborator

Choose a reason for hiding this comment

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

like this?

export function cyrb53Hash(str, seed = 0) {

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I need to use a hash lib which is supported in various languages, at least python and golang. However if that's an issue I can remove it as it's not yet something needed in my payload, it's for future use.

Copy link
Collaborator

Choose a reason for hiding this comment

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

We just don't love the include, isnt sha supported in the browser? I forget. md5 is a rather large build side include I believe.

Copy link
Collaborator

Choose a reason for hiding this comment

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

https://github.com/mbaersch/cyrb53-hasher is in python; fairly simple to implement in go just asking chatgpt to translate our js function to go

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I understand the include issue. I've removed the hash string side as it's not needed yet, and when I'll need to implement it back I'll switch to cyrb53

Copy link
Collaborator

Choose a reason for hiding this comment

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

Here's an alternative we use at raptive https://caniuse.com/?search=digest ; it has wide support for sha-1 and sha-256

Copy link
Contributor Author

Choose a reason for hiding this comment

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

just like I said, I removed it for now as it's not needed yet.

@matthieularere-msq matthieularere-msq changed the title Oxxion rtd ova Oxxion Rtd Module: add bid filtering Jun 14, 2023
@patmmccann patmmccann merged commit 6faa2a3 into prebid:master Jun 27, 2023
2 checks passed
github-ishara-chan-tung pushed a commit to ebuzzing/Prebid.js that referenced this pull request Jul 12, 2023
* oxxion Filtering

* fix test while video is activated

* fix init

* add details

* fix typo

* remove md5
musikele pushed a commit to rubicon-project/Prebid.js that referenced this pull request Aug 28, 2023
* oxxion Filtering

* fix test while video is activated

* fix init

* add details

* fix typo

* remove md5
santii7395 pushed a commit to themaven-net/Prebid.js that referenced this pull request Aug 28, 2023
* oxxion Filtering

* fix test while video is activated

* fix init

* add details

* fix typo

* remove md5
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.

None yet

3 participants