824 Commits

Author SHA1 Message Date
Marcin Grzejszczak
165a6ac1ba Making post zuul filter close the span (#568)
without this change when dealing with Zuul, it's TraceFilter that is closing a span on an error
with this change, we're allowing the PostZuulFilter to close that span and TraceFilter doesn't interfere

fixes #563
2017-04-19 13:37:59 +02:00
Marcin Grzejszczak
54ae618815 Merge branch '1.1.x' 2017-04-19 10:46:59 +02:00
Marcin Grzejszczak
64791deabd Span Adjuster (#567)
with this change, we're adding a possibility to fully customize the span before it gets reported to e.g. Zipkin

fixes #553
2017-04-19 10:35:48 +02:00
Marcin Grzejszczak
3d7b15f62c Merge branch '1.1.x' 2017-04-18 16:20:37 +02:00
Marcin Grzejszczak
a879dc9e30 Added docs for customization executors 2017-04-18 16:20:24 +02:00
Marcin Grzejszczak
8cd372a099 Fixed the code 2017-04-18 14:58:32 +02:00
Marcin Grzejszczak
9dc626a873 Added docs for custom async rest template (#566)
fixes #564
2017-04-18 14:48:58 +02:00
Marcin Grzejszczak
c6d645993f Added docs for custom async rest template (#566)
fixes #564
2017-04-18 14:45:07 +02:00
Marcin Grzejszczak
ef6acec630 Merge branch '1.1.x' 2017-04-12 13:06:09 +02:00
Marcin Grzejszczak
816cfe317a Moved an executor aspect to a bean post processor
without this change when an executor implementation was a final class or had a final method - the application wouldn't start or would produce NPE at startup.
    with this change we have a custom bean post processor that wraps an aspect manualy either via a JDK Proxy when a class is final or a method is final or creates a CGLIB proxy if that's possible.

    fixes #550
2017-04-12 12:14:45 +02:00
Marcin Grzejszczak
46257ea370 Moved an executor aspect to a bean post processor
without this change when an executor implementation was a final class or had a final method - the application wouldn't start or would produce NPE at startup.
    with this change we have a custom bean post processor that wraps an aspect manualy either via a JDK Proxy when a class is final or a method is final or creates a CGLIB proxy if that's possible.

    fixes #550
2017-04-12 12:02:39 +02:00
Marcin Grzejszczak
593aa34806 Added tests for 547 2017-04-11 14:58:00 +02:00
Marcin Grzejszczak
f2798e8827 Wrapping callbacks in trace representation
without this change when user adds a callback then it's not wrapped in a trace representation, thus when executed in a separate thread, it's then missing all the tracing information
    with this change we're wrapping the future in our representation that passes the tracing information around

    fixes #546
2017-04-11 14:18:58 +02:00
Marcin Grzejszczak
cce5b18dec Wrapping callbacks in trace representation
without this change when user adds a callback then it's not wrapped in a trace representation, thus when executed in a separate thread, it's then missing all the tracing information
    with this change we're wrapping the future in our representation that passes the tracing information around

    fixes #546
2017-04-11 14:16:25 +02:00
Marcin Grzejszczak
c9bbc2e4a5 Bumping versions to 1.2.1.BUILD-SNAPSHOT after release 2017-04-11 10:18:41 +02:00
Marcin Grzejszczak
850534689c Going back to snapshots 2017-04-11 10:06:35 +02:00
Marcin Grzejszczak
fdace4f93a Update SNAPSHOT to 1.2.0.RELEASE 2017-04-11 10:03:04 +02:00
Marcin Grzejszczak
15a44f6ce2 Stopped producing warnings if the current span is null
fixes #552
2017-04-10 16:51:31 +02:00
Marcin Grzejszczak
e316e8bc3d Stopped producing warnings if the current span is null
fixes #552
2017-04-10 16:48:50 +02:00
Denis Stepanov
e98d8129e2 Use HystrixFeign only if it's present on classpath (#543) 2017-03-22 10:44:43 +01:00
Denis Stepanov
327ce80c38 Use HystrixFeign only if it's present on classpath (#544) 2017-03-22 10:43:56 +01:00
Marcin Grzejszczak
e68bc5e634 Going back to snapshots 2017-03-17 09:44:27 +01:00
Marcin Grzejszczak
760c1841b7 Update SNAPSHOT to 1.2.0.RC1 2017-03-17 09:39:16 +01:00
Marcin Grzejszczak
1a376e7491 Bumping versions 2017-03-10 17:22:57 +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
bf9f84f2f9 Fixed deps of mockserver 2017-03-04 09:37:37 +01:00
Marcin Grzejszczak
0f29735c11 Managing spans with annotations (#526)
The main arguments for these features are

* api-agnostic means to collaborate with a span
    - use of annotations allows users to add to a span with no library dependency on a span api.
    This allows Sleuth to change its core api less impact to user code.
* reduced surface area for basic span operations.
    - without this feature one has to use the span api, which has lifecycle commands that
    could be used incorrectly. By only exposing scope, tag and log functionality, users can
    collaborate without accidentally breaking span lifecycle.
* collaboration with runtime generated code
    - with libraries such as Spring Data / Feign the implementations of interfaces are generated
    at runtime thus span wrapping of objects was tedious. Now you can provide annotations
     over interfaces and arguments of those interfaces

This PR is an adoption of @Koizumi85 work started here - https://github.com/Koizumi85/spring-cloud-sleuth-annotation

fixes #182
2017-02-27 15:26:03 +01:00
Marcin Grzejszczak
cc07a8f777 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 17:06:32 +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
e0c2656354 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 12:05:28 +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
7caf522ecf 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 16:28:47 +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
6ac48cfc01 Fixing a broken test 2017-02-20 15:47:31 +01:00
Marcin Grzejszczak
f8e6b00f35 Updated the version of the deprecated headers
fixes #519
2017-02-15 21:26:02 +01:00
Marcin Grzejszczak
726be4f944 Caches the result of getting error controller bean
related to #512
2017-02-13 11:25:05 +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
05f812abe0 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:47:37 +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
ce226ba334 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:35:07 +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
634fa90a0d Display overriden service name in logs (addition to #487) (#507)
Use `spring.zipkin.service.name` if present.
2017-02-02 09:13:37 +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
ea30fefdf9 Removed duplicated dependency
fixes #504
2017-02-01 11:20:04 +01:00
Marcin Grzejszczak
abefcaa354 Removed duplicated dependency
fixes #504
2017-02-01 11:19:51 +01:00
Marcin Grzejszczak
b996de50ea 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:49:47 +01:00