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

Improve the checks for relative traversal. #158

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

LQxdu
Copy link

@LQxdu LQxdu commented Oct 17, 2024

Use java.nio.file.Path for consistent sub-directory checking

Use java.nio.file.Path for consistent sub-directory checking
@nemecec
Copy link
Collaborator

nemecec commented Oct 17, 2024

Currently, zt-zip is compatible with Java 6. This patch uses Java 7 API. We have to ponder whether it is time to bump the minimum Java requirement from Java 6 to Java 8.

@LQxdu
Copy link
Author

LQxdu commented Oct 17, 2024

Based on recent statistical research, such as reports from InfoWorld and New Relic, Java 8, 11, and 17 are the most widely used versions, while the number of users still relying on Java 6 and 7 has significantly decreased. For instance, according to the New Relic report, only 0.28% of applications are still using Java 7 in production.

Therefore, I believe it is worth considering raising the minimum Java requirement from Java 6 to Java 8.

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.

2 participants