Skip to content

More detail in config erros #78

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
tj-wazei merged 2 commits into from
Feb 8, 2026
Merged

More detail in config erros #78

Changes from all commits
Commits
Show all changes
2 commits
Select commit Hold shift + click to select a range
d910895
More detail in config erros
Alathreon Feb 8, 2026
38da177
Splotless apply
Alathreon Feb 8, 2026
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
35 changes: 21 additions & 14 deletions JShellAPI/src/main/java/org/togetherjava/jshellapi/Config.java
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -26,33 +26,40 @@ private static boolean checkJShellWrapperImageName(String imageName) {

public Config {
if (regularSessionTimeoutSeconds <= 0)
throw new IllegalArgumentException("Invalid value " + regularSessionTimeoutSeconds);
throw new IllegalArgumentException(
"Invalid regularSessionTimeoutSeconds " + regularSessionTimeoutSeconds);
if (oneTimeSessionTimeoutSeconds <= 0)
throw new IllegalArgumentException("Invalid value " + oneTimeSessionTimeoutSeconds);
throw new IllegalArgumentException(
"Invalid oneTimeSessionTimeoutSeconds " + oneTimeSessionTimeoutSeconds);
if (evalTimeoutSeconds <= 0)
throw new IllegalArgumentException("Invalid value " + evalTimeoutSeconds);
throw new IllegalArgumentException("Invalid evalTimeoutSeconds " + evalTimeoutSeconds);
if (evalTimeoutValidationLeeway <= 0)
throw new IllegalArgumentException("Invalid value " + evalTimeoutSeconds);
throw new IllegalArgumentException(
"Invalid evalTimeoutValidationLeeway " + evalTimeoutSeconds);
if (sysOutCharLimit <= 0)
throw new IllegalArgumentException("Invalid value " + sysOutCharLimit);
throw new IllegalArgumentException("Invalid sysOutCharLimit " + sysOutCharLimit);
if (maxAliveSessions <= 0)
throw new IllegalArgumentException("Invalid value " + maxAliveSessions);
throw new IllegalArgumentException("Invalid maxAliveSessions " + maxAliveSessions);
if (dockerMaxRamMegaBytes <= 0)
throw new IllegalArgumentException("Invalid value " + dockerMaxRamMegaBytes);
throw new IllegalArgumentException(
"Invalid dockerMaxRamMegaBytes " + dockerMaxRamMegaBytes);
if (dockerCPUsUsage <= 0)
throw new IllegalArgumentException("Invalid value " + dockerCPUsUsage);
throw new IllegalArgumentException("Invalid dockerCPUsUsage " + dockerCPUsUsage);
if (dockerCPUSetCPUs != null && !dockerCPUSetCPUs.matches("[1-9]?\\d([-,]\\d?\\d)?"))
throw new IllegalArgumentException("Invalid value " + dockerCPUSetCPUs);
throw new IllegalArgumentException("Invalid dockerCPUSetCPUs " + dockerCPUSetCPUs);
if (schedulerSessionKillScanRateSeconds <= 0)
throw new IllegalArgumentException(
"Invalid value " + schedulerSessionKillScanRateSeconds);
throw new IllegalArgumentException("Invalid schedulerSessionKillScanRateSeconds "
+ schedulerSessionKillScanRateSeconds);
if (dockerResponseTimeout <= 0)
throw new IllegalArgumentException("Invalid value " + dockerResponseTimeout);
throw new IllegalArgumentException(
"Invalid dockerResponseTimeout " + dockerResponseTimeout);
if (dockerConnectionTimeout <= 0)
throw new IllegalArgumentException("Invalid value " + dockerConnectionTimeout);
throw new IllegalArgumentException(
"Invalid dockerConnectionTimeout " + dockerConnectionTimeout);

if (!checkJShellWrapperImageName(jshellWrapperImageName)) {
throw new IllegalArgumentException("Invalid value " + jshellWrapperImageName);
throw new IllegalArgumentException(
"Invalid jshellWrapperImageName " + jshellWrapperImageName);
}
}
}
Loading