ZIP streams and inflater/deflaters need to be closed
Basics
Logistics
Basics
Logistics
Description
While researching some memory leak concerns, I noticed the streams used in the Redirect encoder/decoder aren't being closed formally, and the non-default stream constructors don't properly call the end() method on the Inflater/Deflater objects used.
Supposedly this causes RSS growth, but the GC should be calling finalize, which calls these methods, so this seems unlikely to matter all that much under ordinary use. Still should be cleaned up.
While researching some memory leak concerns, I noticed the streams used in the Redirect encoder/decoder aren't being closed formally, and the non-default stream constructors don't properly call the end() method on the Inflater/Deflater objects used.
Supposedly this causes RSS growth, but the GC should be calling finalize, which calls these methods, so this seems unlikely to matter all that much under ordinary use. Still should be cleaned up.