-
Notifications
You must be signed in to change notification settings - Fork 2.5k
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
Upgrade hystrix to 1.4.4 #300
Comments
wait to merge a765839 until we have something with an RC |
RC for what? Why do we need to wait? |
just worried about changes to the dashboard that haven't really been tried out yet. If you're ok, I'll merge. |
Let's ask @royclarkson or someone to try his dashboard. I assume our sample works because a user asked for this? |
someone noted that 1.0.0 used 1.4.0.RC of hystrix, 1.0.1 snapshot already has hystrix 1.4.1. I noted the latest was 1.4.4 and had dashboard changes that we should bring along. |
one of the changes was to ignore a header, that was my big worry |
I'm working on testing my changes (using the latest snapshot) in PCF today. What's the issue with the transfer encoding header? |
It includes changes to the dashboard and proxy servlet:
https://github.com/Netflix/Hystrix/blob/master/CHANGELOG.md
The text was updated successfully, but these errors were encountered: