Skip to content
New issue

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

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

Already on GitHub? # to your account

Upgrade to Spring boot 3 #50

Merged
merged 1 commit into from
Jul 10, 2023
Merged

Upgrade to Spring boot 3 #50

merged 1 commit into from
Jul 10, 2023

Conversation

ghoshrahul
Copy link
Contributor

Pull request for this issue

* Use jakarta api instead of javax api
* Change the source and target to 1.8
* Fix test cases
* Date format changes. Refer: spring-projects/spring-framework#16531
* Documentation changes
* Version changes
@ghoshrahul ghoshrahul marked this pull request as ready for review April 4, 2023 05:00
@zychu312
Copy link

@tomdesair Merge is highly anticipated. Any updates/plans on this topic?

@mich9463
Copy link

Since this change would prohibit deployment to any pre Jakarta-Servlet 6.0 servlet container, like Tomcat 9, I'd propose a larger bump in version (maybe 1.1).

@ghoshrahul ghoshrahul changed the title Upgrade to java 17 Upgrade to Spring boot 3 May 9, 2023

You can add the latest stable version of this library to your application using Maven by adding the following dependency:

<dependency>
<groupId>me.desair.tus</groupId>
<artifactId>tus-java-server</artifactId>
<version>1.0.0-2.0</version>
<version>1.0.0-3.0-SNAPSHOT</version>
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Should the version be 1.0.0-3.0, since this is a breaking change

<maven.compiler.source>1.7</maven.compiler.source>
<maven.compiler.target>1.7</maven.compiler.target>
<maven.compiler.source>1.8</maven.compiler.source>
<maven.compiler.target>1.8</maven.compiler.target>
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

If we go for the breaking change, then the compiler source and target should be 17?

<scope>test</scope>
</dependency>
<dependency>
Copy link
Contributor Author

@ghoshrahul ghoshrahul May 9, 2023

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Spring test no longer includes spring-web, hence this was externalized

@ksvraja
Copy link

ksvraja commented Jul 8, 2023

Hi @ghoshrahul
I have tried the similar upgrade (for jakarta update), and did not find any need to update the logic in - src/test/java/me/desair/tus/server/creation/ITCreationExtension.java file.

Could you describe the change required for notnull --> any check.

fyi- I also faced the dateformat changes.

@ghoshrahul
Copy link
Contributor Author

Hi @ksvraja ,

Along with the upgrade to spring boot 3, I have also upgraded mockito framework as well from 2.23.4 to 4.8.1

As per the mockito documentation, the notNull(Class<T> clazz) was deprecated and removed from version 3 onwards.

You can read the changes as presented in the source code

Date format changes are required, because spring has made changes to the MockHttpServletResponse class, for which the link is shared in the first comment itself.

@tomdesair
Copy link
Owner

Thank you @ghoshrahul for all the hard work and apologies for the late reply. I'll merge your changes, give this project's CI/CD a refresh and get a new release out of the door.

@tomdesair tomdesair merged commit ce33980 into tomdesair:master Jul 10, 2023
@ghoshrahul
Copy link
Contributor Author

Thanks @tomdesair .

Looking forward to the next release of the tus-java-server :)

@gagniou
Copy link

gagniou commented Jul 17, 2023

hi,

do you have any idea when will the release take place ? I am waiting the 1.0.0-3.0 for a project in springboot 3

@tomdesair
Copy link
Owner

This has now been released to Maven Central as version 1.0.0-3.0: https://repo1.maven.org/maven2/me/desair/tus/tus-java-server/1.0.0-3.0/

# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants