Skip to content

Conversation

@zefir-git
Copy link
Contributor

@zefir-git zefir-git commented Dec 16, 2025

When Maven resolves the dependency, it sends the artifactId as part of the .jar and .pom file name in whatever case was specified for the dependency.

The problem is that the file name is in the case that the user uploaded it (usually the same as their original artifactId), and the .pom file generated by modrinth is always lowercase. This means that unless your artifactId is in lowercase ONLY and the uploaded file is also only in lowercase, Modrinth will fail to serve the correct file.

This PR changes the file name comparison to be case-insensitive.

@Prospector Prospector requested a review from a team December 19, 2025 21:03
Copy link
Contributor

@aecsocket aecsocket left a comment

Choose a reason for hiding this comment

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

LGTM, just make sure CI passes please

@zefir-git zefir-git requested a review from aecsocket January 19, 2026 16:09
@Prospector Prospector enabled auto-merge January 19, 2026 21:59
@Prospector Prospector added this pull request to the merge queue Jan 21, 2026
Merged via the queue into modrinth:main with commit 08fcc61 Jan 21, 2026
4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

3 participants