Upstream/BlueMap
1
0
Fork 0
mirror of https://github.com/BlueMap-Minecraft/BlueMap.git synced 2025-01-09 09:47:58 +01:00
Code Issues Projects Releases Wiki Activity

Make webserver resource-closing more watertight against potential leaks

Browse Source
This commit is contained in:
Lukas Rieger (Blue) 2023-09-05 14:22:30 +02:00
parent 155f56e62a
commit 7156993323
No known key found for this signature in database
GPG Key ID: 2D09EC5ED2687FF2
1 changed files with 20 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

21
BlueMapCommon/src/main/java/de/bluecolored/bluemap/common/web/http/HttpConnection.java
View File

@ -120,7 +120,26 @@ public void accept(SelectionKey selectionKey) {
private void handleIOException(Channel channel, IOException e) { private void handleIOException(Channel channel, IOException e) {
request.clear(); request.clear();
response = null;
if (response != null) {
try {
response.close();
} catch (IOException e2) {
Logger.global.logWarning("Failed to close response: " + e2);
}
response = null;
}
if (futureResponse != null) {
futureResponse.thenAccept(response -> {
try {
response.close();
} catch (IOException e2) {
Logger.global.logWarning("Failed to close response: " + e2);
}
});
futureResponse = null;
}
Logger.global.logDebug("Failed to process selection: " + e); Logger.global.logDebug("Failed to process selection: " + e);
try { try {