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

Draft pull request for 1.5 release #331

Merged
merged 43 commits into from
Apr 8, 2020
Merged

Draft pull request for 1.5 release #331

merged 43 commits into from
Apr 8, 2020

Conversation

sapessi
Copy link
Collaborator

@sapessi sapessi commented Mar 31, 2020

Description of changes: Draft pull request for 1.5 release. The major changes are: A fix for a race condition that caused responses to be flushed before asynchronous processing completed; A new option to force SpringBoot 2 to start a servlet-only web server.

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

dependabot bot and others added 16 commits March 28, 2020 17:04
Bumps `spring.version` from 5.1.9.RELEASE to 5.2.3.RELEASE.

Updates `spring-webmvc` from 5.1.9.RELEASE to 5.2.3.RELEASE
- [Release notes](https://github.com/spring-projects/spring-framework/releases)
- [Commits](spring-projects/spring-framework@v5.1.9.RELEASE...v5.2.3.RELEASE)

Updates `spring-test` from 5.1.9.RELEASE to 5.2.3.RELEASE
- [Release notes](https://github.com/spring-projects/spring-framework/releases)
- [Commits](spring-projects/spring-framework@v5.1.9.RELEASE...v5.2.3.RELEASE)

Signed-off-by: dependabot[bot] <support@github.com>

Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Bumps [spring-webflux](https://github.com/spring-projects/spring-framework) from 5.1.9.RELEASE to 5.2.0.RELEASE.
- [Release notes](https://github.com/spring-projects/spring-framework/releases)
- [Commits](spring-projects/spring-framework@v5.1.9.RELEASE...v5.2.0.RELEASE)

Signed-off-by: dependabot[bot] <support@github.com>

Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
…f the spring-security package used in the tests
* fix issue 317 - use charset from request

* update dependencies

* update build dependencies, remove spring boot 2.0.x

* restoring ci config

Co-authored-by: Stefano Buliani <2996317+sapessi@users.noreply.github.com>
…tests and updated servlet tests to use the new servletApplication option
…uest asynchronously. This was causing race conditions in the SpringBoot 2 WebFlux implementation - requests that had to run through security or validation filters took longer and the library flushed an empty request, which caused the status code to default to 200. This fix addresses issues #279, #304, and #306
… dependency to truly support Servlet-only server
…ler that tells the framework whether to start a reactive or servlet-based embedded server. Also added a new servletApplication method to the builder object.
…s property is null. Unlikely outside of tests but better safe than sorry. This addresses #327
@sapessi sapessi added this to the Release 1.5 milestone Mar 31, 2020
…quests load on startup are initialized right away, as part of the initialization() method call in LambdaServletContainerHandler. Also centralized the lazy Servlet initialization to the ServletExecutionFilter so that we don't have code scattered all around. This begins to address #287
…es the actual JVM start time to calculate the timeout milliseconds. Also added the new method to the builder object and deprecated the current method that receives a milliseconds epoch parameter. I'm not deprecating the constructor of the async initializer class that receives the parameter as it may still be useful for tests. This change was suggested in #287
…e since the JVM is reused for both tests in the and we cannot reuse the actual JVM init time
…r, and security context writer - continuing to address #329
…vletRequest rather than specific implementations of it. This makes it easier to create container handler implementations that support HTTP API, API Gateway, and ALB (#329)
@sapessi sapessi changed the title [Do No Merge] Draft pull request for 1.5 release Draft pull request for 1.5 release Apr 8, 2020
@sapessi
Copy link
Collaborator Author

sapessi commented Apr 8, 2020

Merging this pull requests. All issues are tested and CI succeeds.

@sapessi sapessi merged commit 4380cc0 into master Apr 8, 2020
@sapessi sapessi deleted the v1.5 branch October 6, 2020 18:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment