SpringCloud/spring-cloud-sleuth
1
0
Fork 0
Code Issues Packages Projects Releases Wiki Activity
2,190 Commits 14 Branches 0 Tags
facce03f003d6fda751e1ca12b69a4e2321b3d70
Commit Graph

2190 Commits

This Branch
This Branch
All Branches
Author SHA1 Message Date
Marcin Grzejszczak
facce03f00 Added symbolic link of index.adoc 2019-07-29 14:44:40 +02:00
Marcin Grzejszczak
51c367e45e Removed the - signs from default spring var resolution 
with the current setup we're using the MDC's default interpolation as a default for spring var resolution (which makes no sense)
with this change we're removing the dashes

fixes gh-1396
 2019-07-26 13:05:38 +02:00
Marcin Grzejszczak
b59277f0ef Not using httpStatus() method for custom status codes 
curently whenever we try to retrieve the http status we get an exception for custom codes
we've migrated to using the raw status code. Also, due to https://github.com/spring-projects/spring-framework/issues/23366 we had to manually wrap the ClientResponse so as we don't throw an exception on httpStatus() method

fixes gh-1393
 2019-07-26 12:12:56 +02:00
Marcin Grzejszczak
d642ffd09a Removed the eager injection of a bean in the configuration class 
at the point of creating the configuration class, that bean can use httptracing that can be in progress of being created.

with this fix we're making the call lazy

fixes gh-1354
 2019-07-26 10:36:24 +02:00
Marcin Grzejszczak
b96392c0f3 Removed the cycle by not fetching http tracing 
currently when an exception is thrown upon trying to fetch a Feign client bean, we're trying to fetch the httptracing bean to create a default client. The thing is that httptracing might be in creation
this fix removes the fetching of the httptracing bean since we can use the lazy client

fixes gh-1334
 2019-07-26 10:13:06 +02:00
