Skip to content

Feat/stackittpr 542 allow longer timout than default#6056

Merged
cgoetz-inovex merged 7 commits intomainfrom
feat/STACKITTPR-542-allow-longer-timout-than-default
Mar 31, 2026
Merged

Feat/stackittpr 542 allow longer timout than default#6056
cgoetz-inovex merged 7 commits intomainfrom
feat/STACKITTPR-542-allow-longer-timout-than-default

Conversation

@cgoetz-inovex
Copy link
Copy Markdown
Contributor

Description

relates to #1234

Checklist

  • Issue was linked above
  • No generated code was adjusted manually (check comments in file header)
  • Changelogs
    • Changelog in the root directory was adjusted (see here)
    • Changelog(s) of the service(s) were adjusted (see e.g. here)
  • VERSION file(s) of the service(s) were adjusted
  • Code format was applied: make fmt
  • Examples were added / adjusted (see examples/ directory)
  • Unit tests got implemented or updated
  • Unit tests are passing: make test (will be checked by CI)
  • No linter issues: make lint (will be checked by CI)

If the handler is configured with a timeout like 10 minutes and a user
passes a context with a timeout of 20 minutes, the original solution
timed out after 10 minutes instead of 20. context.WithTimeout() only
sets a new deadline if the new deadline is shorter than the one in the
parent context.

When timing out wrap ctx.Err() into error to allow
errors.Is(err, context.DeadlineExceeded).
@cgoetz-inovex cgoetz-inovex requested a review from a team as a code owner March 30, 2026 15:07
@cgoetz-inovex cgoetz-inovex merged commit eb4c36a into main Mar 31, 2026
17 of 24 checks passed
@cgoetz-inovex cgoetz-inovex deleted the feat/STACKITTPR-542-allow-longer-timout-than-default branch March 31, 2026 09:20
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.

2 participants