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

Utiq id module: reduce log spamming #11922

Merged
merged 1 commit into from
Jul 4, 2024
Merged

Conversation

muuki88
Copy link
Collaborator

@muuki88 muuki88 commented Jul 4, 2024

The utiq module uses busy waiting to provide the eid values. Every busy waiting round it logs messages that are not actionable.

Type of change

  • Refactoring (no functional changes, no api changes)

Description of change

The utiq module uses busy waiting to provide the eid values. Every busy waiting round it logs messages that are not actionable.

Other information

FYI @jkthomas

The utiq module uses busy waiting to provide the `eid` values. Every busy waiting round it logs messages that are not actionable.
Copy link

github-actions bot commented Jul 4, 2024

Whoa there, partner! 🌵🤠 We wrangled some duplicated code in your PR:

Reducing code duplication by importing common functions from a library not only makes our code cleaner but also easier to maintain. Please move the common code from both files into a library and import it in each. Keep up the great work! 🚀

@patmmccann patmmccann self-requested a review July 4, 2024 10:43
@patmmccann patmmccann self-assigned this Jul 4, 2024
@patmmccann patmmccann merged commit ed6a92b into master Jul 4, 2024
4 of 5 checks passed
@patmmccann patmmccann deleted the reduce-utiq-log-spam branch July 4, 2024 10:43
@patmmccann patmmccann changed the title Reduce log spamming Utiq id module: reduce log spamming Jul 4, 2024
DecayConstant pushed a commit to mediavine/Prebid.js that referenced this pull request Jul 18, 2024
The utiq module uses busy waiting to provide the `eid` values. Every busy waiting round it logs messages that are not actionable.
mefjush pushed a commit to adhese/Prebid.js that referenced this pull request Jul 19, 2024
The utiq module uses busy waiting to provide the `eid` values. Every busy waiting round it logs messages that are not actionable.
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.

None yet

2 participants