Adrian Cole
3acf8db61c Latest brave 2019-07-04 08:07:18 +08:00
Marcin Grzejszczak
94c7c229a7 Return 0 status code on exception or invalid code (#1387) 
* Return 0 status code on exception or invalid code

fixes gh-1382
 2019-07-03 09:32:43 +02:00
Marcin Grzejszczak
fd35911d9b Fixed object class methods resolution for executorBPP 2019-07-02 10:51:58 +02:00
Marcin Grzejszczak
9c05e3793c Polish 2019-07-02 10:17:22 +02:00
Patrick Strawderman
41444d3e31 Fix proxying of executors with final methods (#1385) 
When proxying the various Executor interfaces in ExecutorbeanPostProcessor, the decision of whether or not to use a cglib proxy
was determined by checking if the bean's class is final, but final methods would also cause issues when using cglib. A common
example of such an implementation is DelegatingSecurityContextScheduledExecutorService, which has final methods.

To resolve the issue with ThreadPoolTaskExecutor, AsyncTaskExecutor, and ExecutorService, fall back to the JDK proxy if the
bean has any final methods.

fixes #1336
 2019-07-02 10:05:37 +02:00
liaochuntao
c6f03191e8 Invalid property name resolution fix 
fixes gh-1376
 2019-07-02 08:51:49 +02:00
Adrian Cole
7d38e7f616 Updates to latest brave 2019-07-02 09:24:54 +08:00
Spencer Gibb
f5c3e431a3 Bumps spring cloud build to 2.1.7.BUILD-SNAPSHOT 
Bumps spring cloud deps
 2019-06-21 12:50:55 -04:00
buildmaster
74c2b0b618 Bumping versions to 2.1.3.BUILD-SNAPSHOT after release 2019-06-20 21:26:50 +00:00
buildmaster
0a4e7b5d7b Going back to snapshots 2019-06-20 21:26:50 +00:00
buildmaster
cc0b3341a5 Update SNAPSHOT to 2.1.2.RELEASE 2019-06-20 21:25:35 +00:00
Spencer Gibb
023f1e5fda Bumps spring cloud build to 2.1.6.BUILD-SNAPSHOT 2019-06-20 14:49:44 -04:00
Adrian Cole
a5d8893baa Updates to latest Brave (#1373) 2019-06-15 23:49:26 +08:00
Marcin Grzejszczak
5c7519aee7 Esnures that multivalue headers are propagated in the Gateway; fixes gh-1352 2019-05-15 14:22:15 +02:00
Marcin Grzejszczak
9533fbfed0 Making a copy of "nativeHeaders" and working on a copy 
otherwise we got ConcurrentModificationExceptions

fixes gh-1345
 2019-05-03 22:10:24 +04:00
Tim Ysewyn
42b3e4c6c8 Updated pattern to ignore /actuator (#1342) 
Fixes #1338
 2019-04-25 16:32:42 +02:00
Marcin Grzejszczak
74e9988ee2 Made tests less brittle 2019-04-17 08:05:16 +04:00
Marcin Grzejszczak
e7f50d3526 Ensures that NPE is not thrown when errormessage has no failed message 
fixes gh-1335
 2019-04-17 00:12:21 +04:00
Marcin Grzejszczak
9324f3f714 Added a fix for special cases of executors that fail to be wrapped in a cglib proxy 
fixes gh-1232
 2019-04-10 15:02:14 +02:00
Marcin Grzejszczak
3e1184e7f7 Polish 2019-04-02 14:37:37 +02:00
Marcin Grzejszczak
ab82f30623 Added a workaround for the mutability of native headers (#1323) 
Since for some reason, the native headers sometimes are immutable even though the accessor says that the headers are mutable, then we have to ensure their
	 * mutability. We do so by first making a mutable copy of the native headers, then by removing the native headers from the headers map and replacing them with a mutable copy

fixes #1184
 2019-04-02 14:10:36 +02:00
Marcin Grzejszczak
18d535534e Fixed wrong https 2019-03-29 12:05:10 +01:00
Marcin Grzejszczak
0344f238f3 Merge pull request #1298 from spring-operator/polish-urls-xml-2.1.x 
URL Cleanup
 2019-03-29 11:46:17 +01:00
Marcin Grzejszczak
f318b0aa79 Fixed dubbo link 2019-03-29 11:42:28 +01:00
Marcin Grzejszczak
91135dc3ff Merge pull request #1281 from aftersss/hotfix/2.1.x 
Fix NullPointerException
 2019-03-29 11:30:46 +01:00
Spring Operator
929d6c440c URL Cleanup (#1309) 
This commit updates URLs to prefer the https protocol. Redirects are not followed to avoid accidentally expanding intentionally shortened URLs (i.e. if using a URL shortener).

# HTTP URLs that Could Not Be Fixed
These URLs were unable to be fixed. Please review them to see if they can be manually resolved.

* [ ] http://ajaxian.com/archives/100-line-ajax-wrapper (200) with 1 occurrences could not be migrated:
   ([https](https://ajaxian.com/archives/100-line-ajax-wrapper) result ConnectTimeoutException).
* [ ] http://dubbo.io/ (200) with 1 occurrences could not be migrated:
   ([https](https://dubbo.io/) result SSLHandshakeException).

# Fixed URLs

## Fixed But Review Recommended
These URLs were fixed, but the https status was not OK. However, the https status was the same as the http request or http redirected to an https URL, so they were migrated. Your review is recommended.

* [ ] http://jaubourg.net/2010/07/loading-script-as-onclick-handler-of.html (AnnotatedConnectException) with 1 occurrences migrated to:
  https://jaubourg.net/2010/07/loading-script-as-onclick-handler-of.html ([https](https://jaubourg.net/2010/07/loading-script-as-onclick-handler-of.html) result AnnotatedConnectException).
* [ ] http://192.168.99.100:9411/ (ConnectTimeoutException) with 2 occurrences migrated to:
  https://192.168.99.100:9411/ ([https](https://192.168.99.100:9411/) result ConnectTimeoutException).
* [ ] http://host.com:9999/ticker/12/basd (ConnectTimeoutException) with 1 occurrences migrated to:
  https://host.com:9999/ticker/12/basd ([https](https://host.com:9999/ticker/12/basd) result ConnectTimeoutException).
* [ ] http://compose.docker.io/ (UnknownHostException) with 1 occurrences migrated to:
  https://compose.docker.io/ ([https](https://compose.docker.io/) result UnknownHostException).
* [ ] http://non.existing.url (UnknownHostException) with 1 occurrences migrated to:
  https://non.existing.url ([https](https://non.existing.url) result UnknownHostException).
* [ ] http://cloud.spring.io/spring-cloud-static/Dalston.SR4/multi/multi__span_data_as_messages.html (404) with 1 occurrences migrated to:
  https://cloud.spring.io/spring-cloud-static/Dalston.SR4/multi/multi__span_data_as_messages.html ([https](https://cloud.spring.io/spring-cloud-static/Dalston.SR4/multi/multi__span_data_as_messages.html) result 404).
* [ ] http://docssleuth-zipkin-server.cfapps.io/dependency (404) with 4 occurrences migrated to:
  https://docssleuth-zipkin-server.cfapps.io/dependency ([https](https://docssleuth-zipkin-server.cfapps.io/dependency) result 404).
* [ ] http://www.erg.abdn.ac.uk/~gerrit/dccp/notes/ccid2/rto_estimator/ (301) with 1 occurrences migrated to:
  https://erg.abdn.ac.uk/~gerrit/dccp/notes/ccid2/rto_estimator/ ([https](https://www.erg.abdn.ac.uk/~gerrit/dccp/notes/ccid2/rto_estimator/) result 404).
* [ ] http://www.puppycrawl.com/dtds/suppressions_1_1.dtd (404) with 1 occurrences migrated to:
  https://www.puppycrawl.com/dtds/suppressions_1_1.dtd ([https](https://www.puppycrawl.com/dtds/suppressions_1_1.dtd) result 404).
* [ ] http://zipkin.io/pages/existing_instrumentations.html (301) with 1 occurrences migrated to:
  https://zipkin.io/pages/existing_instrumentations.html ([https](https://zipkin.io/pages/existing_instrumentations.html) result 404).

## Fixed Success
These URLs were switched to an https URL with a 2xx status. While the status was successful, your review is still recommended.

* [ ] http://asciidoctor.org with 1 occurrences migrated to:
  https://asciidoctor.org ([https](https://asciidoctor.org) result 200).
* [ ] http://bugs.jquery.com/ticket/1450 with 2 occurrences migrated to:
  https://bugs.jquery.com/ticket/1450 ([https](https://bugs.jquery.com/ticket/1450) result 200).
* [ ] http://bugs.jquery.com/ticket/5280 with 1 occurrences migrated to:
  https://bugs.jquery.com/ticket/5280 ([https](https://bugs.jquery.com/ticket/5280) result 200).
* [ ] http://cloud.spring.io with 2 occurrences migrated to:
  https://cloud.spring.io ([https](https://cloud.spring.io) result 200).
* [ ] http://docs.spring.io/spring-hateoas/docs/current/reference/html/ with 1 occurrences migrated to:
  https://docs.spring.io/spring-hateoas/docs/current/reference/html/ ([https](https://docs.spring.io/spring-hateoas/docs/current/reference/html/) result 200).
* [ ] http://docssleuth-service1.cfapps.io/start with 1 occurrences migrated to:
  https://docssleuth-service1.cfapps.io/start ([https](https://docssleuth-service1.cfapps.io/start) result 200).
* [ ] http://en.wikipedia.org/wiki/Mapping_of_Unicode_characters with 1 occurrences migrated to:
  https://en.wikipedia.org/wiki/Mapping_of_Unicode_characters ([https](https://en.wikipedia.org/wiki/Mapping_of_Unicode_characters) result 200).
* [ ] http://hacks.mozilla.org/2009/07/cross-site-xmlhttprequest-with-cors/ with 1 occurrences migrated to:
  https://hacks.mozilla.org/2009/07/cross-site-xmlhttprequest-with-cors/ ([https](https://hacks.mozilla.org/2009/07/cross-site-xmlhttprequest-with-cors/) result 200).
* [ ] http://opentracing.io/ with 3 occurrences migrated to:
  https://opentracing.io/ ([https](https://opentracing.io/) result 200).
* [ ] http://projects.spring.io/spring-integration/ with 1 occurrences migrated to:
  https://projects.spring.io/spring-integration/ ([https](https://projects.spring.io/spring-integration/) result 200).
* [ ] http://stackoverflow.com/questions/12817946/generate-a-random-bitset-with-n-1s with 1 occurrences migrated to:
  https://stackoverflow.com/questions/12817946/generate-a-random-bitset-with-n-1s ([https](https://stackoverflow.com/questions/12817946/generate-a-random-bitset-with-n-1s) result 200).
* [ ] http://stackoverflow.com/questions/1641507/detect-browser-support-for-cross-domain-xmlhttprequests with 1 occurrences migrated to:
  https://stackoverflow.com/questions/1641507/detect-browser-support-for-cross-domain-xmlhttprequests ([https](https://stackoverflow.com/questions/1641507/detect-browser-support-for-cross-domain-xmlhttprequests) result 200).
* [ ] http://stevesouders.com/misc/test-postmessage.php with 2 occurrences migrated to:
  https://stevesouders.com/misc/test-postmessage.php ([https](https://stevesouders.com/misc/test-postmessage.php) result 200).
* [ ] http://tbaggery.com/2008/04/19/a-note-about-git-commit-messages.html with 2 occurrences migrated to:
  https://tbaggery.com/2008/04/19/a-note-about-git-commit-messages.html ([https](https://tbaggery.com/2008/04/19/a-note-about-git-commit-messages.html) result 200).
* [ ] http://wiki.whatwg.org/wiki/Dynamic_Script_Execution_Order with 1 occurrences migrated to:
  https://wiki.whatwg.org/wiki/Dynamic_Script_Execution_Order ([https](https://wiki.whatwg.org/wiki/Dynamic_Script_Execution_Order) result 200).
* [ ] http://www.splunk.com/ with 2 occurrences migrated to:
  https://www.splunk.com/ ([https](https://www.splunk.com/) result 200).
* [ ] http://www.w3.org/TR/DOM-Level-2-Events/events.html with 1 occurrences migrated to:
  https://www.w3.org/TR/DOM-Level-2-Events/events.html ([https](https://www.w3.org/TR/DOM-Level-2-Events/events.html) result 200).
* [ ] http://htrace.incubator.apache.org/ with 2 occurrences migrated to:
  https://htrace.incubator.apache.org/ ([https](https://htrace.incubator.apache.org/) result 301).
* [ ] http://msdn.microsoft.com/en-us/library/cc197015 with 1 occurrences migrated to:
  https://msdn.microsoft.com/en-us/library/cc197015 ([https](https://msdn.microsoft.com/en-us/library/cc197015) result 301).
* [ ] http://msdn.microsoft.com/en-us/library/cc288060 with 1 occurrences migrated to:
  https://msdn.microsoft.com/en-us/library/cc288060 ([https](https://msdn.microsoft.com/en-us/library/cc288060) result 301).
* [ ] http://plugins.jetbrains.com/plugin/6546 with 2 occurrences migrated to:
  https://plugins.jetbrains.com/plugin/6546 ([https](https://plugins.jetbrains.com/plugin/6546) result 301).
* [ ] http://research.google.com/pubs/pub36356.html with 5 occurrences migrated to:
  https://research.google.com/pubs/pub36356.html ([https](https://research.google.com/pubs/pub36356.html) result 301).
* [ ] http://docsbrewing-zipkin-server.cfapps.io/ with 1 occurrences migrated to:
  https://docsbrewing-zipkin-server.cfapps.io/ ([https](https://docsbrewing-zipkin-server.cfapps.io/) result 302).
* [ ] http://docssleuth-zipkin-server.cfapps.io/ with 5 occurrences migrated to:
  https://docssleuth-zipkin-server.cfapps.io/ ([https](https://docssleuth-zipkin-server.cfapps.io/) result 302).
* [ ] http://eclipse.org with 1 occurrences migrated to:
  https://eclipse.org ([https](https://eclipse.org) result 302).
* [ ] http://eclipse.org/m2e/ with 2 occurrences migrated to:
  https://eclipse.org/m2e/ ([https](https://eclipse.org/m2e/) result 302).
* [ ] http://www.springsource.com/developer/sts with 1 occurrences migrated to:
  https://www.springsource.com/developer/sts ([https](https://www.springsource.com/developer/sts) result 302).

# Ignored
These URLs were intentionally ignored.

* http://127.0.0.1 with 1 occurrences
* http://abc with 1 occurrences
* http://asdasd:111/asd with 1 occurrences
* http://exceptionservice/ with 2 occurrences
* http://foo with 2 occurrences
* http://foo/ with 1 occurrences
* http://fooservice/ with 1 occurrences
* http://fooservice/nonExistent with 1 occurrences
* http://fooservice/noresponse with 1 occurrences
* http://fooservice/notrace with 9 occurrences
* http://fooservice/traceid with 1 occurrences
* http://localhost with 70 occurrences
* http://localhost/?foo=bar with 4 occurrences
* http://localhost:3379 with 1 occurrences
* http://localhost:3380 with 2 occurrences
* http://localhost:3380/async with 1 occurrences
* http://localhost:3380/call with 1 occurrences
* http://localhost:3381 with 1 occurrences
* http://localhost:80/abc with 1 occurrences
* http://localhost:8080/ with 1 occurrences
* http://localhost:9411 with 1 occurrences
* http://localhost:9411/ with 1 occurrences
* http://localhost:9978 with 1 occurrences
* http://localhost:9978/hello/mikesarver with 1 occurrences
* http://localhost:9987/unknown with 1 occurrences
* http://localhost:9988/sleuth/test-not-ok with 1 occurrences
* http://localhost:9998 with 1 occurrences
* http://localhost:9998/sleuth/test-not-ok with 1 occurrences
* http://localhost:9998/sleuth/test-ok with 1 occurrences
* http://some/address with 1 occurrences
* http://zipkin/ with 1 occurrences
* http://zipkin/call with 1 occurrences
* http://zipkin/hi2 with 1 occurrences
* http://zipkinserver/ with 2 occurrences
 2019-03-29 11:22:06 +01:00
Spring Operator
e6c9351118 URL Cleanup 
This commit updates URLs to prefer the https protocol. Redirects are not followed to avoid accidentally expanding intentionally shortened URLs (i.e. if using a URL shortener).

# Fixed URLs

## Fixed But Review Recommended
These URLs were fixed, but the https status was not OK. However, the https status was the same as the http request or http redirected to an https URL, so they were migrated. Your review is recommended.

* [ ] http://www.puppycrawl.com/dtds/suppressions_1_1.dtd (404) with 1 occurrences migrated to:
  https://www.puppycrawl.com/dtds/suppressions_1_1.dtd ([https](https://www.puppycrawl.com/dtds/suppressions_1_1.dtd) result 404).

## Fixed Success
These URLs were switched to an https URL with a 2xx status. While the status was successful, your review is still recommended.

* [ ] http://maven.apache.org/xsd/maven-4.0.0.xsd with 16 occurrences migrated to:
  https://maven.apache.org/xsd/maven-4.0.0.xsd ([https](https://maven.apache.org/xsd/maven-4.0.0.xsd) result 200).
* [ ] http://www.springframework.org/schema/beans/spring-beans.xsd with 1 occurrences migrated to:
  https://www.springframework.org/schema/beans/spring-beans.xsd ([https](https://www.springframework.org/schema/beans/spring-beans.xsd) result 200).
* [ ] http://www.springframework.org/schema/integration/spring-integration.xsd with 1 occurrences migrated to:
  https://www.springframework.org/schema/integration/spring-integration.xsd ([https](https://www.springframework.org/schema/integration/spring-integration.xsd) result 200).
* [ ] http://www.springframework.org/schema/task/spring-task.xsd with 1 occurrences migrated to:
  https://www.springframework.org/schema/task/spring-task.xsd ([https](https://www.springframework.org/schema/task/spring-task.xsd) result 200).

# Ignored
These URLs were intentionally ignored.

* http://maven.apache.org/POM/4.0.0 with 32 occurrences
* http://www.springframework.org/schema/beans with 2 occurrences
* http://www.springframework.org/schema/integration with 2 occurrences
* http://www.springframework.org/schema/task with 2 occurrences
* http://www.w3.org/2001/XMLSchema-instance with 17 occurrences
 2019-03-26 00:18:18 -05:00
wenqi.huang
40eff97856 Fix NullPointerException 2019-03-22 17:18:10 +08:00
Spring Operator
ac9774d8f7 URL Cleanup (#1279) 
This commit updates URLs to prefer the https protocol. Redirects are not followed to avoid accidentally expanding intentionally shortened URLs (i.e. if using a URL shortener).

# Fixed URLs

## Fixed Success
These URLs were switched to an https URL with a 2xx status. While the status was successful, your review is still recommended.

* [ ] http://www.apache.org/licenses/ with 1 occurrences migrated to:
  https://www.apache.org/licenses/ ([https](https://www.apache.org/licenses/) result 200).
* [ ] http://www.apache.org/licenses/LICENSE-2.0 with 338 occurrences migrated to:
  https://www.apache.org/licenses/LICENSE-2.0 ([https](https://www.apache.org/licenses/LICENSE-2.0) result 200).
 2019-03-22 09:56:20 +01:00
Marcin Grzejszczak
b98c4a8738 Greenwich 2019-03-08 16:45:43 +01:00
Marcin Grzejszczak
a9863a1e91 HTTP to HTTPS 2019-03-07 13:52:48 +01:00
buildmaster
3b4dc7b17a Bumping versions to 2.1.2.BUILD-SNAPSHOT after release 2019-03-05 20:06:52 +00:00
buildmaster
ce39889121 Going back to snapshots 2019-03-05 20:06:51 +00:00
buildmaster
cf138b8dcb Update SNAPSHOT to 2.1.1.RELEASE 2019-03-05 20:00:58 +00:00
Brian Devins-Suresh
b52723c0ce Rename test class so it runs (#1225) 2019-03-05 14:49:13 -05:00
Marcin Grzejszczak
f9e029a73f Ignored a flakey test 2019-02-28 15:39:30 +01:00
Marcin Grzejszczak
f044c07f67 NPE guard for null response; fixes gh-1224 2019-02-28 15:28:42 +01:00
Marcin Grzejszczak
fead7a35cc Trying to make tests less brittle 2019-02-28 14:57:16 +01:00
Marcin Grzejszczak
83545453ad Trying to make tests less brittle 2019-02-27 16:04:46 +01:00
Marcin Grzejszczak
b4174ffa82 Trying to make tests less brittle 2019-02-27 16:02:25 +01:00
Marcin Grzejszczak
34a4a10277 Trying to make tests less brittle 2019-02-27 15:24:02 +01:00
Marcin Grzejszczak
83f888ad24 Updated sc-build 2019-02-25 16:16:10 +01:00
Marcin Grzejszczak
cc75be4c55 Polish: 2019-02-21 16:26:49 +01:00
Marcin Grzejszczak
43ff547293 Merge branch '2.0.x' 2019-02-21 16:17:57 +01:00
Marcin Grzejszczak
f6bc3e0fc3 Added error tag for @Scheduled; fixes gh-1215 2019-02-21 16:11:48 +01:00