Skip to content

fix(openai-draw): apply extended timeout to reference image uploads - #205

Merged
3316891527 merged 1 commit into
devfrom
fix/openai-draw-upload-timeout
Nov 13, 2026
Merged

3316891527 merged 1 commit into
devfrom
fix/openai-draw-upload-timeout

Conversation

@3316891527

Copy link
Copy Markdown
Owner

Background

Reference-image requests call Tools.Net.uploadFile, which constructs a separate host HTTP client. The plugin's 600000 ms OkHttp builder settings apply only to text-to-image requests and do not configure the multipart request. Without an explicit option, the host uses a 20 second read timeout and a 15 second write timeout.

Changes

  • Pass timeout_ms: HTTP_TIMEOUT_MS to the multipart reference-image request.
  • Synchronize the TypeScript source, example JavaScript and packaged JavaScript.
  • Declare the already-supported timeout_ms option in Net.uploadFile types.
  • Keep the endpoint, multipart image field names and authentication unchanged.

Validation

  • Reviewed JsTools.uploadFile option forwarding and StandardHttpTools.resolveHttpTimeouts; timeout_ms is parsed as milliseconds and sets the overall request budget, with per-stage values inheriting it when unspecified.
  • git diff --check passed; example and packaged JavaScript are byte-identical.
  • No local builds, automated tests or authenticated image-generation requests were run.
  • This corrects a confirmed timeout configuration omission; successful image editing against a provider has not yet been verified.

Scope and release

  • Based on dev f5ebc03, preserving the merged worldbook fixes.
  • Additive type declaration; no host Kotlin changes or API-key logging.
  • No merge, workflow dispatch, APK release or offline-resource publication in this change.
  • Related issue: N/A; maintainer-reported reference-image timeout.

@3316891527
3316891527 merged commit b97f1ae into dev Nov 13, 2026
4 checks passed
@3316891527
3316891527 deleted the fix/openai-draw-upload-timeout branch November 14, 2026 10:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant