Unified Downloader and Lightcurve factory

why does filename have a default of none? Isn't it required? also maybe just call it url which is more generic.