Commit Graph

885 Commits

Author SHA1 Message Date
Marcin Grzejszczak
e9480898b5 Publish artifact for samples-test-core 2017-03-15 12:57:44 +01:00
Marcin Grzejszczak
53f8f9eaa3 Bumped versions 2017-03-13 09:50:42 +01:00
Marcin Grzejszczak
035caf1ec8 Updated wrong version for sc-sleuth-deps 2017-03-10 11:21:49 +01:00
Marcin Grzejszczak
93afed967c Bumping versions after release 2017-03-10 11:18:18 +01:00
Marcin Grzejszczak
dde7d3b313 Going back to snapshots 2017-03-10 11:16:09 +01:00
Marcin Grzejszczak
1cab456015 Bumping versions before release 2017-03-10 11:10:23 +01:00
Marcin Grzejszczak
39b4617d51 Updated mvnw 2017-03-08 12:06:57 +01:00
Marcin Grzejszczak
64f4825966 Fixed deps 2017-03-04 09:30:53 +01:00
Marcin Grzejszczak
d1b4f73512 Trying to fix flickering 2017-02-27 16:41:41 +01:00
Marcin Grzejszczak
cdba72b44c Added missing aop dependency
fixes #534
2017-02-27 16:05:29 +01:00
Marcin Grzejszczak
8c17451e60 Added missing new trace headers
without this change for kafka the new message headers are not passed
with this change we're finally fixing this

fixes #529
2017-02-24 17:15:37 +01:00
Marcin Grzejszczak
90713eba34 Added support for Feign + OkHttpClient + Sleuth
without this change we get exceptions related to not being able to extend a final class
with this change we're not wrapping any final types and we have a custom bean post processor for okhttp

fixes #513
2017-02-24 16:58:27 +01:00
Marcin Grzejszczak
e773ef4524 Limiting the span name size to 50 chars
without this exceptions can occur when the span name is big
with this we're truncating the name to 50 chars

