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.

Sign up for GitHub

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

Jump to bottom

HTTP REST Connector: migrate from Google HTTP Connector #1069

Closed
igpetrov opened this issue Aug 23, 2023 · 2 comments · Fixed by #2660
Closed

HTTP REST Connector: migrate from Google HTTP Connector #1069

igpetrov opened this issue Aug 23, 2023 · 2 comments · Fixed by #2660
Assignees
@johnBgood
Labels
kind:task Categorizes an issue or PR as general maintenance, i.e. cleanup, refactoring, etc.

Comments

@igpetrov
Copy link
Contributor

What should we do?

Currently, HTTP REST and GraphQL connectors use Google HTTP Connector.

As a part of this task, we need to migrate to something more mature, such as Apache HTTP client, or other options to be considered.

Why should we do it?

As per discussion, we agreed that it makes sense to do.
@igpetrov igpetrov added the kind:task Categorizes an issue or PR as general maintenance, i.e. cleanup, refactoring, etc. label Aug 23, 2023
@sbuettner
Copy link
Contributor

sbuettner commented Oct 12, 2023 •
edited
Loading

Relevant articles:
https://www.wiremock.io/post/java-http-client-comparison
https://www.innoq.com/en/articles/2022/01/java-http-clients/
https://reflectoring.io/comparison-of-java-http-clients/

Relevant clients mentioned:

@Oleksiivanov Oleksiivanov mentioned this issue Mar 18, 2024
Merged
@sbuettner sbuettner self-assigned this Apr 4, 2024
@johnBgood johnBgood assigned johnBgood and unassigned sbuettner Apr 30, 2024
@johnBgood
Copy link
Collaborator

Mentioning this ticket as part of the refactoring: #2520

@johnBgood johnBgood linked a pull request May 27, 2024 that will close this issue
1069 http rest connector migrate from google http connector #2660
Merged
@johnBgood johnBgood mentioned this issue May 27, 2024
Merged
@johnBgood johnBgood mentioned this issue Jun 17, 2024
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@johnBgood johnBgood

Labels
kind:task Categorizes an issue or PR as general maintenance, i.e. cleanup, refactoring, etc.
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

1069 http rest connector migrate from google http connector camunda/connectors
3 participants
@sbuettner @johnBgood @igpetrov