Hacktoberfest 2026: the issues maintainers tagged for October, open and beginner-friendly. Browse Hacktoberfest issues

Http task: no fluent connectionTimeout() builder method

Open
  • Is something wrong?
#134 1 comment 0 reactions 2 assignees View on GitHub

@kowser-orkes is already working on this.

Since Jul 24, 2026.

Assessment

This issue has not been assessed yet.

Description

bug

Summary

The Http builder class has a fluent readTimeout(int) method but no corresponding
connectionTimeout(int) method. Users who need to set connectionTimeOut must break
the fluent builder chain by calling getHttpRequest().setConnectionTimeOut(n) directly.

Server baseline

Conductor 3.32.0-rc.9

Evidence

Static:

  • Http.java:84 — public Http readTimeout(int readTimeout) exists
  • Http.Input (line 229-239) — has connectionTimeOut field, getter, setter
  • No connectionTimeout() fluent method on Http class

Live (2026-07-16): Server correctly reads connectionTimeOut from http_request
(workflow registered with connectionTimeOut: 3000 was accepted by Conductor OSS 3.32.0-rc.9).

Inconsistency

// readTimeout — has fluent method ✓
Http http = new Http("task_ref")
    .url("https://example.com")
    .method(HttpMethod.GET)
    .readTimeout(5000);

// connectionTimeOut — NO fluent method ✗ (must break builder chain)
http.getHttpRequest().setConnectionTimeOut(3000);

Proposed fix

// Http.java — add alongside readTimeout()
public Http connectionTimeout(int connectionTimeout) {
    this.httpRequest.setConnectionTimeOut(connectionTimeout);
    return this;
}

Related

Discovered during systematic SDK audit against Conductor OSS 3.32.0-rc.9.

Dominant language
Java
Stars
15
Forks
10
Avg merge
1d 15h
Merged PRs (30d)
3

Getting set up

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

More from conductor-oss/java-sdk

All issues in conductor-oss/java-sdk

Similar issues

More Java issues

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.