Skip to content

fix(clients): bound Fulcio and TSA HTTP requests - #216

Merged
wolfv merged 3 commits into
fix/fulcio-issuer-v2from
fix/signing-http-timeouts
Sep 20, 2026
Merged

wolfv merged 3 commits into
fix/fulcio-issuer-v2from
fix/signing-http-timeouts

Conversation

@wolfv

@wolfv wolfv commented Sep 6, 2026

Copy link
Copy Markdown
Collaborator

Summary

  • Default Fulcio and TSA clients to a 30-second total request timeout, including response-body reads.
  • Expose Fulcio builder with_timeout and TSA new_with_timeout overrides.
  • Route Fulcio default construction through its builder so every construction path is bounded.

Local HTTP regressions cover stalled headers and bodies. Crate tests also pass without default features.

Stack

Fix 3/6 from the crate review. Based on #215; merge the stack in order. This PR contains only its focused fix relative to its base.

Validation

  • Targeted regression tests pass.
  • cargo test --workspace --all-features --locked passes on the completed stack.
  • Strict Clippy (--all-targets --all-features --locked -- -D warnings) passes for the changed crates.
  • cargo fmt --all -- --check and git diff --check pass.

Signed-off-by: Wolf Vollprecht <w.vollprecht@gmail.com>
Signed-off-by: Wolf Vollprecht <w.vollprecht@gmail.com>
@wolfv
wolfv force-pushed the fix/tsa-status-string branch from bd16ce3 to a35a7e6 Compare September 6, 2026 13:26
@wolfv
wolfv force-pushed the fix/signing-http-timeouts branch from a51cd18 to 48b5820 Compare September 6, 2026 13:26
* fix(verify): enforce Fulcio authority windows at signing times
Signed-off-by: Wolf Vollprecht <w.vollprecht@gmail.com>

