We should have a script that compares the list of redirects as shown on http://wiki.mozilla.org/Firefox2/L10n_Feed_Redirects with the actual redirects in place. Taking. I'm not sure if Core/Testing is the best way to communicate the progress on stuff in testing/tests/l10n. I guess either this or we'd create a Mozilla Localizations/Testing component.
Created attachment 238411 [details] verify-rss-feeds, take 1 This script grabs the data from http://wiki.mozilla.org/Firefox2/L10n_Feed_Redirects in raw form, optional with --oldid=oldid, a wiki version number, and then, for all locales given in a file which can be either the format of shipped-locales or all-locales, takes our standard form of the http feed URL and follows the 302's it gets until it leaves mozilla.com servers. Then it compares that to either the given url for that locale on the wiki, or to the en-US default feed url. -v makes output more verbose, the more verbose the more -v you enter. IT should be able to use this script to validate the redirects. Script is intended to go to testing/tests/l10n/verify-rss-feeds.
Attachment #238411 - Flags: review?
Comment on attachment 238411 [details] verify-rss-feeds, take 1 Did I say that I hate bugzilla for not completing reviewers? Rob, if you can't get to it, just cancel the request?
Attachment #238411 - Flags: review? → review?(rcampbell)
Hey Axel, I looked it over and it looks good (tiny grammatical error on line 91 looses->loses). Ran it and every locale FAILed. Assuming that's expected today, then it looks good-to-go.
Comment on attachment 238411 [details] verify-rss-feeds, take 1 Setting the r flag as mentioned by robcee, no idea why bugzilla didn't want him to do that.
Attachment #238411 - Flags: review?(rcampbell) → review+
Checked in with robcee's nit, marking FIXED. Thanks for the quick turnaround.
Status: NEW → RESOLVED
Last Resolved: 12 years ago
Resolution: --- → FIXED
Component: Testing → Infrastructure
Product: Core → Mozilla Localizations
QA Contact: testing → infrastructure
Version: Trunk → unspecified
You need to log in before you can comment on or make changes to this bug.