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

[Brave News]: More debug logging #37923

Closed
fallaciousreasoning opened this issue Apr 29, 2024 · 4 comments · Fixed by brave/brave-core#23311
Closed

[Brave News]: More debug logging #37923

fallaciousreasoning opened this issue Apr 29, 2024 · 4 comments · Fixed by brave/brave-core#23311

Comments

@fallaciousreasoning
Copy link

@stephendonner is helping me debug an issue on Windows where there seems to be a publisher parsing error.

We should:

  1. Log the request url
  2. Log the response code
  3. Log the failed response content
@stephendonner
Copy link

stephendonner commented Apr 30, 2024

C:\Program Files\BraveSoftware\Brave-Browser-Nightly\Application> .\brave.exe --enable-logging=stderr --vmodule=*/brave_news/*=1

PS C:\Program Files\BraveSoftware\Brave-Browser-Nightly\Application> [6676:2160:0430/101650.224:ERROR:chrome_browser_cloud_management_controller.cc(161)] Cloud management controller initialization aborted as CBCM is not enabled. Please use the `--enable-chrome-browser-cloud-management` command line flag to enable it if you are not using the official Google Chrome build.
[6676:2160:0430/101650.261:INFO:notification_helper_impl_win.cc(176)] Failed to get notification settings from toast notifier
[6676:2160:0430/101650.268:WARNING:browser_management_service.cc(128)] EnterpriseLogoUrl fetch failed with error code -1 and MIME type
[6676:2160:0430/101650.268:WARNING:external_provider_impl.cc(514)] Malformed extension dictionary for extension: odbfpeeihdkbihmopkbjmoonfanlbfcl.  Key external_update_url has value "", which is not a valid URL.
[6676:2160:0430/101650.269:VERBOSE1:brave_news_p3a.cc(89)] Not recording direct feed total (not monthly active user)
[6676:2160:0430/101650.269:VERBOSE1:brave_news_p3a.cc(142)] NewsP3A: total actions update: total = 0 count delta = 0 action enum = 0
[6676:2160:0430/101650.269:VERBOSE1:brave_news_p3a.cc(170)] NewsP3A: card depth update: new_visit_card_depth = 0
[6676:2160:0430/101650.269:VERBOSE1:brave_news_controller.cc(791)] STOPPING TIMERS
[6676:2160:0430/101650.269:VERBOSE1:brave_news_controller.cc(795)] REMOVING DATA FROM MEMORY
[6676:2160:0430/101650.660:WARNING:CONSOLE(5)] "crbug/1173575, non-JS module files deprecated.", source: chrome://resources/js/load_time_data_deprecated.js (5)
[6676:2160:0430/101652.742:WARNING:CONSOLE(5)] "crbug/1173575, non-JS module files deprecated.", source: chrome://resources/js/load_time_data_deprecated.js (5)
[6676:2160:0430/101652.809:ERROR:brave_new_tab_page_handler.cc(519)] image data is not valid
[6676:2160:0430/101655.711:VERBOSE1:publishers_controller.cc(221)] Fetching publishers from https://brave-today-cdn.brave.com/sources.global.json
[6676:2160:0430/101655.712:VERBOSE1:brave_news_controller.cc(746)] STARTING TIMERS
[6676:2160:0430/101655.828:VERBOSE1:publishers_controller.cc(227)] Publishers response status code: -1
[6676:2160:0430/101655.828:ERROR:publishers_parsing.cc(26)] Publisher data expected to be a list: null

[6676:2160:0430/101755.718:VERBOSE1:brave_news_controller.cc(730)] PREFETCHING: ensuring feed has been retrieved
[6676:2160:0430/101755.718:VERBOSE1:feed_fetcher.cc(117)] FetchFeed
[6676:2160:0430/101755.719:VERBOSE1:publishers_controller.cc(221)] Fetching publishers from https://brave-today-cdn.brave.com/sources.global.json
[6676:2160:0430/101755.875:VERBOSE1:publishers_controller.cc(227)] Publishers response status code: -1
[6676:2160:0430/101755.875:ERROR:publishers_parsing.cc(26)] Publisher data expected to be a list: null

[6676:2160:0430/101755.875:ERROR:feed_fetcher.cc(130)] Brave News Publisher list was empty
[6676:2160:0430/101755.876:VERBOSE1:publishers_controller.cc(221)] Fetching publishers from https://brave-today-cdn.brave.com/sources.global.json
[6676:2160:0430/101756.056:VERBOSE1:publishers_controller.cc(227)] Publishers response status code: 200
[6676:2160:0430/101756.065:VERBOSE1:publishers_controller.cc(248)] Notify subscribers to publishers data

@stephendonner
Copy link

image

@fallaciousreasoning
Copy link
Author

Huh, that's super weird - I wonder what status code -1 is?

@fallaciousreasoning
Copy link
Author

Okay, I'm adding another log because I saw there was a separate error code

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

Successfully merging a pull request may close this issue.

3 participants