-
Notifications
You must be signed in to change notification settings - Fork 87
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
Fix device UA being empty on first bid request without blocking application #983
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Could you also measure the average, minimum, and maximum time for extracting the user agent string, as well as the number of errors per 1 million attempts from different web view instances?
The current solution does not have a timeout option, so we need to ensure that the value will be fetched reliably.
PrebidMobile/PrebidMobileRendering/Prebid/PBMCore/PBMBidRequester.m
Outdated
Show resolved
Hide resolved
PrebidMobile/PrebidMobileRendering/Utilities/UserAgentService.swift
Outdated
Show resolved
Hide resolved
min - 0.0004010200500488281 s |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
@OlenaPostindustria Thank you so much for tackling this bug! I have merged the PR for fixing the tests so you should be able to rebase and then the tests should pass! |
… in the first Prebid bid request
7428a1d
to
f435bb7
Compare
LGTM |
#976