I changed this when trying to support file downloads, but the way it's done it'll return an eventlet downloader for all http requests. It didn't hit me until now that this is used here to keep the stuff in memory instead of downloading to disk. So it's your call how you want to handle it, if we flush out nectar_factory to be where we control which downloader is used in all cases so that logic is centralized or if we go back to just manually instantiating the curl downloader here.