Status
People
(Reporter: john_roshi, Assigned: Callek)
Tracking
Details
Attachments
(1 attachment)
|
916.25 KB,
patch
|
InvisibleSmiley
:
review+
|
Details | Diff | Splinter Review |
User Agent: Mozilla/5.0 (X11; Ubuntu; Linux x86_64; rv:39.0) Gecko/20100101 Firefox/39.0 Build ID: 20150806191835 Steps to reproduce: From the Seamonkey Releases page http://www.seamonkey-project.org/releases/#contrib I clicked on the link "Linux/x86_64 .tar.bz2" (Under "Contributed build (other platforms)) The actual URL is: ftp://ftp.mozilla.org/pub/mozilla.org/seamonkey/releases/2.33.1/contrib/seamonkey-2.33.1.en-US.linux-x86_64.tar.bz2 Note: The web page version says "April 15, 2015". Actual results: I get a javascript alert "550 Permission denied." Expected results: A file save dialog should have appeared.
|
(Assignee) | |
Comment 1•3 years ago
|
||
Thank you for the report, we should switch the url to not use the ftp protocol....
Flags: needinfo?(bugspam.Callek)
|
|
(Assignee) | |
Comment 2•3 years ago
|
||
WORKAROUND: Use https://ftp.mozilla.org/pub/mozilla.org/seamonkey/releases/2.33.1/contrib/seamonkey-2.33.1.en-US.linux-x86_64.tar.bz2
|
||
Comment 3•3 years ago
|
||
> The actual URL is: > ftp://ftp.mozilla.org/pub/mozilla.org/seamonkey/releases/2.33.1/contrib/seamonkey-2.33.1.en-US.linux-x86_64.tar.bz2 All ftp://ftp.mozilla.org urls should be rewritten as https://
|
|
||
Updated•3 years ago
|
||
Status: UNCONFIRMED → NEW
Ever confirmed: true
|
||
Comment 4•3 years ago
|
||
The problem is NOT that someone was still trying to use FTP. The problem is that the Web page for "SeaMonkey: Download & Releases" at <http://www.seamonkey-project.org/releases/#contrib> still has FTP links instead of HTTP or HTTPS links.
|
|
(Assignee) | |
Comment 5•3 years ago
|
||
Created attachment 8648346 [details] [diff] [review] [seamonkey-project] v1 Ratty, Jens, If either of you can get to this soon that would be great (I'll take just one of your reviews) David, I don't want to land this until *someone* looks it over, and while I am waiting on Ratty/Jens for a proper review, if you flag this as "feedback+" based on a skim of what I changed I'm happy to land. Most of this patch was bulk-find/replace and not too much manual anything. $ hg diff --stat lib/config.tmpl | 6 +- lib/release.tmpl | 4 +- src/releases/1.0.1.en.html | 186 +++++++++++++++++++++++++++++++------------------------------- src/releases/1.0.2.en.html | 182 ++++++++++++++++++++++++++++++------------------------------ src/releases/1.0.3.en.html | 182 ++++++++++++++++++++++++++++++------------------------------ src/releases/1.0.4.en.html | 144 ++++++++++++++++++++++++------------------------ src/releases/1.0.5.en.html | 152 +++++++++++++++++++++++++------------------------- src/releases/1.0.6.en.html | 184 +++++++++++++++++++++++++++++++------------------------------- src/releases/1.0.7.en.html | 174 +++++++++++++++++++++++++++++----------------------------- src/releases/1.0.8.en.html | 174 +++++++++++++++++++++++++++++----------------------------- src/releases/1.0.9.en.html | 184 +++++++++++++++++++++++++++++++------------------------------- src/releases/1.0.en.html | 170 ++++++++++++++++++++++++++++---------------------------- src/releases/1.0a.en.html | 76 ++++++++++++------------ src/releases/1.0b.en.html | 62 ++++++++++---------- src/releases/1.1.1.en.html | 154 ++++++++++++++++++++++++++-------------------------- src/releases/1.1.10.en.html | 140 +++++++++++++++++++++++----------------------- src/releases/1.1.11.en.html | 164 +++++++++++++++++++++++++++--------------------------- src/releases/1.1.12.en.html | 176 +++++++++++++++++++++++++++++----------------------------- src/releases/1.1.13.en.html | 164 +++++++++++++++++++++++++++--------------------------- src/releases/1.1.14.en.html | 184 +++++++++++++++++++++++++++++++------------------------------- src/releases/1.1.15.en.html | 172 +++++++++++++++++++++++++++++----------------------------- src/releases/1.1.16.en.html | 174 +++++++++++++++++++++++++++++----------------------------- src/releases/1.1.17.en.html | 2 +- src/releases/1.1.18.en.html | 2 +- src/releases/1.1.19.en.html | 2 +- src/releases/1.1.2.en.html | 156 ++++++++++++++++++++++++++-------------------------- src/releases/1.1.3.en.html | 154 ++++++++++++++++++++++++++-------------------------- src/releases/1.1.4.en.html | 152 +++++++++++++++++++++++++------------------------- src/releases/1.1.5.en.html | 144 ++++++++++++++++++++++++------------------------ src/releases/1.1.6.en.html | 146 ++++++++++++++++++++++++------------------------ src/releases/1.1.7.en.html | 166 ++++++++++++++++++++++++++++---------------------------- src/releases/1.1.8.en.html | 164 +++++++++++++++++++++++++++--------------------------- src/releases/1.1.9.en.html | 168 ++++++++++++++++++++++++++++---------------------------- src/releases/1.1.en.html | 142 ++++++++++++++++++++++++------------------------ src/releases/1.1a.en.html | 68 +++++++++++----------- src/releases/1.1b.en.html | 172 +++++++++++++++++++++++++++++----------------------------- src/releases/2.1b3.en.html | 40 ++++++------ src/releases/seamonkey1.0.1/installation-extras.en.html | 2 +- src/releases/seamonkey1.0.2/installation-extras.en.html | 2 +- src/releases/seamonkey1.0.3/installation-extras.en.html | 2 +- src/releases/seamonkey1.0.4/installation-extras.en.html | 2 +- src/releases/seamonkey1.0.5/installation-extras.en.html | 2 +- src/releases/seamonkey1.0.6/installation-extras.en.html | 2 +- src/releases/seamonkey1.0.7/installation-extras.en.html | 2 +- src/releases/seamonkey1.0.8/installation-extras.en.html | 2 +- src/releases/seamonkey1.0.9/installation-extras.en.html | 2 +- src/releases/seamonkey1.0/installation-extras.en.html | 2 +- src/releases/seamonkey1.0a/installation-extras.en.html | 2 +- src/releases/seamonkey1.0b/installation-extras.en.html | 2 +- src/releases/seamonkey1.1.1/installation-extras.en.html | 2 +- src/releases/seamonkey1.1.10/installation-extras.en.html | 2 +- src/releases/seamonkey1.1.11/installation-extras.en.html | 2 +- src/releases/seamonkey1.1.12/installation-extras.en.html | 2 +- src/releases/seamonkey1.1.13/installation-extras.en.html | 2 +- src/releases/seamonkey1.1.14/get-started.html | 4 +- src/releases/seamonkey1.1.14/installation-extras.en.html | 2 +- src/releases/seamonkey1.1.15/get-started.html | 4 +- src/releases/seamonkey1.1.15/installation-extras.en.html | 2 +- src/releases/seamonkey1.1.16/get-started.html | 4 +- src/releases/seamonkey1.1.16/installation-extras.en.html | 2 +- src/releases/seamonkey1.1.17/get-started.html | 4 +- src/releases/seamonkey1.1.17/installation-extras.en.html | 2 +- src/releases/seamonkey1.1.18/get-started.html | 4 +- src/releases/seamonkey1.1.18/installation-extras.en.html | 2 +- src/releases/seamonkey1.1.19/get-started.html | 4 +- src/releases/seamonkey1.1.19/installation-extras.en.html | 2 +- src/releases/seamonkey1.1.2/installation-extras.en.html | 2 +- src/releases/seamonkey1.1.3/installation-extras.en.html | 2 +- src/releases/seamonkey1.1.4/installation-extras.en.html | 2 +- src/releases/seamonkey1.1.5/installation-extras.en.html | 2 +- src/releases/seamonkey1.1.6/installation-extras.en.html | 2 +- src/releases/seamonkey1.1.7/installation-extras.en.html | 2 +- src/releases/seamonkey1.1.8/installation-extras.en.html | 2 +- src/releases/seamonkey1.1.9/installation-extras.en.html | 2 +- src/releases/seamonkey1.1/installation-extras.en.html | 2 +- src/releases/seamonkey1.1a/installation-extras.en.html | 2 +- src/releases/seamonkey1.1b/installation-extras.en.html | 2 +- 77 files changed, 2489 insertions(+), 2489 deletions(-)
Flags: needinfo?(bugspam.Callek)
Attachment #8648346 -
Flags: review?(philip.chee)
Attachment #8648346 -
Flags: review?(jh)
Attachment #8648346 -
Flags: feedback?(david)
|
|
(Assignee) | |
Updated•3 years ago
|
||
Assignee: nobody → bugspam.Callek
|
||
Comment 6•3 years ago
|
||
Comment on attachment 8648346 [details] [diff] [review] [seamonkey-project] v1 Admittedly I only checked about one instance of each link but looks good so r+. Thanks! Ideally we'd make much more use of template variables here but I fully understand you didn't want to go that far with that S&R patch here.
Attachment #8648346 -
Flags: review?(jh) → review+
|
|
||
Comment 7•3 years ago
|
||
Comment on attachment 8648346 [details] [diff] [review] [seamonkey-project] v1 I'll defer to Jens. or rs=me if you need it.
Attachment #8648346 -
Flags: review?(philip.chee)
|
|
(Assignee) | |
Comment 8•3 years ago
|
||
https://hg.mozilla.org/SeaMonkey/seamonkey-project-org/rev/ebbb573521c3
Status: NEW → RESOLVED
Last Resolved: 3 years ago
Resolution: --- → FIXED
|
|
(Assignee) | |
Updated•3 years ago
|
||
Attachment #8648346 -
Flags: feedback?(david)
You need to log in
before you can comment on or make changes to this bug.
Description
•