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

[confighttp] Add max_redirects configuration option #10870

Open
rogercoll opened this issue Aug 13, 2024 · 1 comment
Open

[confighttp] Add max_redirects configuration option #10870

rogercoll opened this issue Aug 13, 2024 · 1 comment

Comments

@rogercoll
Copy link
Contributor

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

Components that use the confighttp to initialize an HTTP Client are not able to define the maximum number of redirection the client should perform. The provided client uses the default CheckRedirect http configuration:

// If CheckRedirect is nil, the Client uses its default policy,
// which is to stop after 10 consecutive requests.

Use case: open-telemetry/opentelemetry-collector-contrib#34623

Describe the solution you'd like

Configuration option to limit the number of redirections, if non provided, the client should continue to work as it is (default redirections)

endpoint: otelcol2:55690
    auth:
      authenticator: some-authenticator-extension
...
max_redirects: 0
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

No branches or pull requests

1 participant