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

Pass client_options in execute and executes_with_redirect HTTP client methods #742

Merged
merged 1 commit into from
Aug 10, 2022

Conversation

jcamiel
Copy link
Collaborator

@jcamiel jcamiel commented Aug 10, 2022

This is a preliminary PR for #624 .

ClientOptions is not saved anymore in Client struct. We pass now an option: &ClientOptions parameter in each method of the HTTP client. This will make easier to override a ClientOption for each executed request.

@jcamiel jcamiel force-pushed the feature/add-options-in-http-client-methods branch from 4403ea9 to 33ed1f4 Compare August 10, 2022 09:59
@jcamiel jcamiel marked this pull request as ready for review August 10, 2022 10:00
@fabricereix
Copy link
Collaborator

/accept

@github-actions
Copy link

✅ Pull request accepted and closed by fabricereix with fast forward merge..

# List of commits merged from Orange-OpenSource/hurl/feature/add-options-in-http-client-methods branch into Orange-OpenSource/hurl/master branch:

  • 33ed1f4 Pass client_options in execute and executes_with_redirect HTTP client methods.

@github-actions github-actions bot merged commit 33ed1f4 into master Aug 10, 2022
@github-actions github-actions bot deleted the feature/add-options-in-http-client-methods branch August 10, 2022 11:18
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