Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Upgrade to spring-cloud:2021.0.0 / spring-boot:2.6.1 #127

Merged
merged 10 commits into from
Dec 27, 2021

Conversation

groldan
Copy link
Member

@groldan groldan commented Dec 24, 2021

Spring-cloud upgrade Hoxton.SR8 -> 2021.0.0

  • spring-cloud: Hoxton.SR8 (2.2.5.RELEASE) -> 2021.0.0
  • spring-boot: 2.3.3.RELEASE -> 2.6.1
  • feign-reactor:2.0.26 -> 3.1.5

Add junit-vintage-engine test dependency alongside
spring-boot-starter-test, spring-boot comes now with
only the junit jupiter engine by default.

Adapt to spring-boot 2.6.1 config:

  • Move custom server.instance-id property to info.instance-id
  • Replace non working @PostConstruct annotations
  • Add allow-circular-references: true to all geoserver related services, it's false by default since spring-boot 2.6.0, breaks geoserver initialization.

* spring-cloud: Hoxton.SR8 (2.2.5.RELEASE) -> 2021.0.0
* spring-boot: 2.3.3.RELEASE -> 2.6.1
* feign-reactor:2.0.26 -> 3.1.5

Add junit-vintage-engine test dependency alongside
spring-boot-starter-test, spring-boot comes now with
only the junit jupiter engine by default.
- Move custom `server.instance-id` property to `info.instance-id`
- Build config's docker image using packeto buildpacks, and experiment
  building a native version.
- Replace non working @PostConstruct annotations
- Add `allow-circular-references: true` to all geoserver related services,
  it's false by default since spring-boot 2.6.0, breaks geoserver initialization.
Tackling spring-boot-maven-plugin build-image builds as
a separate issue.
@groldan groldan force-pushed the dependencies/spring-cloud_2021.0.0 branch from 5f5df97 to 34510fc Compare December 24, 2021 11:56
@groldan groldan marked this pull request as ready for review December 27, 2021 15:20
…ntentNegotiationStrategy

Override `PutIgnoringExtensionContentNegotiationStrategy` defined in
`StyleController$StyleControllerConfiguration` as it causes a
```
java.lang.IllegalArgumentException: Expected lookupPath in request attribute
org.springframework.web.util.UrlPathHelper.PATH".
```
exception for `/actuator/*` requests.
@groldan groldan force-pushed the dependencies/spring-cloud_2021.0.0 branch from ffa8d9b to a9410e5 Compare December 27, 2021 15:25
@groldan groldan merged commit 8b95817 into geoserver:main Dec 27, 2021
@groldan groldan deleted the dependencies/spring-cloud_2021.0.0 branch December 27, 2021 15:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant