Skip to content

Commit 3e20aa2

Browse files
gregturncfieber
authored andcommitted
Upgrade to Spring Boot 1.4
Using the update to spinnaker-dependencies, migrate kork to Spring Boot 1.4 and related dependencies NOTE: Spring Boot 1.4 automatically uses Tomcat 8.5.5. In 8.5, several APIs were refactored including JSSESocketFactory. This patch pins Tomcat to 8.0.24, latest stable version of 8.0 until this refactoring can be dealt with.
1 parent a87a9c0 commit 3e20aa2

File tree

5 files changed

+23
-6
lines changed

5 files changed

+23
-6
lines changed

build.gradle

Lines changed: 15 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -15,7 +15,6 @@
1515
*/
1616

1717
buildscript {
18-
1918
repositories {
2019
jcenter()
2120
maven { url 'http://dl.bintray.com/spinnaker/gradle/' }
@@ -33,7 +32,7 @@ allprojects {
3332
group = 'com.netflix.spinnaker.kork'
3433

3534
ext {
36-
spinnakerDependenciesVersion = project.hasProperty('spinnakerDependenciesVersion') ? project.property('spinnakerDependenciesVersion') : '0.57.0'
35+
spinnakerDependenciesVersion = project.hasProperty('spinnakerDependenciesVersion') ? project.property('spinnakerDependenciesVersion') : '0.61.0'
3736
}
3837

3938
def checkLocalVersions = [spinnakerDependenciesVersion: spinnakerDependenciesVersion]
@@ -59,4 +58,18 @@ allprojects {
5958
jacoco {
6059
toolVersion = '0.7.7.201606060606'
6160
}
61+
62+
configurations {
63+
all {
64+
resolutionStrategy {
65+
eachDependency {
66+
if (it.requested.group.startsWith('org.apache.tomcat')) {
67+
it.useVersion spinnaker.version('tomcat')
68+
}
69+
}
70+
}
71+
}
72+
}
73+
74+
6275
}

kork-core/src/main/java/com/netflix/spinnaker/kork/metrics/SpectatorConfiguration.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -55,7 +55,7 @@ MetricWriter spectatorMetricWriter(Registry registry) {
5555

5656
@Bean
5757
@Primary
58-
@ConditionalOnMissingClass(name = "org.springframework.messaging.MessageChannel")
58+
@ConditionalOnMissingClass("org.springframework.messaging.MessageChannel")
5959
@ConditionalOnMissingBean(name = "primaryMetricWriter")
6060
public MetricWriter primaryMetricWriter(List<MetricWriter> writers) {
6161
return new CompositeMetricWriter(writers);

kork-swagger/kork-swagger.gradle

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,6 @@
11
dependencies {
22
compile spinnaker.dependency('bootAutoConfigure')
3-
compile "io.springfox:springfox-swagger2:2.2.2"
4-
compile "io.springfox:springfox-swagger-ui:2.2.2"
5-
compile "com.wordnik:swagger-annotations:1.3.13"
3+
compile "io.springfox:springfox-swagger2:2.6.0"
4+
compile "io.springfox:springfox-swagger-ui:2.6.0"
5+
compile "io.swagger:swagger-annotations:1.5.10"
66
}

kork-web/kork-web.gradle

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -11,6 +11,7 @@ dependencies {
1111

1212
compileOnly spinnaker.dependency('bootActuator')
1313

14+
spinnaker.group('tomcatEmbedded')
1415
spinnaker.group('spockBase')
1516
}
1617

kork-web/src/main/groovy/com/netflix/spinnaker/tomcat/x509/BlacklistingJSSESocketFactory.java

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -23,6 +23,9 @@
2323
import javax.net.ssl.X509TrustManager;
2424
import java.util.Optional;
2525

26+
/**
27+
* TODO: Handle Tomcat 8.5 refactoring that removes low level JSSESocketFactory. See https://github.com/spring-projects/spring-boot/issues/6164 for more details.
28+
*/
2629
public class BlacklistingJSSESocketFactory extends JSSESocketFactory {
2730
private static final String BLACKLIST_PREFIX = "blacklist:";
2831

0 commit comments

Comments
 (0)