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

[Sharethrough] Improved Headers in ad server request #958

Merged
merged 2 commits into from
Jul 17, 2019
Merged

[Sharethrough] Improved Headers in ad server request #958

merged 2 commits into from
Jul 17, 2019

Conversation

maphe
Copy link
Contributor

@maphe maphe commented Jul 10, 2019

This is forwarding more data to the Sharethrough ad server in order to bid more accurately:

  • add scheme to Origin header
  • add X-Forwarded-For header
  • add User-Agent header

mansinahar
mansinahar previously approved these changes Jul 10, 2019
@mansinahar
Copy link
Contributor

@maphe Just FYI, looks like you force pushed on your branch and now this PR doesn't have any files changed but still has merge conflicts.. lol. Not sure how that happened.

@maphe
Copy link
Contributor Author

maphe commented Jul 16, 2019

@mansinahar thanks for the heads up, I'm investigating internally to figure out what happened, I'll restore the PR as it should be as quickly as possible

…arethrough/prebid-server into sharethrough_master
@maphe
Copy link
Contributor Author

maphe commented Jul 16, 2019

@mansinahar we're back in good shape, there was some tool that got tested internally that messed up the branch. The PR is back to normal.

@mansinahar mansinahar merged commit 8c13c5c into prebid:master Jul 17, 2019
katsuo5 pushed a commit to flux-dev-team/prebid-server-1 that referenced this pull request Dec 1, 2020
katsuo5 pushed a commit to flux-dev-team/prebid-server-1 that referenced this pull request Dec 2, 2020
katsuo5 pushed a commit to flux-dev-team/prebid-server-1 that referenced this pull request Dec 4, 2020
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.

3 participants