- 02 Feb, 2016 2 commits
-
-
Spencer Gibb authored
fixes gh-807
-
- 01 Feb, 2016 5 commits
-
-
Spencer Gibb authored
Fix typo: zule-core -> zuul-core.
-
Spencer Gibb authored
-
Spencer Gibb authored
-
Spencer Gibb authored
* spectator-issue-264: fix registration bug in SpectatorMetricServices
-
Brian Harrington authored
The primary change is to fix a memory leak reported in Netflix/spectator#264. Each time a gauge was updated it was creating a new registration and because the map holds a strong reference these would never get collected. Further, the aggregate value created by the multiple registrations was not correct. In addition I added some test cases around the various inputs and checked that the results were reflected as expected in the registry. I noticed the timer values had a unit of milliseconds, but it isn't immediately clear if the reported value can ever less than 1.0. The conversion to long is now delayed until after converting to nanoseconds so duration values less than 1.0 will now work instead of just recording 0. For the histogram I changed to just using a cast to `long` to avoid boxing to a `Double`. As an FYI for the future, there is a DoubleDistributionSummary we have experimented with in spectator-ext-sandbox that might be more appropriate for this use-case.
-
- 31 Jan, 2016 2 commits
- 29 Jan, 2016 2 commits
-
-
Dave Syer authored
So that it isn't instantated eagerly on startup by the RefreshScope. Fixes gh-789
-
Niklas Herder authored
-
- 28 Jan, 2016 2 commits
-
-
Dave Syer authored
Tests are very sensitive to the Zuul RequestContext which is thread bound.
-
Marcin Grzejszczak authored
-
- 27 Jan, 2016 4 commits
-
-
Dave Syer authored
-
Dave Syer authored
The Apcahe client does not allow empty body for PATCH, nor does it allow a request to be pre-populated with a Content-Length. Spring Cloud tests were trying to do the former, and Feign allows the latter.
-
Dave Syer authored
-
Adrian Ivan authored
Change SimpleRouteLocator to work with both Zuul and DispatcherServlet Fixes gh-794
-
- 26 Jan, 2016 3 commits
-
-
Dave Syer authored
Fixes gh-710
-
Dave Syer authored
Apparently we already support eureka.datacenter as an alias for archaius.deployment.datacenter and it looks like when that code was written we intended to support eureka.environment in a similar way. But it was never finished off, so this change adds the necessary one line of code. Fixes gh-791
-
Dave Syer authored
Fixes gh-790
-
- 25 Jan, 2016 2 commits
-
-
Dave Syer authored
-
Dave Syer authored
Since DiscoverClient now supports metadata natively, we can use that and not rely on Eureka interafces at all. Fixes gh-787. Also fixes https://github.com/spring-cloud/spring-cloud-config/issues/302
-
- 24 Jan, 2016 1 commit
-
-
Dave Syer authored
Eureka supports native placeholders, but Spring can also be used. Added some docs to show how more explicitly. See gh-761
-
- 20 Jan, 2016 3 commits
-
-
Spencer Gibb authored
fixes gh-573
-
Spencer Gibb authored
fixes gh-781
-
Spencer Gibb authored
fixes gh-779
-
- 19 Jan, 2016 3 commits
-
-
Spencer Gibb authored
fixes gh-777
-
Dave Syer authored
Otherwise apps register the wrong health and status URLs by default. If there is no explicit management.port we just need a default that is the same as server.port.
-
Dave Syer authored
-
- 18 Jan, 2016 3 commits
-
-
Dave Syer authored
A new integration test ill be added in spring-cloud-samples/tests. Fixes gh-759
-
Dave Syer authored
-
Dave Syer authored
A change in Zuul in NetflixOSS means that it buffers all requests unless you switch it off at the level of the servlet. And even when you do that you need to ensure that Spring Cloud doesn't re-introduce a buffer through the Servlet30RequestWrapper. This change restores the feature in Spring Cloud Angel that multipart files posted to /zuul/* go straight through the servlet and are not buffered in memory. Fixes gh-773
-
- 16 Jan, 2016 2 commits
-
-
Dave Syer authored
-
Spencer Gibb authored
fixes gh-766
-
- 15 Jan, 2016 6 commits
-
-
Dave Syer authored
-
Dave Syer authored
More often than not what Route.path was used it had to be concatenated with the prefix. This change just encapsulates that concern in the Route itself.
-
Dave Syer authored
If it returns Routes instead of Strings it's a lot more obvious what it's supposed.
-
Dave Syer authored
-
Marcin Grzejszczak authored
-
Marcin Grzejszczak authored
-