782085 - Need event and progress notifications for FTP

Status: RESOLVED WONTFIX

(Core Graveyard :: Networking: FTP, defect)

Description

[Jason Duell]

We need to add an "ftp-on-modify-request" nsIObserverService notification to FTP so we can block the connection if we need to for various policy reasons (see bug 780087).  Presumably we can copy the logic from the HTTP version of the code.

Comment 1

[Jason Duell]

We probably also want ftp-on-examine-response and ftp-on-examine-cached-response, like HTTP has.

We're also going to need a way to monitor how much traffic an FTP connection is using.  That's probably a matter of implementing an nsIHttpActivityObserver and/or nsIProgressEventSink API for ftp, like we have for HTTP.

Comment 2

[Patrick McManus [:mcmanus]]

given the essentially deprecated status of ftp, I'd like to keep this to the absolute minimum set necessary. Maybe ftp-on-modify request would be enough?

Comment 3

[Patrick McManus [:mcmanus]]

We are in a period where ftp is clearly deprecated and in general, making changes to the code is riskier than letting it ride unless there is a patch and reviewer available to make a good judgment about it. So I'm going to wontfix ftp bugs related to enhancements, interop errors, etc.. We will be better off putting our energy into including a different js based ftp stack.