Camino should not set the default FTP client.

RESOLVED FIXED in Camino1.0

Status

--
minor
RESOLVED FIXED
14 years ago
14 years ago

People

(Reporter: moz, Assigned: moz)

Tracking

({fixed1.8})

unspecified
Camino1.0
PowerPC
macOS
fixed1.8
Bug Flags:
camino1.0 +

Details

(URL)

(Assignee)

Description

14 years ago
Whenever a browser is selected from the "Default Web Browser" popup, it sets the
browser as the default FTP client as well. It shouldn't do this. (I don't know
that we should be setting the default Gopher client, either, but who uses Gopher
nowadays? ;) )

The line that does this is:

_LSSetDefaultSchemeHandlerURL(@"ftp", browserURL);

In NSWorkspace+Utils.mm. (Currently line 100:
http://lxr.mozilla.org/mozilla/source/camino/src/extensions/NSWorkspace+Utils.m#100
.)
worth discussion, i'll put on the 10 radar.
Target Milestone: --- → Camino1.0

Comment 2

14 years ago
I concur
Also, Camino doesn't repect the FTP pref, either, if it's set.  I have NetFinder
set as my FTP client and Camino blithely ignores it and handles FTP itself. 
This is not always a bad thing (say, for ADC downloads where the URL keeps
requiring a password), but I don't think Camino should be doing it.  I can open
another bug on this if needed.

Comment 4

14 years ago
Simon, I join your concurring opinion.  I don't care either way about gopher.

Regarding comment 3, I'd rather not pass things to the ftp handler unless
Safari-like behavior was implemented: if an ftp URL doesn't end in a slash,
attempt an FTP download, retrieve if a file, and pass to the ftp handler if a
directory.  Usernames and passwords, when needed, can be passed in the URL. 
That's another bug.

or maybe:

If the Finder is the ftp handler, handle ftp internally, otherwise, pass it to
the ftp handler.  (It's still another bug.)
Flags: camino1.0+

Updated

14 years ago
Assignee: mikepinkerton → mark

Updated

14 years ago
Assignee: mark → mozilla

Comment 6

14 years ago
I checked in Wevah's suggestion to remove the offending line from NSWorkspace+Utils.mm (comment 0), r=me.  Trunk and branch.
Status: NEW → RESOLVED
Last Resolved: 14 years ago
Keywords: fixed1.8
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.