266173 - obsoled URLs still provide content and doesn't redirect users (Warning: Invalid argument supplied for foreach())

Status: RESOLVED WONTFIX

(addons.mozilla.org Graveyard :: Public Pages, defect)

Description

[Adam Hauner]

Link found in Google, page is displaying warning, some information about
extension doesn't appear.

Warning: Invalid argument supplied for foreach() in
/opt/update/extensions/moreinfo.php on line 139

Comment 1

[Wolf [:wolf]]

The correct URL is,
http://update.mozilla.org/extensions/moreinfo.php?id=60&vid=645

This relates to Bug 256485, that the vid parameter needs to go away to have more
portable URLs.

Though this isn't a bug though, that URL is outdated, as there's not a vid=63
anymore.

Comment 2

[Adam Hauner]

Wolf, so redirect users to correct URL with 301 Moved Pernamently. You provided
URLs with "vid" in history, Google remember it, other SE remember it many users
link it from their pages, so you have to deal with it and provide backward
compability solution. Now, Mozilla Update still provide content (wrong content!)
on these URLs with 200 OK status.

Futhermore, I recommend redirecting Warning and Notices to log instead of
showing them and analyze them periodicaly.
-> REOPENing

Comment 3

[Wolf [:wolf]]

The fix for this is to fix Bug 256485. :-)

Mozilla Update is *beta* URLs are not guaranteed. Backwards compatibility is not
guaranteed. So adding redirects for those URLs isn't going to happen.  Once that
bug is fixed, the vid will be simply ignored if it's invalid :-) and fall back
to the default state of the page, and pick the right version record.

This is either wontfix or duped to 256485.

Comment 4

[Adam Hauner]

Not guarantee published URLs does mean nobody wanna link you (and naming it beta
doen't change anything on this fact). Ignoring "vid" parameter will just create
a lot of different URLs with the same content, what is another really bad thing.