The-pack-family/APIv3
3
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

Fail properly if MaxMind fails to respond / returns invalid data

Browse Source
This commit is contained in:
Colin McDonald 2016-07-03 16:28:31 -04:00
parent 77c3e49048
commit 527f465c12
2 changed files with 17 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

11
src/main/java/net/frozenorb/apiv3/model/IpIntel.java
View File

@ -45,7 +45,7 @@ public final class IpIntel {
MaxMindUtils.getInsights(id, (maxMindResult, error2) -> {
if (error2 != null) {
callback.onResult(null, error2);
} else {
} else if (maxMindResult != null) {
IpIntel newIpIntel = new IpIntel(id, maxMindResult);
ipIntelCollection.insertOne(newIpIntel, (ignored, error3) -> {
@ -55,6 +55,9 @@ public final class IpIntel {
callback.onResult(newIpIntel, null);
}
});
} else {
// MaxMind failed to return result
callback.onResult(null, null);
}
});
}
@ -90,6 +93,12 @@ public final class IpIntel {
return;
}
// MaxMind failed to return result
if (maxMindResult == null) {
createNewIntelFuture.complete();
return;
}
IpIntel newIpIntel = new IpIntel(ip, maxMindResult);
ipIntelCollection.insertOne(newIpIntel, (ignored, error3) -> {

8
src/main/java/net/frozenorb/apiv3/util/MaxMindUtils.java
View File

@ -26,7 +26,13 @@ public class MaxMindUtils {
httpsClient.get(443, "geoip.maxmind.com", "/geoip/v2.1/insights/" + ip, (response) -> {
response.bodyHandler((body) -> {
JsonObject bodyJson = new JsonObject(body.toString());
callback.onResult(new MaxMindResult(bodyJson), null);
try {
MaxMindResult maxMindResult = new MaxMindResult(bodyJson);
callback.onResult(maxMindResult, null);
} catch (Exception ignored) {
callback.onResult(null, null);
}
});
response.exceptionHandler((error) -> callback.onResult(null, error));