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

new test: remove backend between requests #15

Open
Geal opened this issue Oct 3, 2018 · 2 comments
Open

new test: remove backend between requests #15

Geal opened this issue Oct 3, 2018 · 2 comments

Comments

@Geal
Copy link
Member

Geal commented Oct 3, 2018

  • set up an application with one backend
  • start a server for that backend, and another server
  • those servers indicate their id in the response
  • the client sends a request (in keep alive)
  • check that we receive a response, and that it comes from the first backend
  • change the application's configuration in sozu to remove the first backend and replace it with the second one
  • the client sends a request
  • check that we receive a response, and that it comes from the second backend
@NotBad4U NotBad4U added this to the first test suite milestone Oct 5, 2018
@NotBad4U
Copy link
Contributor

NotBad4U commented Oct 9, 2018

Done in c75d14b

@Geal
Copy link
Member Author

Geal commented Oct 10, 2018

how can we be sure that the requests are done in keep-alive? It is not enough to pass the Connection: keep-alive header (and is probably redundant, since HTTP 1.1 is keep-alive by default), we have to be sure that the same TCP connection is used.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants