Fix for S3 Scraper

sonatype · Apr 25, 2013 · 8777c5c · 8777c5c
1 parent 639b8dc
commit 8777c5c
Showing 1 changed file with 5 additions and 4 deletions.
diff --git a/...ain/java/org/sonatype/nexus/proxy/maven/routing/internal/scrape/AmazonS3IndexScraper.java b/...ain/java/org/sonatype/nexus/proxy/maven/routing/internal/scrape/AmazonS3IndexScraper.java
@@ -80,6 +80,7 @@ protected List<String> diveIn( final ScrapeContext context, final Page page )
     {
         String prefix = null;
         Page initialPage = page;
+        String initialPageUrl = page.getUrl();
         if ( initialPage.getHttpResponse().getStatusLine().getStatusCode() != 200 )
         {
             // we probably have the NoSuchKey response from S3, usually when repo root is not in bucket root
@@ -93,16 +94,16 @@ protected List<String> diveIn( final ScrapeContext context, final Page page )
                 return null;
             }
             // repo.remoteUrl does not have query parameters...
-            String fixedUrl =
+            initialPageUrl =
                 context.getRemoteRepositoryRootUrl().substring( 0,
                     context.getRemoteRepositoryRootUrl().length() - prefix.length() );
-            getLogger().debug( "Retrying URL {} to scrape Amazon S3 hosted repository on remote URL {}", fixedUrl,
+            getLogger().debug( "Retrying URL {} to scrape Amazon S3 hosted repository on remote URL {}", initialPageUrl,
                 context.getRemoteRepositoryRootUrl() );
-            initialPage = Page.getPageFor( context, fixedUrl + "?prefix=" + prefix );
+            initialPage = Page.getPageFor( context, initialPageUrl + "?prefix=" + prefix );
         }
 
         final HashSet<String> entries = new HashSet<String>();
-        diveIn( context, initialPage, initialPage.getUrl(), prefix, entries );
+        diveIn( context, initialPage, initialPageUrl, prefix, entries );
         return new ArrayList<String>( entries );
     }