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

Moved bid validation into some adapter middleware. #772

Merged
merged 2 commits into from
Dec 14, 2018

Conversation

dbemiller
Copy link
Contributor

@dbemiller dbemiller commented Dec 13, 2018

This fixes the same issue as #770... but makes some more structural changes that minimize the chances of it happening again later. It's mostly a cut/paste job with some very minor tweaks.

This moves bid validation into a piece of adapter middleware, so that the validation logic isn't interleaved with unrelated metrics logic the way it is today.

In future PRs (though I doubt they'll come from me), you could probably isolate metrics logging, panic recovery, analytics module stuff, into other, similar types of middleware. The more code that moves into small pieces that do one well-defined thing each, the easier a time you'll have finding & fixing bugs like this.

@dbemiller dbemiller merged commit 097d629 into master Dec 14, 2018
@dbemiller dbemiller deleted the bid-validation-middleware branch December 14, 2018 20:41
katsuo5 pushed a commit to flux-dev-team/prebid-server-1 that referenced this pull request Dec 1, 2020
* Moved bidd validation into some adapter middleware.

* Ran gofmt -s to fix the build.
katsuo5 pushed a commit to flux-dev-team/prebid-server-1 that referenced this pull request Dec 2, 2020
* Moved bidd validation into some adapter middleware.

* Ran gofmt -s to fix the build.
katsuo5 pushed a commit to flux-dev-team/prebid-server-1 that referenced this pull request Dec 4, 2020
* Moved bidd validation into some adapter middleware.

* Ran gofmt -s to fix the build.
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