Closed Bug 471839 Opened 16 years ago Closed 16 years ago

Modify some of the URLs used in bookmarks file

Categories

(Firefox :: General, defect)

defect
Not set
normal

Tracking

()

RESOLVED FIXED
Firefox 3.1b3

People

(Reporter: reed, Assigned: reed)

References

Details

(Keywords: fixed1.9.1)

Attachments

(1 file)

Attached patch patch - v1 (deleted) — Splinter Review
Bug 461979 changed some of the URLs used in the bookmarks file, but there's a couple of changes I'd like to change to what it did. * Change https://add-ons.mozilla.com/ to https://addons.mozilla.org/ * Change http://@AB_CD@.fxfeeds.mozilla.com/ to http://fxfeeds.mozilla.com/. Not sure who should review this, so tagging Pike.
Attachment #355083 - Flags: review?(l10n)
Status: NEW → ASSIGNED
(In reply to comment #0) > * Change https://add-ons.mozilla.com/ to https://addons.mozilla.org/ Why? The former was established as the convention for in-UI references, IIRC.
(In reply to comment #1) > (In reply to comment #0) > > * Change https://add-ons.mozilla.com/ to https://addons.mozilla.org/ > > Why? The former was established as the convention for in-UI references, IIRC. Nope, we stopped doing that for AMO a long time ago... Just look at firefox.js and see all the prefs that use https://addons.mozilla.org/ now. Plus, I have bug 454300 open to fix the rest. It doesn't make sense at all to redirect through a DV SSL cert in order to get to an EV cert at AMO. If we're going to get any use out of EV at all, we should go directly to the domain that has the EV cert.
Comment on attachment 355083 [details] [diff] [review] patch - v1 r=me. Note that this doesn't reduce any of the requirements on our server architecture, as we're not changing existing bookmarks. And the links in the existing bookmarks need to continue to work. Changing them would actually make our support requirements on the server side less transparent, and add more entry points to maintain in the future, so I'm not sure if this is a good thing to do.
Attachment #355083 - Flags: review?(l10n) → review+
http://hg.mozilla.org/mozilla-central/rev/45f3d6c184e4 We'll continue to support old URLs, but this means we send users through less round-abouts, which is always a good thing.
Status: ASSIGNED → RESOLVED
Closed: 16 years ago
Resolution: --- → FIXED
Attachment #355083 - Flags: approval1.9.1?
Comment on attachment 355083 [details] [diff] [review] patch - v1 a191=beltzner
Attachment #355083 - Flags: approval1.9.1? → approval1.9.1+
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: