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

Tracker extension points for non-usersync trackers #1762

Closed
samuelhorwitz opened this issue Oct 24, 2017 · 3 comments
Closed

Tracker extension points for non-usersync trackers #1762

samuelhorwitz opened this issue Oct 24, 2017 · 3 comments
Labels

Comments

@samuelhorwitz
Copy link
Contributor

Type of issue

Feature request

Description

While it is not urgent and we are fine with leaving it out for now, our team at Kargo would like to drop a pixel tracker upon receipt of an ad inside Prebid. However doing so is disallowed due to the fact that it would be chaotic and potentially slow to just allow people to add trackers randomly in their adapter code.

It would be nice if certain events had tracker callback hooks or some other solution where you could register that you would like a tracker to fire that isn't tied to user syncing. For example, inside of response handling.

The issue was brought up here initially #1729 (comment)

@mkendall07
Copy link
Member

@samuelhorwitz
Why not user usersync functionality for this?

@optimatic58
Copy link
Contributor

Not sure if this is the correct place to request this feature (we were directed here). We would simply like to be able to record failure/error logs at our end via ajax call. This would allow us to proactively address any issues that may be causing ad failure. Perhaps deferred ajax calls can be supported as suggested by @dbemiller or a way for us to log and review a sample of those errors to troubleshoot any potential issues.

@mkendall07
Copy link
Member

so the way I look at this is that anything that causes a request to fire that isn't related to fetching demand is unnecessary. So we probably should not have named the functionality "user syncing". It really just means anything that is not a demand fetch. So it's totally appropriate to put other pings into the user syncing queue. Thanks

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

4 participants