Hacktoberfest 2026: los issues que los mantenedores marcaron para octubre, abiertos y aptos para principiantes. Explorar issues de Hacktoberfest

Http task: no fluent connectionTimeout() builder method

Abierto
#134 1 comentario 0 reacciones 2 asignados Ver en GitHub

@kowser-orkes ya está trabajando en esto.

Desde el 24/7/2026.

Evaluación

Este issue todavía no se ha evaluado.

Descripción

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.

Lenguaje dominante
Java
Estrellas
12
Forks
10
Merge medio
1 d 10 h
PR fusionados (30 d)
2

Guía de contribución

Abrir la guía de contribución

Primeros pasos

  1. Lee el issue completo y luego la guía de contribución del proyecto.
  2. Comenta en el issue que vas a ocuparte — evita que dos personas hagan lo mismo.
  3. Haz un fork del repositorio y trabaja en una rama.
  4. Abre un pull request que haga referencia al número del issue.

Más de conductor-oss/java-sdk

Todos los issues de conductor-oss/java-sdk

Issues similares

Más issues de Java

Recibe los nuevos issues en tu correo

Un resumen breve de issues de GitHub para principiantes.