twitter-archive / diffy

Find potential bugs in your services with Diffy
https://twitter.com/diffyproject
Apache License 2.0
3.83k stars 368 forks source link

Diffy relies too much in the content-type header #18

Closed camiloribeiro closed 5 years ago

camiloribeiro commented 8 years ago

I was running some tests here to know how it works, and in one of my tests I have the following setup:

service primary:

service secondary:

service candidate:

In this scenario the diffy never checks the response and hangs forever.

Another test that I did could give you some more context on that issue. The test data in that case is the following:

service primary:

service secondary:

service candidate:

In this case diffy holds for a long time, something around 40 seconds, and returns saying that everything is fine and there is no difference in the responses.

In a third scenario for this issue, if I use the same response code for primary and secondary, it returns fast as usual, but still without differences.

Please let me know if you have any questions regarding the scenarios. I am using my docker-compose setup to test it, so if you need feel free to clone it.

puneetkhanduri commented 5 years ago

Issue resolved in https://github.com/opendiffy/diffy