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

AOL Adapter: Perform requests over https by default. #3957

Merged

Conversation

slimkrazy
Copy link
Contributor

Type of change

  • Other

Description of change

Update hostname to use go-forwards domain
Perform requests over HTTPS by default.

  • official adapter submission

slimkrazy added 2 commits July 1, 2019 12:57
Preserve insecure requests only if explicitly specified via the server override parameter (to aid with testing).
@slimkrazy slimkrazy closed this Jul 1, 2019
@slimkrazy
Copy link
Contributor Author

Submitting too soon.

@slimkrazy slimkrazy reopened this Jul 8, 2019
@mike-chowla mike-chowla self-assigned this Jul 9, 2019
@mike-chowla mike-chowla self-requested a review July 9, 2019 16:59
Copy link
Contributor

@mike-chowla mike-chowla left a comment

Choose a reason for hiding this comment

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

LGTM

@mike-chowla mike-chowla merged commit d44fe45 into prebid:master Jul 9, 2019
leonardlabat pushed a commit to criteo-forks/Prebid.js that referenced this pull request Jul 30, 2019
* Perform requests over https by default.
Preserve insecure requests only if explicitly specified via the server override parameter (to aid with testing).

* Slight refactor.
VideoReach pushed a commit to VideoReach/Prebid.js that referenced this pull request Aug 1, 2019
* Perform requests over https by default.
Preserve insecure requests only if explicitly specified via the server override parameter (to aid with testing).

* Slight refactor.
sa1omon pushed a commit to gamoshi/Prebid.js that referenced this pull request Nov 28, 2019
* Perform requests over https by default.
Preserve insecure requests only if explicitly specified via the server override parameter (to aid with testing).

* Slight refactor.
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