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

Update Sharethrough bid adapter endpoint #4578

Merged

Conversation

madma
Copy link
Contributor

@madma madma commented Dec 11, 2019

Type of change

  • Refactoring (no functional changes, no api changes)

Description of change

  • Sharethrough bid adapter endpoint has been updated to always use https
  • Protocol of the referrer is now sent as an additional parameter in bid requests

Copy link
Collaborator

@harpere harpere left a comment

Choose a reason for hiding this comment

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

there are merge conflicts

Copy link
Collaborator

@harpere harpere left a comment

Choose a reason for hiding this comment

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

merge conflicts

@madma
Copy link
Contributor Author

madma commented Dec 12, 2019

conflicts resolved @harpere -- thanks for reviewing!

@madma madma requested a review from harpere December 12, 2019 17:54
@madma
Copy link
Contributor Author

madma commented Dec 18, 2019

hi, just following up on this @harpere. thanks!

@harpere harpere merged commit 4c9bf6f into prebid:master Dec 18, 2019
sa1omon pushed a commit to CleanMediaNet/Prebid.js that referenced this pull request Dec 19, 2019
* Send parameter indicating the protocol of the requesting page

- `secure` parameter will equal `true` if the requesting page uses `https` and `false` otherwise

Story: [#169736764](https://www.pivotaltracker.com/story/show/169736764)

* Add fallback logic to catch protocol of iframe when prebid does not reach window.top

* Wrap location.protocol for testing; Add specs

* Use only https butler endpoints in tests
@madma madma deleted the md/169736764/https-endpoint-and-param branch December 23, 2019 17:04
tadam75 pushed a commit to smartadserver/Prebid.js that referenced this pull request Jan 9, 2020
* Send parameter indicating the protocol of the requesting page

- `secure` parameter will equal `true` if the requesting page uses `https` and `false` otherwise

Story: [#169736764](https://www.pivotaltracker.com/story/show/169736764)

* Add fallback logic to catch protocol of iframe when prebid does not reach window.top

* Wrap location.protocol for testing; Add specs

* Use only https butler endpoints in tests
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.

2 participants