Skip to content
New issue

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

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

Already on GitHub? # to your account

Expose accessor method for underlying ClientHttpRequestFactorySettings #43444

Closed
wants to merge 1 commit into from

Conversation

quaff
Copy link
Contributor

@quaff quaff commented Dec 9, 2024

After this commit, we can customize ClientHttpRequestFactorySettings base on underlying one.

After this commit, we can customize ClientHttpRequestFactorySettings base on underlying one
@philwebb
Copy link
Member

philwebb commented Dec 9, 2024

I'm not keen to add access for the settings. I've reopened #43258 so we can continue the discussion there.

@philwebb philwebb closed this Dec 9, 2024
@philwebb philwebb added status: declined A suggestion or change that we don't feel we should currently apply and removed status: waiting-for-triage An issue we've not yet triaged labels Dec 9, 2024
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
status: declined A suggestion or change that we don't feel we should currently apply
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants