spring-projects/spring-aot-smoke-tests

Investigate using mccutchen/go-httpbin

wilkinsona opened this issue · 8 comments

I think we should take another look at the changes made for #180. It may be possible to revert much of them and use mccutchen/go-httpbin running in a container in place of http(s)://httpbin.org.

https://hub.docker.com/r/kennethreitz/httpbin is another option. It's the "original" implementation and is what's running at https://httpbin.org.

I found https://hub.docker.com/r/kennethreitz/httpbin while looking for a suitable docker replacement. The problem with it is that it doesn't support ARM machines. I'm in favor of the Go re-implementation.

Thanks, Moritz. I hadn't released that httpbin was x64 only. Let's use go-httpbin then.

I think this has broken the tests for HTTPS. For example rest-template started to fail with bc7195d. See https://ci.spring.io/teams/spring-aot-smoke-tests/pipelines/spring-aot-smoke-tests-3.0.x/jobs/rest-template-app-test/builds/81.

Oh :( I'm going to take a look.

We're using an old docker compose version which doesn't support the configs element in the compose file. #187 updates docker compose to a newer version.

And i botched the regex. It worked on my machine because it was localhost, but on the CI it's an IP address and \w+ doesn't match IP addresses.