Skip to content

Switch RestServiceHandler from RestTemplate to RestClient.#884

Draft
paveljandejsek wants to merge 2 commits into
EmbedITCZ:masterfrom
paveljandejsek:rest-template-replacement
Draft

Switch RestServiceHandler from RestTemplate to RestClient.#884
paveljandejsek wants to merge 2 commits into
EmbedITCZ:masterfrom
paveljandejsek:rest-template-replacement

Conversation

@paveljandejsek
Copy link
Copy Markdown
Collaborator

@paveljandejsek paveljandejsek commented May 16, 2026

#883
RestServiceHandler is a mess at this moment so this is just a non-ideal minimalistic change (Ideally we would not even create RequestEntity, instead of current creating it and then using body and headers from it...)
Considerations:

  • renaming the initTemplate to initRestClient, removing RestTemplateConfigurer - lots of breaking changes sadly 🙁
  • by keeping the old naming pattern we are still leaking implementation details
  • should we use autoconfigured RestClientBuilder/RestClient instead of creating our own? (I would almost say so, but this PR contains the same "old approach")
  • instead of clientConfigurer (aka old templateConfigurer) naming I used the fuller restClientConfigurer
  • proposed oauth2 support added in separate commit - without any test configuration for now, will need to be added along with some test (ideally once we agree that this is how we should implement it or decide how else to implement it)

@paveljandejsek paveljandejsek force-pushed the rest-template-replacement branch from 1f255ab to 6206a83 Compare May 17, 2026 18:15
@paveljandejsek paveljandejsek force-pushed the rest-template-replacement branch from 6206a83 to 61960d9 Compare May 17, 2026 18:23
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.

1 participant