Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Explore adding configurable http timeouts #2331

Closed
Racer159 opened this issue Feb 23, 2024 · 1 comment
Closed

Explore adding configurable http timeouts #2331

Racer159 opened this issue Feb 23, 2024 · 1 comment
Labels
enhancement ✨ New feature or request

Comments

@Racer159
Copy link
Contributor

Is your feature request related to a problem? Please describe.

We should explore adding configurable timeouts to Zarf's operations so that operations can be optimized for different environments that may be more austere.

Describe the solution you'd like

Ideally there would be a way to configure the default Transport timeouts that are used across Zarf - we should likely construct a version of Transport that we can use throughout the application and let these be configured through env vars or other configuration.

Describe alternatives you've considered

We could leave the timeouts as is, but this may reduce some of Zarf's applicability to very austere environments.

Additional context

This came out of discussions on #2319 when adding an http request header timeout.

@Noxsios
Copy link
Contributor

Noxsios commented Mar 8, 2024

Closed by #2345

@Noxsios Noxsios closed this as completed Mar 8, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement ✨ New feature or request
Projects
Status: Done
Status: Closed
Development

No branches or pull requests

2 participants