* fix(sign): reject unsatisfied TUF service requirements (#218)
Signed-off-by: Wolf Vollprecht <w.vollprecht@gmail.com>

* fix(conformance): honor staging during bundle verification (#219)
* fix(sign): reject unsatisfied TUF service requirements
Signed-off-by: Wolf Vollprecht <w.vollprecht@gmail.com>

* fix(conformance): honor staging during bundle verification
Signed-off-by: Wolf Vollprecht <w.vollprecht@gmail.com>

---------

Signed-off-by: Wolf Vollprecht <w.vollprecht@gmail.com>
Base automatically changed from fix/tsa-status-string to fix/fulcio-issuer-v2 September 20, 2026 10:06
@wolfv
wolfv merged commit 003b52e into fix/fulcio-issuer-v2 Sep 20, 2026
17 of 18 checks passed
@wolfv
wolfv deleted the fix/signing-http-timeouts branch September 20, 2026 10:08
wolfv added a commit that referenced this pull request Sep 20, 2026
* fix(crypto): support current Fulcio issuer extension

Signed-off-by: Wolf Vollprecht <w.vollprecht@gmail.com>

* fix(tsa)!: parse optional RFC 3161 status text (#215)

Signed-off-by: Wolf Vollprecht <w.vollprecht@gmail.com>

* fix(clients): bound Fulcio and TSA HTTP requests (#216)

* fix(tsa)!: parse optional RFC 3161 status text
Signed-off-by: Wolf Vollprecht <w.vollprecht@gmail.com>

* fix(clients): bound Fulcio and TSA HTTP requests
Signed-off-by: Wolf Vollprecht <w.vollprecht@gmail.com>

* fix(verify): enforce Fulcio authority windows at signing times (#217)
* fix(verify): enforce Fulcio authority windows at signing times
Signed-off-by: Wolf Vollprecht <w.vollprecht@gmail.com>

* fix(sign): reject unsatisfied TUF service requirements (#218)
Signed-off-by: Wolf Vollprecht <w.vollprecht@gmail.com>

* fix(conformance): honor staging during bundle verification (#219)
* fix(sign): reject unsatisfied TUF service requirements
Signed-off-by: Wolf Vollprecht <w.vollprecht@gmail.com>

* fix(conformance): honor staging during bundle verification
Signed-off-by: Wolf Vollprecht <w.vollprecht@gmail.com>

---------

Signed-off-by: Wolf Vollprecht <w.vollprecht@gmail.com>
wolfv added a commit that referenced this pull request Sep 22, 2026
* fix(crypto): support current Fulcio issuer extension

Signed-off-by: Wolf Vollprecht <w.vollprecht@gmail.com>

* fix(tsa)!: parse optional RFC 3161 status text (#215)

Signed-off-by: Wolf Vollprecht <w.vollprecht@gmail.com>

* fix(clients): bound Fulcio and TSA HTTP requests (#216)

* fix(tsa)!: parse optional RFC 3161 status text
Signed-off-by: Wolf Vollprecht <w.vollprecht@gmail.com>

* fix(clients): bound Fulcio and TSA HTTP requests
Signed-off-by: Wolf Vollprecht <w.vollprecht@gmail.com>

* fix(verify): enforce Fulcio authority windows at signing times (#217)
* fix(verify): enforce Fulcio authority windows at signing times
Signed-off-by: Wolf Vollprecht <w.vollprecht@gmail.com>

* fix(sign): reject unsatisfied TUF service requirements (#218)
Signed-off-by: Wolf Vollprecht <w.vollprecht@gmail.com>

* fix(conformance): honor staging during bundle verification (#219)
* fix(sign): reject unsatisfied TUF service requirements
Signed-off-by: Wolf Vollprecht <w.vollprecht@gmail.com>

* fix(conformance): honor staging during bundle verification
Signed-off-by: Wolf Vollprecht <w.vollprecht@gmail.com>

---------

Signed-off-by: Wolf Vollprecht <w.vollprecht@gmail.com>
wolfv added a commit that referenced this pull request Sep 22, 2026
* fix(crypto): support current Fulcio issuer extension

Signed-off-by: Wolf Vollprecht <w.vollprecht@gmail.com>

* fix(tsa)!: parse optional RFC 3161 status text (#215)

Signed-off-by: Wolf Vollprecht <w.vollprecht@gmail.com>

* fix(clients): bound Fulcio and TSA HTTP requests (#216)

* fix(tsa)!: parse optional RFC 3161 status text
Signed-off-by: Wolf Vollprecht <w.vollprecht@gmail.com>

* fix(clients): bound Fulcio and TSA HTTP requests
Signed-off-by: Wolf Vollprecht <w.vollprecht@gmail.com>

* fix(verify): enforce Fulcio authority windows at signing times (#217)
* fix(verify): enforce Fulcio authority windows at signing times
Signed-off-by: Wolf Vollprecht <w.vollprecht@gmail.com>

* fix(sign): reject unsatisfied TUF service requirements (#218)
Signed-off-by: Wolf Vollprecht <w.vollprecht@gmail.com>

* fix(conformance): honor staging during bundle verification (#219)
* fix(sign): reject unsatisfied TUF service requirements
Signed-off-by: Wolf Vollprecht <w.vollprecht@gmail.com>

* fix(conformance): honor staging during bundle verification
Signed-off-by: Wolf Vollprecht <w.vollprecht@gmail.com>

---------

Signed-off-by: Wolf Vollprecht <w.vollprecht@gmail.com>
baszalmstra pushed a commit that referenced this pull request Sep 23, 2026
* fix(crypto): support current Fulcio issuer extension

Signed-off-by: Wolf Vollprecht <w.vollprecht@gmail.com>

* fix(tsa)!: parse optional RFC 3161 status text (#215)

Signed-off-by: Wolf Vollprecht <w.vollprecht@gmail.com>

* fix(clients): bound Fulcio and TSA HTTP requests (#216)

* fix(tsa)!: parse optional RFC 3161 status text
Signed-off-by: Wolf Vollprecht <w.vollprecht@gmail.com>

* fix(clients): bound Fulcio and TSA HTTP requests
Signed-off-by: Wolf Vollprecht <w.vollprecht@gmail.com>

* fix(verify): enforce Fulcio authority windows at signing times (#217)
* fix(verify): enforce Fulcio authority windows at signing times
Signed-off-by: Wolf Vollprecht <w.vollprecht@gmail.com>

* fix(sign): reject unsatisfied TUF service requirements (#218)
Signed-off-by: Wolf Vollprecht <w.vollprecht@gmail.com>

* fix(conformance): honor staging during bundle verification (#219)
* fix(sign): reject unsatisfied TUF service requirements
Signed-off-by: Wolf Vollprecht <w.vollprecht@gmail.com>

* fix(conformance): honor staging during bundle verification
Signed-off-by: Wolf Vollprecht <w.vollprecht@gmail.com>

---------

Signed-off-by: Wolf Vollprecht <w.vollprecht@gmail.com>
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