fixes #530
2017-02-24 11:53:09 +01:00
Marcin Grzejszczak
c202f53a85 Showing a workaround to make message serializable
this issue is there in core of Spring Framework  (https://jira.spring.io/browse/SPR-15262) but this tests shows how to apply a workaround to get ridd of that problem until we bump up version of SF.

related to https://github.com/spring-cloud/spring-cloud-sleuth/issues/520 and https://github.com/spring-cloud/spring-cloud-sleuth/issues/523
SO - http://stackoverflow.com/questions/42277170/messagedeliveryexception-thrown-when-using-amqp-backed-channel-with-brixton-sr7

fixes #520
2017-02-23 15:56:29 +01:00
Marcin Grzejszczak
6f0a77cb1f Fixed the missing surefire plugin 2017-02-21 18:01:46 +01:00
Marcin Grzejszczak
f8e6b00f35 Updated the version of the deprecated headers
fixes #519
2017-02-15 21:26:02 +01:00
Marcin Grzejszczak
4948498664 Ensured that we guard against negative ports
fixes #518
2017-02-15 13:55:06 +01:00
Marcin Grzejszczak
4d8d0dae1f Updated libs
fixes #517
2017-02-15 10:36:02 +01:00
Marcin Grzejszczak
694b929db3 Caches the result of getting error controller bean
related to #512
2017-02-13 11:24:44 +01:00
Marcin Grzejszczak
7f6dda7179 Lack of ErrorController will not result in blowing up the application
without this change when someone has a missing ErrorController bean then TraceHandlerInterceptor blows up
with this change that problem gets fixed

fixes #512
2017-02-13 10:45:59 +01:00
Marcin Grzejszczak
11ef72369c Updated the docs with logging pattern 2017-02-08 10:39:44 +01:00
Marcin Grzejszczak
5930e15a9e Removing colour from the logging pattern\n\nwithout this change due to the presence of colours in the logging pattern, the logs could be corrupt in some cases\nwith this change we're removing the colour\n\nfixes #510 2017-02-07 17:34:45 +01:00
Marcin Grzejszczak
8f11bbe9f0 Going back to snapshots 2017-02-03 13:39:59 +01:00
Marcin Grzejszczak
ac0c209f94 Bumping versions before release 2017-02-03 13:30:35 +01:00
Arthur Gavlyukovskiy
3179637eb8 Display overriden service name in logs (addition to #487) (#507)
Use `spring.zipkin.service.name` if present.
2017-02-02 09:12:06 +01:00
Marcin Grzejszczak
abefcaa354 Removed duplicated dependency
fixes #504
2017-02-01 11:19:51 +01:00
Marcin Grzejszczak
4060c0b805 Fixing the moment when SS is set
without this change there's a problem with the time when the SS is set on a span. Currently it's done in TraceFilter's finally block. The problem is that this code is executed after the response has been sent back to the client. Thus CR sometimes was set faster than SS (it doesn't make any sense from the logical point of view).

with this change we're introducing wrappers over the HttpServletResponse where we annotate the span with SS just after the response gets sent to the recipient.

fixes #492 #431
2017-02-01 10:31:50 +01:00
Marcin Grzejszczak
5c117534aa Revert "Fixing the moment when SS is set (#505)"
This reverts commit 92bcae68f3.
2017-01-31 16:20:31 +01:00
Marcin Grzejszczak
92bcae68f3 Fixing the moment when SS is set (#505)
without this change there's a problem with the time when the SS is set on a span. Currently it's done in TraceFilter's finally block. The problem is that this code is executed after the response has been sent back to the client. Thus CR sometimes was set faster than SS (it doesn't make any sense from the logical point of view).

with this change we're introducing wrappers over the HttpServletResponse where we annotate the span with SS just after the response gets sent to the recipient.

fixes #492 #431
2017-01-31 15:57:22 +01:00
Marcin Grzejszczak
7151a9cdb3 Giving an option to reuse a custom feign client
without this change you couldn't register your own client (for example to set your ssl config) for Sleuth to reuse it
with this change we're reusing a registered feign client bean or creating a default one if necessary

fixes #502
2017-01-26 18:47:43 +01:00
Marcin Grzejszczak
a84e624bc4 Fixed broken tests 2017-01-23 13:07:11 +01:00
Marcin Grzejszczak
e027d4cfb5 Making the branch 1.5.0 boot compatible
fixes #499
2017-01-23 12:37:55 +01:00
Adrian Cole
a396a09603 Updates to latest zipkin 1.19 (#495)
Most changes are not very interesting to sleuth, though the server is
quite a lot better with UI fixes galore. For sleuth mainly, small api
unaffecting glitch fixes around the Span.Builder.
2017-01-19 20:17:24 +08:00
Marcin Grzejszczak
49e92443fb Going back to snapshots 2017-01-12 13:03:16 +01:00
Marcin Grzejszczak
7dd5a26802 Bumped versions before release 2017-01-12 12:28:53 +01:00
Marcin Grzejszczak
4d7def0362 Merge branch '1.0.x' into 1.1.x 2017-01-11 12:32:09 +01:00
Marcin Grzejszczak
a559c11bcd Due to possible performance problems when DiscoveryClient gives nonexistent DNS name as local address Local endpoint is created from server properties by default.
Local Ip determined from spring.commons InetUtils.
    To enable local endpoint resolution via service discovery, DiscoveryClient must be present and 'spring.zipkin.service.locator.discovery' should be set to true.
    FallbackHavingEndpointLocator was caching and recreating EndpointLocator in the same time - removed the caching part.

    Fixes gh-403
2017-01-11 12:31:18 +01:00
Marcin Grzejszczak
33173ebd05 Merge branch '1.0.x' into 1.1.x 2017-01-11 11:21:55 +01:00
Ryan Baxter
e2427b6a7e Merge pull request #446 from ryanjbaxter/disable-retry
Disable Feign Retry
2017-01-10 11:23:40 -05:00
Marcin Grzejszczak
863ba87f5f Going back to snapshots 2017-01-09 17:02:20 +01:00
Marcin Grzejszczak
e4546afd49 Bumping versions before release 2017-01-09 16:22:49 +01:00
Marcin Grzejszczak
2097acb486 Adding the spring.instance_id tag
without this tag it's impossible to discern from which server was the given span originated
    with this change we're adding a tag in which we're passing the instance id value. The value can be taken either from Cloud Foundry or from a concatanation of some local properties like instance_id / application name / application port etc.

    fixes #369
2017-01-03 14:21:32 +01:00
Marcin Grzejszczak
9d8b2a279f Adding the spring.instance_id tag
without this tag it's impossible to discern from which server was the given span originated
    with this change we're adding a tag in which we're passing the instance id value. The value can be taken either from Cloud Foundry or from a concatanation of some local properties like instance_id / application name / application port etc.

    fixes #369
2017-01-03 14:20:56 +01:00
Marcin Grzejszczak
9d60602115 Added test for non web apps, fixed the autoconfig in that case
without this change the non web apps can't start
    with this change the missing bean gets registered

    also in terms of non boot apps if there is no port or address set we're providing some default values. In terms of the service name one can always use the override via the spring.zipkin.service.name property

    fixes #32
2017-01-02 14:44:54 +01:00
Marcin Grzejszczak
f085aae3b6 Added test for non web apps, fixed the autoconfig in that case
without this change the non web apps can't start
    with this change the missing bean gets registered

    also in terms of non boot apps if there is no port or address set we're providing some default values. In terms of the service name one can always use the override via the spring.zipkin.service.name property

    fixes #32
2017-01-02 14:39:26 +01:00
Marcin Grzejszczak
e117531d75 Added property for overriding the service name in Zipkin
without this only either service discovery service id / spring.application.name can be chosen as a service name for zipkin
    with this change you can pass spring.zipkin.service.name property to change override that both for HTTP and Stream collectors

    fixes #324
2017-01-02 10:18:48 +01:00
Marcin Grzejszczak
e982c4df80 Added property for overriding the service name in Zipkin
without this only either service discovery service id / spring.application.name can be chosen as a service name for zipkin
    with this change you can pass spring.zipkin.service.name property to change override that both for HTTP and Stream collectors

    fixes #324
2017-01-02 10:18:20 +01:00
Arthur Gavlyukovskiy
2a06734106 Created @ConfigurationProperties classes and updated additional metadata (#486)
* Created @ConfigurationProperties classes and updated additional metadata
2016-12-30 12:36:18 +01:00
Arthur Gavlyukovskiy
6aa7328cd9 Created @ConfigurationProperties classes and updated additional metadata (#486)
* Created @ConfigurationProperties classes and updated additional metadata
2016-12-30 12:35:46 +01:00
Marcin Grzejszczak
ab77e8ee74 Added checks to close / detach only if tracing
without this change during asynchronous communication some components are trying to detach or close spans that were aready detached. This leads to exceptions utils warnings and spans were not closed.
    with this change we're adding additional checks to ensure that we're tracing

    fixes #447
2016-12-30 11:36:16 +01:00