compare tarball string as well in the generation phase to prevent bad…

… tarball url (#2487)

* check tarball string as well in the generation phase

* correct tarball path compare

* update to preserve Dist content while update it's path only

* add detection of null in dist

* add setter

addons/pkg-npm/common/src/main/java/org/commonjava/indy/pkg/npm/content/PackageMetadataGenerator.java

@@ -321,11 +321,19 @@ private boolean writePackageMetadata( List<StoreResource> firstLevelFiles, Artif
                 }
 
                 // Generate tarball url if missing
+                String tarball = "http://indy/" + packagePath.getTarPath(); // here we use mock host. indy will amend it with the right hostname
                 if ( versionMetadata.getDist() == null )
                 {
-                    String tarball = "http://indy/" + packagePath.getTarPath(); // here we use mock host. indy will amend it with the right hostname
                     //logger.debug( "Generate dist tarball: {}", tarball );
-                    versionMetadata.setDist( new Dist(  tarball ) );
+                    versionMetadata.setDist( new Dist( tarball ) );
+                } else {
+                    // Detect bad tarball and replace with correct one
+                    if ( versionMetadata.getDist().getTarball() == null || !versionMetadata.getDist().getTarball().endsWith(packagePath.getTarPath()) )
+                    {
+                        Dist dist = versionMetadata.getDist();
+                        dist.setTarball( tarball );
+                        versionMetadata.setDist( dist );
+                    }
                 }
             }
             catch ( IOException e )

addons/pkg-npm/model-java/src/main/java/org/commonjava/indy/pkg/npm/model/Dist.java

@@ -25,7 +25,7 @@ public class Dist
 
     private final String shasum;
 
-    private final String tarball;
+    private String tarball;
 
     private final String integrity;
 
@@ -69,6 +69,11 @@ public String getTarball()
         return tarball;
     }
 
+    public void setTarball( String tarball )
+    {
+        this.tarball = tarball;
+    }
+
     public String getIntegrity()
     {
         return integrity;