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

Add bid size and secure markup validations, controlled by host through configuration #830

Merged
merged 22 commits into from
Dec 17, 2020

Conversation

schernysh
Copy link
Contributor

@schernysh schernysh commented Jul 28, 2020

This PR addresses bids validations as outlined in prebid/prebid-server#1288

@schernysh schernysh requested a review from rpanchyk July 28, 2020 14:19
docs/metrics.md Outdated Show resolved Hide resolved
rpanchyk
rpanchyk previously approved these changes Dec 14, 2020
DGarbar
DGarbar previously approved these changes Dec 14, 2020
# Conflicts:
#	src/main/java/org/prebid/server/spring/config/ServiceConfiguration.java
@rpanchyk rpanchyk dismissed stale reviews from DGarbar and themself via 6db8896 December 17, 2020 13:12
@rpanchyk rpanchyk merged commit e8f6b8f into master Dec 17, 2020
@rpanchyk rpanchyk deleted the bid-attributes-validation branch December 17, 2020 13:16
nickluck9 pushed a commit that referenced this pull request Aug 9, 2021
…h configuration (#830)

* Add bid size and secure markup validations, controlled by host through configuration

* Add account-level bid validations overrides

* Apply account level overrides for bid validations

* Represent sizes list as string

* Update validation error metrics

* Include new configuration properties in documentation

* Fix broken test

* Make use of BidderAliases to correctly resolve aliases on metrics reporting

* Change validation logic for banners: validate max size if enforced by configuration or account

* Change validation logic for secure creative: look for secure markers in addition to verifying there is no insecure markers

* Add warnings support for bids validation

* Fix compilation error after merge

* Update validation metrics

* Fix documentation and remove obsolete class
nickluck9 pushed a commit that referenced this pull request Aug 10, 2021
…h configuration (#830)

* Add bid size and secure markup validations, controlled by host through configuration

* Add account-level bid validations overrides

* Apply account level overrides for bid validations

* Represent sizes list as string

* Update validation error metrics

* Include new configuration properties in documentation

* Fix broken test

* Make use of BidderAliases to correctly resolve aliases on metrics reporting

* Change validation logic for banners: validate max size if enforced by configuration or account

* Change validation logic for secure creative: look for secure markers in addition to verifying there is no insecure markers

* Add warnings support for bids validation

* Fix compilation error after merge

* Update validation metrics

* Fix documentation and remove obsolete class
nickluck9 pushed a commit that referenced this pull request Aug 10, 2021
…h configuration (#830)

* Add bid size and secure markup validations, controlled by host through configuration

* Add account-level bid validations overrides

* Apply account level overrides for bid validations

* Represent sizes list as string

* Update validation error metrics

* Include new configuration properties in documentation

* Fix broken test

* Make use of BidderAliases to correctly resolve aliases on metrics reporting

* Change validation logic for banners: validate max size if enforced by configuration or account

* Change validation logic for secure creative: look for secure markers in addition to verifying there is no insecure markers

* Add warnings support for bids validation

* Fix compilation error after merge

* Update validation metrics

* Fix documentation and remove obsolete class
SerhiiNahornyi pushed a commit that referenced this pull request Nov 10, 2021
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