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 only download the page's linked alternate feeds when UI list is opened #33246

Closed
petemill opened this issue Sep 25, 2023 · 6 comments · Fixed by brave/brave-core#20287

Comments

@petemill
Copy link
Member

When visiting a site and the site advertises RSS feeds (via <link> elements), Brave downloads the content of those URLs in order to determine feed type, validity and the feed name.

Instead, we should use the existance of those elements to determine whether to show the url bar icon:
image

And only when that icon is clicked, should we fetch the feeds to get the necessary information (if needed).
We can show a loading indicator in the UI whilst the information is being fetched.

@LaurenWags
Copy link
Member

@fallaciousreasoning could we get a test plan for this one? Thanks!

@fallaciousreasoning
Copy link

Hi @LaurenWags, this one probably needs something like Fiddler too, unfortunately:

  1. Setup Fiddler
  2. Navigate to https://hacks.mozilla.org/ with Brave News enabled
  3. The feed should not be fetched yet in Fiddler (the feed url is https://hacks.mozilla.org/feed/)
  4. Click the Brave News button in the LocationBar
  5. The feed should be fetched https://hacks.mozilla.org/feed/ and the title of the Feed should populate

@LaurenWags
Copy link
Member

thanks @fallaciousreasoning! we can give that a try and report back if we have problems 👍🏻

@fallaciousreasoning
Copy link

Let me know if it's not working / if we need to work out an easier way to test 😄

@stephendonner
Copy link

Verified PASSED using

Brave | 1.60.94 Chromium: 118.0.5993.70 (Official Build) beta (64-bit)
-- | --
Revision | 6a9dda6d700daa1da53ec97072458dd271ba5c01
OS | Windows 10 Version 22H2 (Build 19045.3570)

Steps:

  1. launched Fiddler Everywhere
  2. installed 1.60.94
  3. launched Brave
  4. loaded hacks.mozilla.org
  5. confirmed webpage (HTML, CSS, JS)-associated traffic in Fiddler Everywhere
  6. clicked on the Manage Subscriptions RSS-feed icon in the URL bar
  7. confirmed https://hacks.mozilla.org/feed traffic in Fiddler Everywhere
traffic before traffic after
image image (1)

@hffvld hffvld added the QA/In-Progress Indicates that QA is currently in progress for that particular issue label Oct 24, 2023
@hffvld
Copy link
Contributor

hffvld commented Oct 24, 2023

Verified on Pixel 7 using version(s):

Device/OS: Pixel 7 [panther_beta-user 14 U1B2.230922.010 release-keys]
Brave build: 1.60.96 
Chromium: 118.0.5993.88 (Official Build) beta (64-bit) 
Revision: b4c059ad1d32d6e0cd1466123f03ccb97b3cf858

STEPS:

  1. Fresh install and launch Brave
  2. Enalbe Brave News
  3. Navigate to https://hacks.mozilla.org/ > Verify via Fiddler/Charles Proxy that the feed has not been fetched yet (the feed URL is https://hacks.mozilla.org/feed/)
  4. Thre-dot menu > Enable Desktop site
  5. Tap the Feed button on the top right side > Verify via Fiddler/Charles Proxy that the feed is fetched now (the feed URL is https://hacks.mozilla.org/feed/)

ACTUAL RESULTS:


Reproduced the issue with older build 1.59.120

2023-10-23_19-13-20.mp4

VERIFIED in the newer build 1.60.96

2023-10-23_19-17-17.mp4

@hffvld hffvld added QA Pass - Android ARM and removed QA/In-Progress Indicates that QA is currently in progress for that particular issue labels Oct 24, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment