Update Sovrn bid adapter to send tmax

[ikagotso]

No description provided.

[smounitz]

The way I understand prebid timeout:

Prebid.js has a default timeout of 3000ms with a buffer of 400ms. So if a publisher does not explicitly set the timeout in the config pbjs.setConfig({ bidderTimeout: 3000})) the bidders will have 2600ms to respond.

Can we please check this, and also add a test asserting that a default tmax is passed even when the publisher does not set a timeout

[bharris-sovrn]

We should add merge protections to this repo.

[ikagotso]

The way I understand prebid timeout:

Prebid.js has a default timeout of 3000ms with a buffer of 400ms. So if a publisher does not explicitly set the timeout in the config pbjs.setConfig({ bidderTimeout: 3000})) the bidders will have 2600ms to respond.

Can we please check this, and also add a test asserting that a default tmax is passed even when the publisher does not set a timeout

According to this page: https://docs.prebid.org/features/timeouts.html

you are talking about Auction Timeout (Default 3000MS) and Timeout Buffer (Default 400MS).

Shouldn't we be taking into consideration Prebid Server s2sConfig Timeout ?

[smounitz]

Irakli and I looked into the timeout buffer and it turns out that it gets added to the timeout.
We also tested that if the publisher doesn't set a timeout the default of 3000ms is passed.

So everything is good to go

[ikagotso]

Not merging because similar PR is on original Prebid.js repo

[dmitrybor]

Merged into the original Prebid.js repo and synced back to master