-
Notifications
You must be signed in to change notification settings - Fork 3.9k
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
✨ The Ozone Project: Add RTC endpoint to callout-vendors.js #26039
Conversation
Thanks for your pull request. It looks like this may be your first contribution to a Google open source project (if not, look below for help). Before we can look at your pull request, you'll need to sign a Contributor License Agreement (CLA). 📝 Please visit https://cla.developers.google.com/ to sign. Once you've signed (or fixed any issues), please reply here with What to do if you already signed the CLAIndividual signers
Corporate signers
ℹ️ Googlers: Go here for more info. |
@googlebot I signed it! |
CLAs look good, thanks! ℹ️ Googlers: Go here for more info. |
@keithwrightbos - are you able to approve this PR and merge it for us please ... if there's any details required to push this through please let us know. |
0902383
to
ac4e9d6
Compare
@rcebulko - do you think you can help us with this error - https://github.com/ampproject/amphtml/pull/26039/checks?check_run_id=398670458 - we've rebased from the master branch but still getting errors. Likewise @keithwrightbos could you help with the owners approval please ? |
Sauce has been a bit flaky of late. I've just started looking into it, hopefully can get it resolved soon 👍 |
hey @keithwrightbos - all the checks are now passing - would you please be able to approve/merge this PR in or provide a timeframe or alternatively @rsimha / @lannka / @gmajoulet could either of you assist with reviewing and approving this PR for me please ? |
Thanks @lannka ! |
This PR adds The Ozone Project RTC callout URL in callout-vendors.js.
NOTE:
Our endpoint will be responding with content only to requests from the domains that are whitelisted in our prebid-server setup.
@mrjoro The Ozone Project agreed to the corporate CLA AFAIK but I am not sure what's the status of that at the moment.
@keithwrightbos Could you please review and let me know if anything else is required.
Thanks,
Greg