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

Fix RestClient to use defaultRequest on RestClient.Builder #32034

Conversation

injae-kim
Copy link
Contributor

Closes gh-32028

Motivation

Modification

  • Fix RestClient to use defaultRequest on RestClient.Builder

Result

  • Now RestClient uses defaultRequest that passed from RestClientBuilder.build() well

Comment on lines +461 to +463
if (defaultRequest != null) {
defaultRequest.accept(this);
}
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

private ClientRequest.Builder initRequestBuilder() {
if (defaultRequest != null) {
defaultRequest.accept(this);
}

FYI) almost same usage of defaultRequest on WebClient~!

@poutsma
Copy link
Contributor

poutsma commented Jan 17, 2024

Superseded by 0ada78a.

@poutsma poutsma closed this Jan 17, 2024
@poutsma poutsma added status: declined A suggestion or change that we don't feel we should currently apply in: web Issues in web modules (web, webmvc, webflux, websocket) and removed status: waiting-for-triage An issue we've not yet triaged or decided on labels Jan 17, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
in: web Issues in web modules (web, webmvc, webflux, websocket) 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.

DefaultRestClientBuilder.defaultRequest(...) doesn't do anything
3 participants