{"record":{"id":"f0abfd17126ab8bd","repo":"Tencent/WeKnora","slug":"timeout-cannot-be-negative","errorCode":null,"errorMessage":"timeout cannot be negative","messagePattern":"timeout cannot be negative","errorType":"validation","errorClass":null,"httpStatus":null,"severity":"error","filePath":"internal/sandbox/sandbox.go","lineNumber":378,"sourceCode":"\t\tCubeHTTPTimeout: DefaultCubeHTTPTimeout,\n\t}\n}\n\n// ValidateConfig validates sandbox configuration\nfunc ValidateConfig(config *Config) error {\n\tif config == nil {\n\t\treturn errors.New(\"config is nil\")\n\t}\n\n\tswitch config.Type {\n\tcase SandboxTypeDocker, SandboxTypeCube, SandboxTypeE2B, SandboxTypeDisabled:\n\t\t// Valid types\n\tdefault:\n\t\treturn errors.New(\"invalid sandbox type\")\n\t}\n\n\tif config.DefaultTimeout < 0 {\n\t\treturn errors.New(\"timeout cannot be negative\")\n\t}\n\n\tif config.MaxMemory < 0 {\n\t\treturn errors.New(\"memory limit cannot be negative\")\n\t}\n\n\tif config.MaxCPU < 0 {\n\t\treturn errors.New(\"CPU limit cannot be negative\")\n\t}\n\n\treturn nil\n}\n","sourceCodeStart":360,"sourceCodeEnd":391,"githubUrl":"https://github.com/Tencent/WeKnora/blob/988cbb03305e055d8ebb7d46d9ac6cc0803cd074/internal/sandbox/sandbox.go#L360-L391","documentation":"ValidateConfig rejects a SandboxConfig whose DefaultTimeout is negative. Timeouts must be zero (no default) or positive; a negative value indicates a computation or config parsing bug, so Manager creation refuses to proceed.","triggerScenarios":"Passing a SandboxConfig with DefaultTimeout < 0 to NewManager or NewSessionBoundManager, e.g. from subtracting timestamps, misreading duration units, or a negative number in a config file.","commonSituations":"Loading YAML/JSON config with a negative timeout; computing a deadline with time.Since instead of time.Until and storing the negative result.","solutions":["Set DefaultTimeout to a positive duration (e.g. 30 * time.Second) or leave it at zero for no default.","Audit where the value is computed — fix any time.Since/time.Until mixup producing negatives.","Validate DefaultTimeout >= 0 before constructing the Manager and fail fast with context."],"exampleFix":"// before\ncfg.DefaultTimeout = time.Until(deadline) // negative if deadline passed\n// after\nif d := time.Until(deadline); d > 0 {\n    cfg.DefaultTimeout = d\n} else {\n    cfg.DefaultTimeout = 30 * time.Second\n}","handlingStrategy":"validation","validationCode":"if cfg.DefaultTimeout < 0 {\n    return fmt.Errorf(\"invalid sandbox config: DefaultTimeout=%v must be >= 0\", cfg.DefaultTimeout)\n}","typeGuard":null,"tryCatchPattern":"if err := sandbox.ValidateConfig(cfg); err != nil {\n    if strings.Contains(err.Error(), \"timeout\") {\n        cfg.DefaultTimeout = 30 * time.Second // safe default\n        return nil\n    }\n    return err\n}","preventionTips":["Use time.Duration typed fields everywhere; avoid raw ints.","Never assign the result of time.Since as a timeout; use time.Until with a positivity check.","Validate configs in unit tests."],"tags":["go","sandbox","configuration","validation","timeout"],"backgroundTag":"invalid-config-value","analyzedSha":"988cbb03305e055d8ebb7d46d9ac6cc0803cd074","analyzedAt":"2026-09-02T14:41:08.344Z","contentChangedAt":null,"schemaVersion":2},"datasetVersion":"2026-09-08T10:18:20.063Z"}