Skip to content

Commit a13a4af

Browse files
oschwaldclaude
andcommitted
STF-322: Restore interrupt flag in InterruptedException rewrap path
The existing catch (InterruptedException) block in responseFor() rewraps into GeoIp2Exception without restoring the thread's interrupt status, silently swallowing the cancellation signal. Per Java's interruption protocol, code that catches InterruptedException without rethrowing it should re-set the flag so callers up the stack can observe the cancellation. This is an independent bug fix bundled into the STF-322 retry work because the retry feature exposes the path more often. Per project commit hygiene it lands as a separate commit so it can be cherry-picked or reverted on its own. Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
1 parent 185cde8 commit a13a4af

1 file changed

Lines changed: 1 addition & 0 deletions

File tree

src/main/java/com/maxmind/geoip2/WebServiceClient.java

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -414,6 +414,7 @@ private <T> T responseFor(String path, InetAddress ipAddress, Class<T> cls)
414414
response.body().close();
415415
}
416416
} catch (InterruptedException e) {
417+
Thread.currentThread().interrupt();
417418
throw new GeoIp2Exception("Interrupted sending request", e);
418419
}
419420
}

0 commit comments

Comments
 (0)