Skip to content

🦺 fix: Force local checksum mode on validate #1282

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.

Sign up for GitHub

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

Open
wants to merge 2 commits into
base: main
Choose a base branch
from
Open

🦺 fix: Force local checksum mode on validate #1282

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
2 commits
Select commit Hold shift + click to select a range
20150b2
Force local checksum mode on validate
LogFlames Jul 31, 2025
ec15b59
📝 Update Documentation with current version
LogFlames Jul 31, 2025
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
14 changes: 12 additions & 2 deletions maven_plugin/src/main/java/io/github/chains_project/maven_lockfile/AbstractLockfileMojo.java
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -93,10 +93,20 @@ protected AbstractChecksumCalculator getChecksumCalculator() throws MojoExecutio
}

protected AbstractChecksumCalculator getChecksumCalculator(Config config) throws MojoExecutionException {
return getChecksumCalculator(config, false);
}

protected AbstractChecksumCalculator getChecksumCalculator(Config config, boolean forceLocalChecksumMode)
throws MojoExecutionException {
ProjectBuildingRequest artifactBuildingRequest = newResolveArtifactProjectBuildingRequest();
ProjectBuildingRequest pluginBuildingRequest = newResolvePluginProjectBuildingRequest();

switch (checksumModeDeprecation(config.getChecksumMode())) {
String checksumMode = checksumModeDeprecation(config.getChecksumMode());
if (forceLocalChecksumMode) {
checksumMode = "local";
}

switch (checksumMode) {
case "local":
return new FileSystemChecksumCalculator(
dependencyResolver,
Expand All @@ -107,7 +117,7 @@ protected AbstractChecksumCalculator getChecksumCalculator(Config config) throws
return new RemoteChecksumCalculator(
config.getChecksumAlgorithm(), artifactBuildingRequest, pluginBuildingRequest);
default:
throw new MojoExecutionException("Invalid checksum mode: " + config.getChecksumMode());
throw new MojoExecutionException("Invalid checksum mode: " + checksumMode);
}
}

Expand Down
2 changes: 1 addition & 1 deletion maven_plugin/src/main/java/io/github/chains_project/maven_lockfile/ValidateChecksumMojo.java
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -45,7 +45,7 @@ public void execute() throws MojoExecutionException {
getLog().warn("No config was found in the lock file. Using default config.");
}
MetaData metaData = new MetaData(environment, config);
AbstractChecksumCalculator checksumCalculator = getChecksumCalculator(config);
AbstractChecksumCalculator checksumCalculator = getChecksumCalculator(config, true);
LockFile lockFileFromProject = LockFileFacade.generateLockFileFromProject(
session, project, dependencyCollectorBuilder, checksumCalculator, metaData);
if (!Objects.equals(lockFileFromFile.getEnvironment(), lockFileFromProject.getEnvironment())) {
Expand Down
Loading