On 12/11/2011 7:53 PM, dsimcha wrote: > Should the protocol detection be case-insensitive, i.e. "ftp://" == > "FTP://"? Oh, one more thing: Factor the protocol detection out into a function. You have the same expression cut and pasted everywhere: if(url.startsWith("ftp://") || url.startsWith("ftps://") ...