-
Notifications
You must be signed in to change notification settings - Fork 2.4k
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
Brave Ads crash when updating catalog issuers #12031
Comments
@jsecretan this issue occurs if a mismatched version of the catalog is downloaded which is likely only going to occur if the catalog is rewritten using a proxy |
Verification passed on
Verified test plan from brave/brave-core#6816
Verified no crash
Verified retry is scheduled:
Verified that after turning off rewrite, the catalog was parsed properly:
Verified passed with
Verified test plan from brave/brave-core#6816
No crash observed or listed in brave://crashes Verified unable to parse the catalog
Verified retry is scheduled:
Verified that after turning off rewrite, the catalog was parsed properly:
Confirmed able to view an ad once catalog was parsed properly. |
https://brave.sp.backtrace.io/p/brave/debug?time=all&filters=(_deleted%3D0%2C(callstack%2Cregex%2C%22(BatAds%7CAdsService%7CAdsImpl%7CConfirmationsImpl)%22)%2C(ver%2Cregex%2C%22(86.1%7C85.1%7C84.1%7C83.1)%22))&fingerprint=0000000000000000000000000000000000000000000000000000000000000202
[00] ads::ConfirmationsState::ToJson
[01] ads::Confirmations::Save
[02] ads::Confirmations::SetCatalogIssuers
[03] ads::GetCatalog::OnDownloaded
[04] bat_ads::OnUrlRequest(std::__1::function<void
[05] bat_ads::mojom::BatAdsClient_UrlRequest_ForwardToCallback::Accept
[06] mojo::InterfaceEndpointClient::HandleIncomingMessageThunk::Accept
[07] mojo::internal::MultiplexRouter::ProcessIncomingMessage
[08] mojo::internal::MultiplexRouter::Accept
[09] mojo::Connector::ReadAllAvailableMessages
[10] mojo::SimpleWatcher::OnHandleReady
[11] base::TaskAnnotator::RunTask
[12] base::sequence_manager::internal::ThreadControllerWithMessagePumpImpl::DoWork
[13] base::MessagePumpDefault::Run
[14] base::sequence_manager::internal::ThreadControllerWithMessagePumpImpl::Run
[15] base::RunLoop::Run
[16] content::UtilityMain
[17] content::ContentMainRunnerImpl::Run
[18] service_manager::Main
[19] content::ContentMain
[20] ChromeMain
The text was updated successfully, but these errors were encountered: