Skip to content

Commit 2b48530

Browse files
fn: Prevent exceptions in skin queue schedulers
1 parent 12de021 commit 2b48530

1 file changed

Lines changed: 18 additions & 10 deletions

File tree

  • plugins/fancynpcs/src/main/java/de/oliver/fancynpcs/skins/mineskin

plugins/fancynpcs/src/main/java/de/oliver/fancynpcs/skins/mineskin/MineSkinAPI.java

Lines changed: 18 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -63,28 +63,36 @@ public SkinInfo generateSkin(GenerateRequest req) throws RatelimitException {
6363
// TODO use queueResp.getRateLimit().next() instead
6464
throw new RatelimitException(System.currentTimeMillis() + 1000 * 10); // retry in next run
6565
}
66+
6667
FancyNpcs.getInstance().getFancyLogger().warn("Could not fetch skin: " + error.code() + ": " + error.message());
67-
FancyNpcs.getInstance().getFancyLogger().debug("QueueResp: " + queueResp.toString());
68-
FancyNpcs.getInstance().getFancyLogger().debug("JobResp: " + jobResp.toString());
68+
69+
if (queueResp != null) FancyNpcs.getInstance().getFancyLogger().debug("QueueResp: " + queueResp);
70+
if (jobResp != null) FancyNpcs.getInstance().getFancyLogger().debug("JobResp: " + jobResp);
6971
}
7072
} else if (cause instanceof SocketTimeoutException timeoutException) {
7173
FancyNpcs.getInstance().getFancyLogger().warn("Timeout while fetching skin: " + timeoutException.getMessage());
72-
FancyNpcs.getInstance().getFancyLogger().debug("QueueResp: " + queueResp.toString());
73-
FancyNpcs.getInstance().getFancyLogger().debug("JobResp: " + jobResp.toString());
74+
75+
if (queueResp != null) FancyNpcs.getInstance().getFancyLogger().debug("QueueResp: " + queueResp);
76+
if (jobResp != null) FancyNpcs.getInstance().getFancyLogger().debug("JobResp: " + jobResp);
77+
7478
throw new RatelimitException(System.currentTimeMillis() + 1000 * 10); // retry in next run
7579
} else {
7680
FancyNpcs.getInstance().getFancyLogger().error("Error in mineskin request: " + cause.getMessage());
77-
FancyNpcs.getInstance().getFancyLogger().debug("QueueResp: " + queueResp.toString());
78-
FancyNpcs.getInstance().getFancyLogger().debug("JobResp: " + jobResp.toString());
81+
82+
if (queueResp != null) FancyNpcs.getInstance().getFancyLogger().debug("QueueResp: " + queueResp);
83+
if (jobResp != null) FancyNpcs.getInstance().getFancyLogger().debug("JobResp: " + jobResp);
7984
}
8085
} catch (InterruptedException e) {
8186
FancyNpcs.getInstance().getFancyLogger().error("Thread was interrupted while waiting for skin generation.");
82-
FancyNpcs.getInstance().getFancyLogger().debug("QueueResp: " + queueResp.toString());
83-
FancyNpcs.getInstance().getFancyLogger().debug("JobResp: " + jobResp.toString());
87+
88+
if (queueResp != null) FancyNpcs.getInstance().getFancyLogger().debug("QueueResp: " + queueResp);
89+
if (jobResp != null) FancyNpcs.getInstance().getFancyLogger().debug("JobResp: " + jobResp);
90+
8491
} catch (Exception e) {
8592
FancyNpcs.getInstance().getFancyLogger().error("Unexpected error in skin generation: " + e.getMessage());
86-
FancyNpcs.getInstance().getFancyLogger().debug("QueueResp: " + queueResp.toString());
87-
FancyNpcs.getInstance().getFancyLogger().debug("JobResp: " + jobResp.toString());
93+
94+
if (queueResp != null) FancyNpcs.getInstance().getFancyLogger().debug("QueueResp: " + queueResp);
95+
if (jobResp != null) FancyNpcs.getInstance().getFancyLogger().debug("JobResp: " + jobResp);
8896
}
8997

9098
return null;

0 commit comments

Comments
 (0)