Imported Upstream version 17.23.2
[platform/upstream/libzypp.git] / zypp / Fetcher.cc
index 7496513..ce2f614 100644 (file)
  *
 */
 #include <iostream>
+#include <fstream>
 #include <list>
+#include <map>
 
-#include "zypp/base/Logger.h"
+#include "zypp/base/Easy.h"
+#include "zypp/base/LogControl.h"
+#include "zypp/base/LogTools.h"
+#include "zypp/base/PtrTypes.h"
 #include "zypp/base/DefaultIntegral.h"
+#include "zypp/base/String.h"
 #include "zypp/Fetcher.h"
+#include "zypp/ZYppFactory.h"
+#include "zypp/CheckSum.h"
 #include "zypp/base/UserRequestException.h"
+#include "zypp/parser/susetags/ContentFileReader.h"
+#include "zypp/parser/susetags/RepoIndex.h"
 
-using namespace std;
+#undef ZYPP_BASE_LOGGER_LOGGROUP
+#define ZYPP_BASE_LOGGER_LOGGROUP "zypp:fetcher"
 
 ///////////////////////////////////////////////////////////////////
 namespace zypp
 { /////////////////////////////////////////////////////////////////
 
   /**
+   * class that represents indexes which add metadata
+   * to fetcher jobs and therefore need to be retrieved
+   * in advance.
+   */
+  struct FetcherIndex
+  {
+    FetcherIndex( const OnMediaLocation &loc )
+      : location(loc)
+    {}
+    /** Index localtion. */
+    OnMediaLocation             location;
+    /** Whether we read this index. */
+    DefaultIntegral<bool,false> read;
+  };
+
+  typedef shared_ptr<FetcherIndex> FetcherIndex_Ptr;
+
+  /** std::set ordering (less semantic) */
+  struct SameFetcherIndex
+  {
+    bool operator()( const FetcherIndex_Ptr & lhs, const FetcherIndex_Ptr & rhs ) const
+    {
+      if ( lhs == rhs )
+        return false; // incl. NULL == NULL
+      if ( ! lhs )
+        return true;  // NULL < nonNULL
+      if ( ! rhs )
+        return false; // nonNULL > NULL
+      // both nonNULL ==> compare medianr and path
+      if ( lhs->location.medianr() == rhs->location.medianr() )
+        return lhs->location.filename() < rhs->location.filename();
+      //else
+        return lhs->location.medianr() < rhs->location.medianr();
+    }
+  };
+
+  /**
    * Class to encapsulate the \ref OnMediaLocation object
-   * and the \ref FileChcker together
+   * and the \ref FileChecker together
    */
   struct FetcherJob
   {
-    FetcherJob( const OnMediaLocation &loc )
+    enum Flag
+    {
+        None      = 0x0000,
+        Directory = 0x0001,
+        Recursive = 0x0002,
+        RecursiveDirectory = Directory | Recursive,
+        // check checksums even if there is no such
+        // checksum (warns of no checksum)
+        AlwaysVerifyChecksum = 0x0004,
+    };
+    ZYPP_DECLARE_FLAGS(Flags, Flag);
+
+
+    FetcherJob( const OnMediaLocation &loc, const Pathname dfile = Pathname())
       : location(loc)
+      , deltafile(dfile)
+      , flags(None)
     {
+      //MIL << location << endl;
+    }
 
+    ~FetcherJob()
+    {
+      //MIL << location << " | * " << checkers.size() << endl;
     }
 
     OnMediaLocation location;
-    CompositeFileChecker checkers;
+    Pathname deltafile;
+    //CompositeFileChecker checkers;
+    std::list<FileChecker> checkers;
+    Flags flags;
   };
 
+  ZYPP_DECLARE_OPERATORS_FOR_FLAGS(FetcherJob::Flags);
+  typedef shared_ptr<FetcherJob> FetcherJob_Ptr;
+
+  std::ostream & operator<<( std::ostream & str, const FetcherJob_Ptr & obj )
+  {
+    return str << obj->location;
+  }
+
   ///////////////////////////////////////////////////////////////////
   //
   //   CLASS NAME : Fetcher::Impl
   //
   /** Fetcher implementation. */
-  struct Fetcher::Impl
+  class Fetcher::Impl
   {
+    friend std::ostream & operator<<( std::ostream & str, const Fetcher::Impl & obj );
 
   public:
+    Impl();
+
+    ~Impl() {}
+
+    void setOptions( Fetcher::Options options );
+    Fetcher::Options options() const;
+
+    void addIndex( const OnMediaLocation &resource );
+
+    void enqueueDir( const OnMediaLocation &resource, bool recursive, const FileChecker &checker = FileChecker() );
+    void enqueueDigestedDir( const OnMediaLocation &resource, bool recursive, const FileChecker &checker = FileChecker() );
 
-    void enqueue( const OnMediaLocation &resource, const FileChecker &checker  );
-    void enqueueDigested( const OnMediaLocation &resource, const FileChecker &checker );
+    void enqueue( const OnMediaLocation &resource, const FileChecker &checker = FileChecker()  );
+    void enqueueDigested( const OnMediaLocation &resource, const FileChecker &checker = FileChecker(), const Pathname &deltafile = Pathname() );
     void addCachePath( const Pathname &cache_dir );
     void reset();
     void start( const Pathname &dest_dir,
@@ -63,6 +154,64 @@ namespace zypp
       static shared_ptr<Impl> _nullimpl( new Impl );
       return _nullimpl;
     }
+  private:
+      /**
+       * download the indexes and reads them
+       */
+      void downloadAndReadIndexList( MediaSetAccess &media, const Pathname &dest_dir);
+
+      /**
+       * download the indexes and reads them
+       */
+      void downloadIndex( MediaSetAccess &media, const OnMediaLocation &resource, const Pathname &dest_dir);
+
+      /**
+       * reads a downloaded index file and updates internal
+       * attributes table
+       *
+       * The index lists files relative to a directory, which is
+       * normally the same as the index file is located.
+       */
+      void readIndex( const Pathname &index, const Pathname &basedir );
+
+      /** specific version of \ref readIndex for CHECKSUMS file */
+      void readChecksumsIndex( const Pathname &index, const Pathname &basedir );
+
+      /** specific version of \ref readIndex for content file */
+      void readContentFileIndex( const Pathname &index, const Pathname &basedir );
+
+      /** reads the content of a directory but keeps a cache **/
+      void getDirectoryContent( MediaSetAccess &media, const OnMediaLocation &resource, filesystem::DirContent &content );
+
+      /**
+       * Tries to locate the file represented by job by looking at
+       * the cache (matching checksum is mandatory). Returns the
+       * location of the cached file or an empty \ref Pathname.
+       */
+      Pathname locateInCache( const OnMediaLocation & resource_r, const Pathname & destDir_r );
+      /**
+       * Validates the provided file against its checkers.
+       * \throws Exception
+       */
+      void validate( const Pathname & localfile_r, const std::list<FileChecker> & checkers_r );
+
+      /**
+       * scan the directory and adds the individual jobs
+       */
+       void addDirJobs( MediaSetAccess &media, const OnMediaLocation &resource,
+                        const Pathname &dest_dir, FetcherJob::Flags flags );
+
+      /**
+       * auto discovery and reading of indexes
+       */
+      void autoaddIndexes( const filesystem::DirContent &content,
+                           MediaSetAccess &media,
+                           const OnMediaLocation &resource,
+                           const Pathname &dest_dir );
+      /**
+       * Provide the resource to \ref dest_dir
+       */
+      void provideToDest( MediaSetAccess & media_r, const Pathname & destDir_r , const FetcherJob_Ptr & jobp_r );
 
   private:
     friend Impl * rwcowClone<Impl>( const Impl * rhs );
@@ -70,147 +219,584 @@ namespace zypp
     Impl * clone() const
     { return new Impl( *this ); }
 
-    std::list<FetcherJob> _resources;
-    std::list<Pathname> _caches;
+    std::list<FetcherJob_Ptr>   _resources;
+    std::set<FetcherIndex_Ptr,SameFetcherIndex> _indexes;
+    std::set<Pathname> _caches;
+    // checksums read from the indexes
+    std::map<std::string, CheckSum> _checksums;
+    // cache of dir contents
+    std::map<std::string, filesystem::DirContent> _dircontent;
+
+    Fetcher::Options _options;
   };
   ///////////////////////////////////////////////////////////////////
 
+  void Fetcher::Impl::enqueueDigested( const OnMediaLocation &resource, const FileChecker &checker, const Pathname &deltafile )
+  {
+    FetcherJob_Ptr job;
+    job.reset(new FetcherJob(resource, deltafile));
+    job->flags |= FetcherJob:: AlwaysVerifyChecksum;
+    _resources.push_back(job);
+  }
 
-  void Fetcher::Impl::enqueueDigested( const OnMediaLocation &resource, const FileChecker &checker )
+  Fetcher::Impl::Impl()
+      : _options(0)
   {
-    CompositeFileChecker composite;
-    composite.add(ChecksumFileChecker(resource.checksum()));
-    composite.add(checker);
-    enqueue(resource, composite);
   }
-  
+
+  void Fetcher::Impl::setOptions( Fetcher::Options options )
+  { _options = options; }
+
+  Fetcher::Options Fetcher::Impl::options() const
+  { return _options; }
+
+  void Fetcher::Impl::enqueueDir( const OnMediaLocation &resource,
+                                  bool recursive,
+                                  const FileChecker &checker )
+  {
+    FetcherJob_Ptr job;
+    job.reset(new FetcherJob(resource));
+    if ( checker )
+        job->checkers.push_back(checker);
+    if ( recursive )
+        job->flags |= FetcherJob::Recursive;
+    job->flags |= FetcherJob::Directory;
+
+    _resources.push_back(job);
+  }
+
+  void Fetcher::Impl::enqueueDigestedDir( const OnMediaLocation &resource,
+                                          bool recursive,
+                                          const FileChecker &checker )
+  {
+    FetcherJob_Ptr job;
+    job.reset(new FetcherJob(resource));
+    if ( checker )
+        job->checkers.push_back(checker);
+    if ( recursive )
+        job->flags |= FetcherJob::Recursive;
+    job->flags |= FetcherJob::Directory;
+    job->flags |= FetcherJob::AlwaysVerifyChecksum;
+
+    _resources.push_back(job);
+
+  }
+
   void Fetcher::Impl::enqueue( const OnMediaLocation &resource, const FileChecker &checker )
   {
-    FetcherJob job(resource);
-    job.checkers.add(checker);
-    _resources.push_back(resource);
+    FetcherJob_Ptr job;
+    job.reset(new FetcherJob(resource));
+    if ( checker )
+      job->checkers.push_back(checker);
+    _resources.push_back(job);
   }
 
+  void Fetcher::Impl::addIndex( const OnMediaLocation &resource )
+  {
+    MIL << "adding index " << resource << endl;
+    _indexes.insert(FetcherIndex_Ptr(new FetcherIndex(resource)));
+  }
+
+
   void Fetcher::Impl::reset()
   {
     _resources.clear();
-    _caches.clear();
+    _indexes.clear();
+    _checksums.clear();
+    _dircontent.clear();
   }
-  
+
   void Fetcher::Impl::addCachePath( const Pathname &cache_dir )
   {
-    _caches.push_back(cache_dir);
+    PathInfo info(cache_dir);
+    if ( info.isExist() )
+    {
+      if ( info.isDir() )
+      {
+        DBG << "Adding fetcher cache: '" << cache_dir << "'." << endl;
+        _caches.insert(cache_dir);
+      }
+      else
+      {
+        // don't add bad cache directory, just log the error
+        ERR << "Not adding cache: '" << cache_dir << "'. Not a directory." << endl;
+      }
+    }
+    else
+    {
+        ERR << "Not adding cache '" << cache_dir << "'. Path does not exists." << endl;
+    }
+
   }
-  
-  void Fetcher::Impl::start( const Pathname &dest_dir,
-                             MediaSetAccess &media,
-                             const ProgressData::ReceiverFnc & progress_receiver )
+
+  Pathname Fetcher::Impl::locateInCache( const OnMediaLocation & resource_r, const Pathname & destDir_r )
   {
-    ProgressData progress(_resources.size());
-    progress.sendTo(progress_receiver);
+    Pathname ret;
+    // No checksum - no match
+    if ( resource_r.checksum().empty() )
+      return ret;
 
-    for ( list<FetcherJob>::const_iterator it_res = _resources.begin(); it_res != _resources.end(); ++it_res )
+    // first check in the destination directory
+    Pathname cacheLocation = destDir_r / resource_r.filename();
+    if ( PathInfo(cacheLocation).isExist() && is_checksum( cacheLocation, resource_r.checksum() ) )
     {
-      bool got_from_cache = false;
-      for ( list<Pathname>::const_iterator it_cache = _caches.begin(); it_cache != _caches.end(); ++it_cache )
+      swap( ret, cacheLocation );
+      return ret;
+    }
+
+    MIL << "start fetcher with " << _caches.size() << " cache directories." << endl;
+    for( const Pathname & cacheDir : _caches )
+    {
+      cacheLocation = cacheDir / resource_r.filename();
+      if ( PathInfo(cacheLocation).isExist() && is_checksum( cacheLocation, resource_r.checksum() ) )
       {
-        // Pathinfos could be cached to avoid too many stats?
-        PathInfo info(*it_cache);
-        if ( info.isDir() )
-        {
-          // does the current file exists in the current cache?
-          Pathname cached_file = *it_cache + (*it_res).location.filename();
-          if ( PathInfo( cached_file ).isExist() )
+       MIL << "file " << resource_r.filename() << " found in cache " << cacheDir << endl;
+       swap( ret, cacheLocation );
+       return ret;
+      }
+    }
+
+    return ret;
+  }
+
+  void Fetcher::Impl::validate( const Pathname & localfile_r, const std::list<FileChecker> & checkers_r )
+  {
+    try
+    {
+      MIL << "Checking job [" << localfile_r << "] (" << checkers_r.size() << " checkers )" << endl;
+
+      for ( const FileChecker & chkfnc : checkers_r )
+      {
+        if ( chkfnc )
+          chkfnc( localfile_r );
+        else
+          ERR << "Invalid checker for '" << localfile_r << "'" << endl;
+      }
+
+    }
+    catch ( const FileCheckException &e )
+    {
+      ZYPP_RETHROW(e);
+    }
+    catch ( const Exception &e )
+    {
+      ZYPP_RETHROW(e);
+    }
+    catch (...)
+    {
+      ZYPP_THROW(Exception("Unknown error while validating " + localfile_r.asString()));
+    }
+  }
+
+  void Fetcher::Impl::autoaddIndexes( const filesystem::DirContent &content,
+                                      MediaSetAccess &media,
+                                      const OnMediaLocation &resource,
+                                      const Pathname &dest_dir )
+  {
+      auto fnc_addIfInContent( [&]( const std::string & index_r ) -> bool
+      {
+       if ( find( content.begin(), content.end(), filesystem::DirEntry(index_r,filesystem::FT_FILE) ) == content.end() )
+         return false;
+       // add the index of this directory
+       OnMediaLocation indexloc( resource );
+       indexloc.changeFilename( resource.filename() + index_r );
+       addIndex( indexloc );
+       // we need to read it now
+       downloadAndReadIndexList( media, dest_dir );
+       return true;
+      } );
+
+      if ( _options & AutoAddChecksumsIndexes )
+      {
+       fnc_addIfInContent( "CHECKSUMS" ) || fnc_addIfInContent( "SHA1SUMS" );
+      }
+      if ( _options & AutoAddContentFileIndexes )
+      {
+       fnc_addIfInContent( "content" );
+      }
+  }
+
+  void Fetcher::Impl::getDirectoryContent( MediaSetAccess &media,
+                                           const OnMediaLocation &resource,
+                                           filesystem::DirContent &content )
+  {
+      if ( _dircontent.find(resource.filename().asString())
+           != _dircontent.end() )
+      {
+          filesystem::DirContent filled(_dircontent[resource.filename().asString()]);
+
+          std::copy(filled.begin(), filled.end(), std::back_inserter(content));
+      }
+      else
+      {
+          filesystem::DirContent tofill;
+          media.dirInfo( tofill,
+                         resource.filename(),
+                         false /* dots */,
+                         resource.medianr());
+          std::copy(tofill.begin(), tofill.end(), std::back_inserter(content));
+          _dircontent[resource.filename().asString()] = tofill;
+      }
+  }
+
+  void Fetcher::Impl::addDirJobs( MediaSetAccess &media,
+                                  const OnMediaLocation &resource,
+                                  const Pathname &dest_dir, FetcherJob::Flags flags  )
+  {
+      // first get the content of the directory so we can add
+      // individual transfer jobs
+      MIL << "Adding directory " << resource.filename() << endl;
+      filesystem::DirContent content;
+      try {
+       getDirectoryContent(media, resource, content);
+      }
+      catch ( media::MediaFileNotFoundException & exception )
+      {
+       ZYPP_CAUGHT( exception );
+       WAR << "Skiping subtree hidden at " << resource.filename() << endl;
+       return;
+      }
+
+      // this method test for the option flags so indexes are added
+      // only if the options are enabled
+      autoaddIndexes(content, media, resource, dest_dir);
+
+      for ( filesystem::DirContent::const_iterator it = content.begin();
+            it != content.end();
+            ++it )
+      {
+          // skip CHECKSUMS* as they were already retrieved
+          if ( str::hasPrefix(it->name, "CHECKSUMS") || str::hasPrefix(it->name, "SHA1SUMS") )
+              continue;
+
+          Pathname filename = resource.filename() + it->name;
+
+          switch ( it->type )
           {
-            // check the checksum
-            if ( is_checksum( cached_file, (*it_res).location.checksum() ) )
-            {
-              // cached
-              MIL << "file " << (*it_res).location.filename() << " found in previous cache. Using cached copy." << endl;
-              // checksum is already checked.
-              // we could later implement double failover and try to download if file copy fails.
-  
-              // replicate the complete path in the target directory
-              Pathname dest_full_path = dest_dir + (*it_res).location.filename();
-              if ( assert_dir( dest_full_path.dirname() ) != 0 )
-                ZYPP_THROW( Exception("Can't create " + dest_full_path.dirname().asString()));
-  
-              if ( filesystem::copy(cached_file, dest_full_path ) != 0 )
-              { //copy_file2dir
-                //ZYPP_THROW(SourceIOException("Can't copy " + cached_file.asString() + " to " + destination.asString()));
-                ERR << "Can't copy " << cached_file + " to " + dest_dir << endl;
-                // try next cache
-                continue;
+          case filesystem::FT_NOT_AVAIL: // old directory.yast contains no typeinfo at all
+          case filesystem::FT_FILE:
+          {
+              CheckSum chksm(resource.checksum());
+              if ( _checksums.find(filename.asString()) != _checksums.end() )
+              {
+                  // the checksum can be replaced with the one in the index.
+                  chksm = _checksums[filename.asString()];
+                  //MIL << "resource " << filename << " has checksum in the index file." << endl;
               }
-  
-              got_from_cache = true;
+              else
+                  WAR << "Resource " << filename << " has no checksum in the index either." << endl;
+
+              if ( flags & FetcherJob::AlwaysVerifyChecksum )
+                  enqueueDigested(OnMediaLocation(filename, resource.medianr()).setChecksum(chksm));
+              else
+                  enqueue(OnMediaLocation(filename, resource.medianr()).setChecksum(chksm));
               break;
-            }
           }
-          else
+          case filesystem::FT_DIR: // newer directory.yast contain at least directory info
+              if ( flags & FetcherJob::Recursive )
+                  addDirJobs(media, filename, dest_dir, flags);
+              break;
+          default:
+              // don't provide devices, sockets, etc.
+              break;
+          }
+      }
+  }
+
+  void Fetcher::Impl::provideToDest( MediaSetAccess & media_r, const Pathname & destDir_r , const FetcherJob_Ptr & jobp_r )
+  {
+    const OnMediaLocation & resource( jobp_r->location );
+
+    try
+    {
+      scoped_ptr<MediaSetAccess::ReleaseFileGuard> releaseFileGuard; // will take care provided files get released
+
+      // get cached file (by checksum) or provide from media
+      Pathname tmpFile = locateInCache( resource, destDir_r );
+      if ( tmpFile.empty() )
+      {
+       MIL << "Not found in cache, retrieving..." << endl;
+       tmpFile = media_r.provideFile( resource, resource.optional() ? MediaSetAccess::PROVIDE_NON_INTERACTIVE : MediaSetAccess::PROVIDE_DEFAULT, jobp_r->deltafile );
+       releaseFileGuard.reset( new MediaSetAccess::ReleaseFileGuard( media_r, resource ) ); // release it when we leave the block
+      }
+
+      // The final destination: locateInCache also checks destFullPath!
+      // If we find a cache match (by checksum) at destFullPath, take
+      // care it gets deleted, in case the validation fails.
+      ManagedFile destFullPath( destDir_r / resource.filename() );
+      if ( tmpFile == destFullPath )
+       destFullPath.setDispose( filesystem::unlink );
+
+      // validate the file (throws if not valid)
+      validate( tmpFile, jobp_r->checkers );
+
+      // move it to the final destination
+      if ( tmpFile == destFullPath )
+       destFullPath.resetDispose();    // keep it!
+      else
+      {
+       if ( assert_dir( destFullPath->dirname() ) != 0 )
+         ZYPP_THROW( Exception( "Can't create " + destFullPath->dirname().asString() ) );
+
+       if ( filesystem::hardlinkCopy( tmpFile, destFullPath ) != 0 )
+         ZYPP_THROW( Exception( "Can't hardlink/copy " + tmpFile.asString() + " to " + destDir_r.asString() ) );
+      }
+    }
+    catch ( Exception & excpt )
+    {
+      if ( resource.optional() )
+      {
+       ZYPP_CAUGHT( excpt );
+       WAR << "optional resource " << resource << " could not be transferred." << endl;
+       return;
+      }
+      else
+      {
+       excpt.remember( "Can't provide " + resource.filename().asString() );
+       ZYPP_RETHROW( excpt );
+      }
+    }
+  }
+
+  // helper class to consume a content file
+  struct ContentReaderHelper : public parser::susetags::ContentFileReader
+  {
+    ContentReaderHelper()
+    {
+      setRepoIndexConsumer( bind( &ContentReaderHelper::consumeIndex, this, _1 ) );
+    }
+
+    void consumeIndex( const parser::susetags::RepoIndex_Ptr & data_r )
+    { _repoindex = data_r; }
+
+    parser::susetags::RepoIndex_Ptr _repoindex;
+  };
+
+  // generic function for reading indexes
+  void Fetcher::Impl::readIndex( const Pathname &index, const Pathname &basedir )
+  {
+    if ( index.basename() == "CHECKSUMS" || index.basename() == "SHA1SUMS" )
+      readChecksumsIndex(index, basedir);
+    else if ( index.basename() == "content" )
+      readContentFileIndex(index, basedir);
+    else
+      WAR << index << ": index file format not known" << endl;
+  }
+
+  // reads a content file index
+  void Fetcher::Impl::readContentFileIndex( const Pathname &index, const Pathname &basedir )
+  {
+      ContentReaderHelper reader;
+      reader.parse(index);
+      MIL << index << " contains " << reader._repoindex->mediaFileChecksums.size() << " checksums." << endl;
+      for_( it, reader._repoindex->mediaFileChecksums.begin(), reader._repoindex->mediaFileChecksums.end() )
+      {
+          // content file entries don't start with /
+          _checksums[(basedir + it->first).asString()] = it->second;
+      }
+  }
+
+  // reads a CHECKSUMS (old SHA1SUMS) file index
+  void Fetcher::Impl::readChecksumsIndex( const Pathname &index, const Pathname &basedir )
+  {
+      std::ifstream in( index.c_str() );
+      if ( ! in.fail() )
+      {
+         std::string buffer;
+          while ( getline( in, buffer ) )
           {
-            // File exists in cache but with a different checksum
-            // so just try next cache
-            continue;
+
+             if ( buffer[0] == '#' )
+               continue;       // simple comment
+
+             CheckSum checksum( str::stripFirstWord( buffer, /*ltrim before strip*/true ) );
+             if ( checksum.empty() )
+               continue;       // empty line | unknown cheksum format
+
+             if ( buffer.empty() )
+             {
+               WAR << "Missing filename in CHECKSUMS file: " << index.asString() << " (" << checksum << ")" << endl;
+               continue;
+             }
+
+             _checksums[(basedir/buffer).asString()] = checksum;
           }
+      }
+      else
+          ZYPP_THROW(Exception("Can't open CHECKSUMS file: " + index.asString()));
+  }
+
+  void Fetcher::Impl::downloadIndex( MediaSetAccess &media, const OnMediaLocation &resource, const Pathname &dest_dir)
+  {
+    MIL << "downloading index " << resource << endl;
+    // create a new fetcher with a different state to transfer the
+    // file containing checksums and its signature
+    Fetcher fetcher;
+    // signature checker for index. We havent got the signature from
+    // the nextwork yet.
+    SignatureFileChecker sigchecker;
+
+    // build the name of the index and the signature
+    OnMediaLocation idxloc(resource);
+    OnMediaLocation sigloc(resource);
+    OnMediaLocation keyloc(resource);
+
+    // we should not fail the download if those don't exists
+    // the checking will warn later
+    sigloc.setOptional(true);
+    keyloc.setOptional(true);
+
+    // calculate signature and key name
+    sigloc.changeFilename( sigloc.filename().extend(".asc") );
+    keyloc.changeFilename( keyloc.filename().extend(".key") );
+
+    //assert_dir(dest_dir + idxloc.filename().dirname());
+
+    // transfer the signature
+    fetcher.enqueue(sigloc);
+    fetcher.start( dest_dir, media );
+    // if we get the signature, update the checker
+    if ( PathInfo(dest_dir + sigloc.filename()).isExist() )
+        sigchecker = SignatureFileChecker(dest_dir + sigloc.filename());
+
+    fetcher.reset();
+
+    // now the key
+    fetcher.enqueue(keyloc);
+    fetcher.start( dest_dir, media );
+    fetcher.reset();
+
+    // try to import the key
+    if ( PathInfo(dest_dir + keyloc.filename()).isExist() )
+        getZYpp()->keyRing()->importKey(PublicKey(dest_dir + keyloc.filename()), false);
+    else
+        WAR << "No public key specified by user for index '" << keyloc.filename() << "'"<< endl;
+
+    // now the index itself
+    fetcher.enqueue( idxloc, FileChecker(sigchecker) );
+    fetcher.start( dest_dir, media );
+    fetcher.reset();
+ }
+
+  // this method takes all the user pointed indexes, gets them and also tries to
+  // download their signature, and verify them. After that, its parses each one
+  // to fill the checksum cache.
+  void Fetcher::Impl::downloadAndReadIndexList( MediaSetAccess &media, const Pathname &dest_dir)
+  {
+      // if there is no indexes, then just return to avoid
+      // the directory listing
+      if ( _indexes.empty() )
+      {
+          MIL << "No indexes to read." << endl;
+          return;
+      }
+
+      for_( it_idx, _indexes.begin(), _indexes.end() )
+      {
+        if ( (*it_idx)->read )
+        {
+          DBG << "Already read index " << PathInfo(dest_dir + (*it_idx)->location.filename()) << endl;
         }
         else
         {
-          // skip bad cache directory and try with next one
-          ERR << "Skipping cache : " << *it_cache << endl;
-          continue;
+          // base::LogControl::TmpLineWriter shutUp;
+          downloadIndex( media, (*it_idx)->location, dest_dir );
+          // now we have the indexes in dest_dir
+          readIndex( dest_dir + (*it_idx)->location.filename(), (*it_idx)->location.filename().dirname() );
+          // Take care we don't process it again
+          MIL << "Remember read index " << PathInfo(dest_dir + (*it_idx)->location.filename()) << endl;
+          (*it_idx)->read = true;
         }
       }
-  
-      if ( ! got_from_cache )
+      MIL << "done reading indexes" << endl;
+  }
+
+  // start processing all fetcher jobs.
+  // it processes any user pointed index first
+  void Fetcher::Impl::start( const Pathname &dest_dir,
+                             MediaSetAccess &media,
+                             const ProgressData::ReceiverFnc & progress_receiver )
+  {
+    ProgressData progress(_resources.size());
+    progress.sendTo(progress_receiver);
+
+    downloadAndReadIndexList(media, dest_dir);
+
+    for ( const FetcherJob_Ptr & jobp : _resources )
+    {
+      if ( jobp->flags & FetcherJob::Directory )
       {
-        // try to get the file from the net
-        try
-        {
-          Pathname tmp_file = media.provideFile((*it_res).location);
-          Pathname dest_full_path = dest_dir + (*it_res).location.filename();
-          if ( assert_dir( dest_full_path.dirname() ) != 0 )
-                ZYPP_THROW( Exception("Can't create " + dest_full_path.dirname().asString()));
-          if ( filesystem::copy(tmp_file, dest_full_path ) != 0 )
+          const OnMediaLocation location(jobp->location);
+          addDirJobs(media, location, dest_dir, jobp->flags);
+          continue;
+      }
+
+      // may be this code can be factored out
+      // together with the autodiscovery of indexes
+      // of addDirJobs
+      if ( ( _options & AutoAddChecksumsIndexes ) ||
+           ( _options & AutoAddContentFileIndexes ) )
+      {
+          // if auto indexing is enabled, then we need to read the
+          // index for each file. We look only in the directory
+          // where the file is. this is expensive of course.
+          filesystem::DirContent content;
+          getDirectoryContent(media, jobp->location.filename().dirname(), content);
+          // this method test for the option flags so indexes are added
+          // only if the options are enabled
+          MIL << "Autodiscovering signed indexes on '"
+              << jobp->location.filename().dirname() << "' for '"
+              << jobp->location.filename() << "'" << endl;
+
+          autoaddIndexes(content, media, jobp->location.filename().dirname(), dest_dir);
+
+          // also look in the root of the media
+          content.clear();
+          getDirectoryContent(media, Pathname("/"), content);
+          // this method test for the option flags so indexes are added
+          // only if the options are enabled
+          MIL << "Autodiscovering signed indexes on '"
+              << "/" << "' for '"
+              << jobp->location.filename() << "'" << endl;
+
+          autoaddIndexes(content, media, Pathname("/"), dest_dir);
+      }
+
+      // if the checksum is empty, but the checksum is in one of the
+      // indexes checksum, then add a checker
+      if ( jobp->location.checksum().empty() )
+      {
+          if ( _checksums.find(jobp->location.filename().asString())
+               != _checksums.end() )
           {
-            ZYPP_THROW( Exception("Can't copy " + tmp_file.asString() + " to " + dest_dir.asString()));
+              CheckSum chksm = _checksums[jobp->location.filename().asString()];
+              ChecksumFileChecker digest_check(chksm);
+              jobp->checkers.push_back(digest_check);
+          }
+          else
+          {
+              // if the index checksum is empty too, we only add the checker
+              // if the  AlwaysVerifyChecksum option is set on
+              if ( jobp->flags & FetcherJob::AlwaysVerifyChecksum )
+              {
+                  // add the checker with the empty checksum
+                  ChecksumFileChecker digest_check(jobp->location.checksum());
+                  jobp->checkers.push_back(digest_check);
+              }
           }
-          
-          
-        }
-        catch (const Exception & excpt_r)
-        {
-          ZYPP_CAUGHT(excpt_r);
-          ZYPP_THROW(Exception("Can't provide " + (*it_res).location.filename().asString() + " : " + excpt_r.msg() ));
-        }
       }
       else
       {
-        // We got the file from cache
-        // continue with next file
-        continue;
+          // checksum is not empty, so add a checksum checker
+          ChecksumFileChecker digest_check(jobp->location.checksum());
+          jobp->checkers.push_back(digest_check);
       }
-      
-      // no matter where did we got the file, try to validate it:
-       Pathname localfile = dest_dir + (*it_res).location.filename();
-       // call the checker function
-       try {
-         (*it_res).checkers(localfile);
-       }
-       catch ( const FileCheckException &e )
-       {
-          ZYPP_RETHROW(e);
-       }
-       catch ( const Exception &e )
-       {
-          ZYPP_RETHROW(e);
-       }
-       catch (...)
-       {
-          ZYPP_THROW(Exception("Unknown error while validating " + (*it_res).location.filename().asString()));
-       }
-
-       if ( ! progress.incr() )
+
+      // Provide and validate the file. If the file was not transferred
+      // and no exception was thrown, it was an optional file.
+      provideToDest( media, dest_dir, jobp );
+
+      if ( ! progress.incr() )
         ZYPP_THROW(AbortRequestException());
     } // for each job
   }
@@ -218,47 +804,66 @@ namespace zypp
   /** \relates Fetcher::Impl Stream output */
   inline std::ostream & operator<<( std::ostream & str, const Fetcher::Impl & obj )
   {
-    return str << "Fetcher::Impl";
+      for ( std::list<FetcherJob_Ptr>::const_iterator it_res = obj._resources.begin(); it_res != obj._resources.end(); ++it_res )
+      {
+          str << *it_res;
+      }
+      return str;
   }
 
-  ///////////////////////////////////////////////////////////////////
-  //
-  //   CLASS NAME : Fetcher
-  //
-  ///////////////////////////////////////////////////////////////////
-
-  ///////////////////////////////////////////////////////////////////
-  //
-  //   METHOD NAME : Fetcher::Fetcher
-  //   METHOD TYPE : Ctor
-  //
   Fetcher::Fetcher()
-  : _pimpl( Impl::nullimpl() )
+  : _pimpl( new Impl() )
   {}
 
-  ///////////////////////////////////////////////////////////////////
-  //
-  //   METHOD NAME : Fetcher::~Fetcher
-  //   METHOD TYPE : Dtor
-  //
   Fetcher::~Fetcher()
   {}
 
-  void Fetcher::enqueueDigested( const OnMediaLocation &resource, const FileChecker &checker )
+  void Fetcher::setOptions( Fetcher::Options options )
   {
-    _pimpl->enqueue(resource, checker);
+    _pimpl->setOptions(options);
+  }
+
+  Fetcher::Options Fetcher::options() const
+  {
+    return _pimpl->options();
+  }
+
+  void Fetcher::enqueueDigested( const OnMediaLocation &resource, const FileChecker &checker, const Pathname &deltafile )
+  {
+    _pimpl->enqueueDigested(resource, checker, deltafile);
+  }
+
+  void Fetcher::enqueueDir( const OnMediaLocation &resource,
+                            bool recursive,
+                            const FileChecker &checker )
+  {
+      _pimpl->enqueueDir(resource, recursive, checker);
+  }
+
+  void Fetcher::enqueueDigestedDir( const OnMediaLocation &resource,
+                                    bool recursive,
+                                    const FileChecker &checker )
+  {
+      _pimpl->enqueueDigestedDir(resource, recursive, checker);
+  }
+
+
+  void Fetcher::addIndex( const OnMediaLocation &resource )
+  {
+    _pimpl->addIndex(resource);
   }
-  
+
+
   void Fetcher::enqueue( const OnMediaLocation &resource, const FileChecker &checker  )
   {
     _pimpl->enqueue(resource, checker);
   }
-  
+
   void Fetcher::addCachePath( const Pathname &cache_dir )
   {
     _pimpl->addCachePath(cache_dir);
   }
-  
+
   void Fetcher::reset()
   {
     _pimpl->reset();
@@ -271,12 +876,6 @@ namespace zypp
     _pimpl->start(dest_dir, media, progress_receiver);
   }
 
-
-  /******************************************************************
-  **
-  **   FUNCTION NAME : operator<<
-  **   FUNCTION TYPE : std::ostream &
-  */
   std::ostream & operator<<( std::ostream & str, const Fetcher & obj )
   {
     return str << *obj._pimpl;