Skip to content
This repository has been archived by the owner on Nov 14, 2024. It is now read-only.

Delete distributorId before sending error #87

Merged
merged 11 commits into from
Nov 16, 2022
Merged

Delete distributorId before sending error #87

merged 11 commits into from
Nov 16, 2022

Conversation

leonelcuevas
Copy link
Contributor

@leonelcuevas leonelcuevas commented Nov 9, 2022

CM-631

Delete distributorId before sending error, to prevent infinite loop

Author Todo List:

  • Add/adjust tests (if applicable)
  • Build in CI passes
  • Latest master revision is merged into the branch
  • Self-Review
  • Set Ready For Review status

@leonelcuevas leonelcuevas marked this pull request as ready for review November 10, 2022 16:01
@leonelcuevas leonelcuevas requested a review from a team as a code owner November 10, 2022 16:01
@@ -14,7 +14,7 @@ const plugins = [
strip()
]

const filenames = ['initializer.js', 'standard-live-connect.js', 'minimal-live-connect.js']
Copy link
Contributor

Choose a reason for hiding this comment

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

Where is this used?

Copy link
Contributor

Choose a reason for hiding this comment

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

I rolled back that change, we thought to use the global bus in prebid, but we decided to do that in the initializer

Co-authored-by: Wiem Zine El Abidine <welabidine@liveintent.com>
@leonelcuevas leonelcuevas merged commit 318aa85 into master Nov 16, 2022
@leonelcuevas leonelcuevas deleted the cm-631 branch November 16, 2022 12:32
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants