Closed
Bug 471839
Opened 16 years ago
Closed 16 years ago
Modify some of the URLs used in bookmarks file
Categories
(Firefox :: General, defect)
Firefox
General
Tracking
()
RESOLVED
FIXED
Firefox 3.1b3
People
(Reporter: reed, Assigned: reed)
References
Details
(Keywords: fixed1.9.1)
Attachments
(1 file)
(deleted),
patch
|
Pike
:
review+
beltzner
:
approval1.9.1+
|
Details | Diff | 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)
Assignee | ||
Updated•16 years ago
|
Status: NEW → ASSIGNED
Comment 1•16 years ago
|
||
(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.
Assignee | ||
Comment 2•16 years ago
|
||
(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 3•16 years ago
|
||
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+
Assignee | ||
Comment 4•16 years ago
|
||
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
Assignee | ||
Updated•16 years ago
|
Attachment #355083 -
Flags: approval1.9.1?
Comment 5•16 years ago
|
||
Comment on attachment 355083 [details] [diff] [review]
patch - v1
a191=beltzner
Attachment #355083 -
Flags: approval1.9.1? → approval1.9.1+
Assignee | ||
Comment 6•16 years ago
|
||
Keywords: fixed1.9.1
You need to log in
before you can comment on or make changes to this bug.
Description
•