Don't think you should have instance variables in services (since you don't know if and when service class instances will be shared).  They should be local to `generateArchiveOfFiles`.