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

Adds named httpclient support #1418

Merged
merged 2 commits into from
Apr 12, 2023
Merged

Adds named httpclient support #1418

merged 2 commits into from
Apr 12, 2023

Conversation

redbaty
Copy link
Contributor

@redbaty redbaty commented Sep 20, 2022

What kind of change does this PR introduce?
Makes it possible to set a custom name to the HTTP Client factory

What is the current behavior?
You can't use the same interface to point at multiple environments for example.

What is the new behavior?
You can use custom names to the same interface but with different environment/handlers.

What might this PR break?
In the original implementation UniqueName.ForType<T>() was used as the client's name, so if a custom name is used it might break somewhere, tho I've looked and not found any other references for this other than tests.

Please check if the PR fulfills these requirements

  • Tests for the changes have been added (for bug fixes / features)
  • Docs have been added / updated (for bug fixes / features)

Other information:

@redbaty redbaty changed the title Adds named httpclient support [WIP] Adds named httpclient support Sep 20, 2022
Used to resolve Refit http clients
@dnfadmin
Copy link

dnfadmin commented Sep 20, 2022

CLA assistant check
All CLA requirements met.

@redbaty redbaty changed the title [WIP] Adds named httpclient support Adds named httpclient support Sep 20, 2022
@glennawatson glennawatson merged commit b78bbc7 into reactiveui:main Apr 12, 2023
anaisbetts added a commit that referenced this pull request Apr 17, 2023
anaisbetts added a commit that referenced this pull request Apr 17, 2023
anaisbetts added a commit that referenced this pull request Apr 21, 2023
@github-actions
Copy link

This pull request has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs.

@github-actions github-actions bot locked as resolved and limited conversation to collaborators Apr 27, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants