Can you please create a release with this new code.

[herrhilmi]

swagger-parser/modules/swagger-parser-v3/src/main/java/io/swagger/v3/parser/util/RefUtils.java

Line 251 in 50d8cfb

try (InputStream inputStream = new FileInputStream(path.toFile())) {

Hi,

In fact, the below version has a memory leak in Line 111. The FileInputStream used to read the spec is never closed.

io.swagger.parser.v3 swagger-parser 2.0.32

I think the current code could help solve this, since it's used within try-with-resources block.

Thank you

[frantuma]

@herrhilmi thanks for reporting this, can you please clarify what you mean with the below version has a memory leak in Line 111? (e.g. which version, and can you share the pointer to the line in the file you are referring to?

[herrhilmi]

@frantuma

In fact, I use io.swagger.parser.v3:swagger-parser:2.O.32 in my code. This dependency has a transitive dependency to
io.swagger:swagger-parser:1.0.59 as shown in the line below within pom

swagger-parser/pom.xml

Line 397 in 50d8cfb

<swagger-parser-v2-version>1.0.59</swagger-parser-v2-version>

Leak memory is found in this class

swagger-parser/modules/swagger-parser/src/main/java/io/swagger/parser/util/RefUtils.java

Line 141 in 0cd775a

result = IOUtils.toString(new FileInputStream(pathToUse.toFile()), "UTF-8");

Thank you

[frantuma]

@herrhilmi thanks for the details, we have fixed that in #1721 , and we'll cut a release for v1 and v2 with the fix in around 2 weeks.

Closing ticket, please reopen if still experiencing issues