Added file.Close() call to file cache Get() method

go-spatial / tegola

Tegola is a Mapbox Vector Tile server written in Go

http://tegola.io/

MIT License

1.28k stars 196 forks source link

Added file.Close() call to file cache Get() method #810

Closed ARolek closed 2 years ago

ARolek commented 3 years ago

The Get() method for the file cache was not closing files that it opened. This will eventually cause an issue with too many file descriptors being open.

closes #586

coveralls commented 3 years ago

Pull Request Test Coverage Report for Build 88149d38e-PR-810

1 of 1 (100.0%) changed or added relevant line in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.009%) to 45.029%

Totals
Change from base Build 5013d2756:	0.009%
Covered Lines:	5439
Relevant Lines:	12079

💛 - Coveralls

ARolek commented 2 years ago

closed via https://github.com/go-spatial/tegola/commit/047810325c10d6724808d9762918cc5044f09a75