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

Ads.Cert Call Sign support #2241

Merged
merged 46 commits into from
Jul 27, 2022
Merged
Show file tree
Hide file tree
Changes from 1 commit
Commits
Show all changes
46 commits
Select commit Hold shift + click to select a range
1312664
Initial preparation for Ads.Cert Call Sign support
May 6, 2022
1722b81
First initial working version
May 10, 2022
384bb22
Added request option and bidder config to enable or disable ads certs
May 23, 2022
a94d723
Code clean up and README file added
May 24, 2022
1547a0b
Code refactoring and clean up
May 25, 2022
7f91db7
Unit tests fix
May 25, 2022
3e6f780
Merge branch 'master' into ads_cert
May 25, 2022
82715a9
Merge fixes
May 25, 2022
9c28dd6
Code refactoring and clean up
May 26, 2022
0bc246d
Added more go docs for newly defines structures
May 26, 2022
aea4f71
Added unit tests
May 26, 2022
9bffd99
Added unit tests
May 26, 2022
b8baa68
Added unit tests
May 27, 2022
ac47bd5
Minor clean up
May 28, 2022
c78b16d
Added remote signer.
May 29, 2022
540c9f8
Removed optional code
Jun 1, 2022
26dac9b
Added tests for NewAdCertsSigner
Jun 1, 2022
de1d8fa
Unit tests for remote signer
Jun 2, 2022
4b9e937
CR fixes part I
Jun 2, 2022
7b6ffe9
CR fixes part II
Jun 3, 2022
8991f4e
Code clean up
Jun 8, 2022
6257437
Merge branch 'master' into ads_cert
Jun 8, 2022
6f87497
Merge fixes
Jun 8, 2022
a091df9
Merge branch 'master' into ads_cert
Jun 22, 2022
1e012b0
renamed "in-process" to "inprocess" to avoid env variables conflict
Jun 23, 2022
8b3a795
Renamed adscert to adsCert
Jun 23, 2022
4462439
Code refactoring
Jun 23, 2022
8b0617b
Code refactoring and signers config validation
Jun 24, 2022
f0ac967
Code refactoring
Jun 24, 2022
bb4c9d9
Code refactoring
Jun 24, 2022
cea7973
Merge branch 'master' into ads_cert
Jun 30, 2022
1636dd3
Resolved merge errors
Jun 30, 2022
2d29975
Merge branch 'master' into ads_cert
Jul 5, 2022
37d2529
Merge with master
Jul 5, 2022
2178fc6
Code review comments
Jul 8, 2022
8709b76
Merge branch 'master' into ads_cert
Jul 11, 2022
72cae9b
Merge conflict fix
Jul 11, 2022
7486973
Moved validation to config.go
Jul 11, 2022
a1177a9
Minor fixes
Jul 13, 2022
d3d50bb
Modified config from experiment.adscert.enabled to experiment.adscert…
Jul 13, 2022
a92f723
Merge branch 'master' into ads_cert
Jul 18, 2022
999e6da
Code review fixes
Jul 19, 2022
d0a9966
Code review fixes
Jul 19, 2022
b9fc196
Code review fixes
Jul 20, 2022
5b81715
Code cleanup
Jul 27, 2022
c3c0fc2
Code cleanup
Jul 27, 2022
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
Renamed adscert to adsCert
  • Loading branch information
vsolovei committed Jun 23, 2022
commit 8b3a795c8dda1eb2ea59db947ded5f8023dac686
4 changes: 2 additions & 2 deletions config/bidderinfo.go
Original file line number Diff line number Diff line change
Expand Up @@ -24,9 +24,9 @@ type BidderInfo struct {
Experiment BidderInfoExperiment `yaml:"experiment"`
}

// BidderInfoExperiment specifies non-production ready features.
// BidderInfoExperiment specifies non-production ready feature config for a bidder
type BidderInfoExperiment struct {
mansinahar marked this conversation as resolved.
Show resolved Hide resolved
AdsCert AdsCert `yaml:"adscert"`
AdsCert AdsCert `yaml:"adsCert"`
}

// AdsCert enables Call Sign feature for bidder
Expand Down
2 changes: 1 addition & 1 deletion config/experiment.go
Original file line number Diff line number Diff line change
Expand Up @@ -5,7 +5,7 @@ type Experiment struct {
AdCerts ExperimentAdsCert `mapstructure:"adscert"`
}

//ExperimentAdCerts configures and enables functionality to generate and send Ads Cert Auth header to bidders
//ExperimentAdsCert configures and enables functionality to generate and send Ads Cert Auth header to bidders
type ExperimentAdsCert struct {
SyntaxNode marked this conversation as resolved.
Show resolved Hide resolved
Enabled bool `mapstructure:"enabled"`
InProcess AdsCertInProcess `mapstructure:"inprocess"`
Expand Down
2 changes: 1 addition & 1 deletion docs/adscertsigner.md
Original file line number Diff line number Diff line change
Expand Up @@ -81,7 +81,7 @@ In Prebid Server configs set parameters for this server:
Workaround for bidders that don't have Call Signs support yet: in configs modify bidder URL to `http://adscertdelivery.com/openrtb2?prebid_disabled=1`. In this case this bidder will not return bids, because this endpoint doesn't exist, but it will imitate support of Call Signs. Bidder parameters still should be valid.

Every bidder by default doesn't support AdsCert. Some bidders cannot handle unsupported headers properly. To enable this feature add next config to {bidder}.yaml file:
`experiment.adscert.enable: true`. With this config bidder will receive `X-Ads-Cert-Auth` header even if this is not the only bidder in request.
`experiment.adsCert.enable: true`. With this config bidder will receive `X-Ads-Cert-Auth` header even if this is not the only bidder in request.

Request extension should have `request.ext.prebid.experiment.adscert.enabled: true`

Expand Down