1796410 - Metadata is saved for purely redirect urls

Status: NEW

(Firefox for Android :: History, defect)

Description

[Chris Peterson [:cpeterson]]

From github: https://github.com/mozilla-mobile/android-components/issues/10743.

STR:

• type something like reddit.com/r/vancouver into the awesomebar, and go to that link
• refresh the page, perform ^^ a few times

Visiting this url will result in a series of redirects, some of which register as navigation events, triggering us to save metadata for the final url in the redirect chain (A=https://www.reddit.com/r/patientgamers/) and for the intermediate step (B=https://www.reddit.com/r/patientgamers). There are also two more urls in my moz_places (http://reddit.com/r/patientgamers and https://reddit.com/r/patientgamers) which didn't end up in the metadata.

Checking moz_historyvisits, i see that B has only visit_type=5 visits, which is a permanent redirect visit.

The desired outcome, I think, is that we don't save metadata for URLs which are redirecting. We can probably do this at Fenix/A-C level (does content or session state for the tab have anything about visit flags we got from GV? e.g. "this is a redirect").

And we can certainly do this at A-S level (check moz_historyvisits, bail out if only stored visits are redirects... and perhaps if most recent visits were redirects? not very certain how to account of potential variety of visit types here).

┆Issue is synchronized with this Jira Task

Change performed by the Move to Bugzilla add-on.

Comment 1

[BugBot [:suhaib / :marco/ :calixte]]

The severity field is not set for this bug.
:cpeterson, could you have a look please?

For more information, please visit auto_nag documentation.