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

Configure Rest client to proxy request via wiremock #164

Open
rmanibus opened this issue Oct 11, 2024 · 6 comments · May be fixed by #166
Open

Configure Rest client to proxy request via wiremock #164

rmanibus opened this issue Oct 11, 2024 · 6 comments · May be fixed by #166

Comments

@rmanibus
Copy link

rmanibus commented Oct 11, 2024

When using this extension, each rest client instance needs to be manually configured for testing against wiremock using something like:

quarkus:
  rest-client:
    my-rest-client:
      url: "http://localhost:${quarkus.wiremock.devservices.port}"

We could avoid this by using wiremock as a rest client proxy in dev mode and test.

@rmanibus
Copy link
Author

cc @geoand

@rmanibus rmanibus changed the title Configure Rest client to proxy request to wiremock Configure Rest client to proxy request via wiremock Oct 12, 2024
@rmanibus
Copy link
Author

I explored two different implementations

@geoand
Copy link
Contributor

geoand commented Oct 14, 2024

I like the idea!

@chberger
Copy link
Contributor

@rmanibus I like the idea as well. Please give me some time to review your PRs. I’ll get back to you once I’ve had a chance to go over them.

@chberger chberger pinned this issue Oct 24, 2024
@rmanibus
Copy link
Author

It could be really cool if we could make domain matching works on top of that. I tried multiple solutions but haven't been able to make it work yet

Copy link

@quarkiverse/quarkiverse-wiremock This is being labeled as stale.

@github-actions github-actions bot added the Stale label Dec 23, 2024
@chberger chberger removed the Stale label Dec 30, 2024
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 a pull request may close this issue.

3 participants