Skip to content

Commit 333c777

Browse files
authored
Fix for maven conditional upload scenarios (#835)
1 parent 4514999 commit 333c777

1 file changed

Lines changed: 9 additions & 2 deletions

File tree

build-info-extractor-maven3/src/main/java/org/jfrog/build/extractor/maven/BuildInfoRecorder.java

Lines changed: 9 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -470,8 +470,15 @@ private void addModuleToBuild(MavenProject project, ModuleBuilder module) {
470470
private void addArtifactsToCurrentModule(MavenProject project, ModuleBuilder module) {
471471
addDefaultPublisherAttributes(conf, project.getName(), "Maven", project.getVersion());
472472

473-
// Check if artifact collection is disabled for build info
474-
if (!conf.publisher.isPublishArtifacts()) {
473+
// Check if artifact metadata is explicitly requested for scanning/detailed summary
474+
boolean artifactMetadataRequested = conf.publisher.shouldAddDeployableArtifacts();
475+
boolean publishArtifacts = conf.publisher.isPublishArtifacts();
476+
477+
// Skip artifact collection only if BOTH conditions are true:
478+
// 1. Publishing is disabled (publish.artifacts=false) AND
479+
// 2. Artifact metadata is NOT explicitly requested (no deployable.artifacts.map)
480+
// This preserves the mvn verify fix while supporting conditional upload scenarios
481+
if (!publishArtifacts && !artifactMetadataRequested) {
475482
logger.info("Artifact publishing is disabled - skipping artifact collection for build info");
476483
return;
477484
}

0 commit comments

Comments
 (0)