Skip to content

Validate url domain and path - #4522

Merged
Net-burst merged 3 commits into
masterfrom
validate-url-domain-and-path
May 29, 2026
Merged

Validate url domain and path#4522
Net-burst merged 3 commits into
masterfrom
validate-url-domain-and-path

Conversation

@Lightwood13

@Lightwood13 Lightwood13 commented May 29, 2026

Copy link
Copy Markdown
Collaborator

🔧 Type of changes

  • bugfix

✨ What's the context?

prebid/prebid-server#4802

Bidders with host validation added:

  • acuityads
  • adhese
  • bidmachine
  • kayzen
  • rediads
  • relevantdigital
  • silvermob
  • smartyads
  • tradplus
  • trafficgate
  • yeahmobi
  • zeroclickfraud

Bidders with path segment validation added:

  • adtonos
  • adview
  • axonix
  • bidmachine
  • blis
  • clydo (uses enum for host part)
  • contxtful
  • elementaltv
  • eplanning
  • gamoshi
  • imds
  • kueezrtb
  • madvertise
  • onetag
  • operaads
  • ownadx
  • roulax
  • silvermob
  • smilewanted
  • tappx
  • thetradedesk
  • tradplus
  • ucfunnel
  • vidazoo
  • yieldlab

Bidders without changes, because they use enum in json schema or validate in the bidder code that value can only take enum values:

  • adot
  • algorix
  • between
  • taboola

Bidders without changes, because they replace macro with a number:

  • improvedigital
  • intertech
  • invibes (static string + number)
  • limelightDigital
  • yandex

@Net-burst Net-burst left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

LGTM

@Net-burst
Net-burst merged commit d0c723c into master May 29, 2026
8 checks passed
@Net-burst
Net-burst deleted the validate-url-domain-and-path branch May 29, 2026 18:37
CTMBNara added a commit that referenced this pull request Jun 30, 2026
CTMBNara added a commit that referenced this pull request Jul 12, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants