Imported Upstream version 15.0.0
[platform/upstream/libzypp.git] / zypp / KeyRing.cc
index 13fdbf5..3523b45 100644 (file)
 #include <cstdio>
 #include <unistd.h>
 
+#include <boost/format.hpp>
+
 #include "zypp/TmpPath.h"
 #include "zypp/ZYppFactory.h"
 #include "zypp/ZYpp.h"
 
-#include "zypp/base/Logger.h"
+#include "zypp/base/LogTools.h"
 #include "zypp/base/IOStream.h"
 #include "zypp/base/String.h"
 #include "zypp/base/Regex.h"
+#include "zypp/base/Gettext.h"
+#include "zypp/base/WatchFile.h"
 #include "zypp/PathInfo.h"
 #include "zypp/KeyRing.h"
 #include "zypp/ExternalProgram.h"
 #include "zypp/TmpPath.h"
 
-using namespace std;
-using namespace zypp::filesystem;
+using std::endl;
 
 #undef  ZYPP_BASE_LOGGER_LOGGROUP
 #define ZYPP_BASE_LOGGER_LOGGROUP "zypp::KeyRing"
 
+/** \todo Fix duplicate define in PublicKey/KeyRing */
 #define GPG_BINARY "/usr/bin/gpg2"
 
 ///////////////////////////////////////////////////////////////////
@@ -42,32 +46,138 @@ namespace zypp
 
   IMPL_PTR_TYPE(KeyRing);
 
-  static bool printLine( const string &line )
+  namespace
   {
-    MIL <<  line << endl;
-    return true;
+    KeyRing::DefaultAccept _keyRingDefaultAccept( KeyRing::ACCEPT_NOTHING );
   }
 
-  namespace
+  KeyRing::DefaultAccept KeyRing::defaultAccept()
+  { return _keyRingDefaultAccept; }
+
+  void KeyRing::setDefaultAccept( DefaultAccept value_r )
   {
-    bool _keyRingDefaultAccept( getenv("ZYPP_KEYRING_DEFAULT_ACCEPT_ALL") );
+    MIL << "Set new KeyRing::DefaultAccept: " << value_r << endl;
+    _keyRingDefaultAccept = value_r;
   }
 
-  bool KeyRingReport::askUserToAcceptUnsignedFile( const string &file )
-  { return _keyRingDefaultAccept; }
+  void KeyRingReport::infoVerify( const std::string & file_r, const PublicKeyData & keyData_r, const KeyContext & keycontext )
+  {}
 
-  bool KeyRingReport::askUserToAcceptUnknownKey( const string &file, const string &id )
-  { return _keyRingDefaultAccept; }
+  bool KeyRingReport::askUserToAcceptUnsignedFile( const std::string & file, const KeyContext & keycontext )
+  { return _keyRingDefaultAccept.testFlag( KeyRing::ACCEPT_UNSIGNED_FILE ); }
 
-  bool KeyRingReport::askUserToTrustKey( const PublicKey &key )
-  { return _keyRingDefaultAccept; }
+  KeyRingReport::KeyTrust
+  KeyRingReport::askUserToAcceptKey( const PublicKey & key, const KeyContext & keycontext )
+  {
+    if ( _keyRingDefaultAccept.testFlag( KeyRing::TRUST_KEY_TEMPORARILY ) )
+      return KEY_TRUST_TEMPORARILY;
+    if ( _keyRingDefaultAccept.testFlag( KeyRing::TRUST_AND_IMPORT_KEY ) )
+      return KEY_TRUST_AND_IMPORT;
+    return KEY_DONT_TRUST;
+  }
+
+  bool KeyRingReport::askUserToAcceptUnknownKey( const std::string & file, const std::string & id, const KeyContext & keycontext )
+  { return _keyRingDefaultAccept.testFlag( KeyRing::ACCEPT_UNKNOWNKEY ); }
+
+  bool KeyRingReport::askUserToAcceptVerificationFailed( const std::string & file, const PublicKey & key, const KeyContext & keycontext )
+  { return _keyRingDefaultAccept.testFlag( KeyRing::ACCEPT_VERIFICATION_FAILED ); }
+
+  namespace
+  {
+    ///////////////////////////////////////////////////////////////////
+    /// \class CachedPublicKeyData
+    /// \brief Functor returning the keyrings data (cached).
+    /// \code
+    ///   const std::list<PublicKeyData> & cachedPublicKeyData( const Pathname & keyring );
+    /// \endcode
+    ///////////////////////////////////////////////////////////////////
+    struct CachedPublicKeyData // : private base::NonCopyable - but KeyRing uses RWCOW though also NonCopyable :(
+    {
+      const std::list<PublicKeyData> & operator()( const Pathname & keyring_r ) const
+      { return getData( keyring_r ); }
+
+    private:
+      struct Cache
+      {
+       // Empty copy ctor to allow insert into std::map as
+       // scoped_ptr is noncopyable.
+       Cache() {}
+       Cache( const Cache & rhs ) {}
+
+       void assertCache( const Pathname & keyring_r )
+       {
+         // .kbx since gpg2-2.1
+         if ( !_keyringK )
+           _keyringK.reset( new WatchFile( keyring_r/"pubring.kbx", WatchFile::NO_INIT ) );
+         if ( !_keyringP )
+           _keyringP.reset( new WatchFile( keyring_r/"pubring.gpg", WatchFile::NO_INIT ) );
+       }
+
+       bool hasChanged() const
+       {
+         bool k = _keyringK->hasChanged();     // be sure both files are checked
+         bool p = _keyringP->hasChanged();
+         return k || p;
+       }
+
+       std::list<PublicKeyData> _data;
+
+      private:
+       scoped_ptr<WatchFile> _keyringK;
+       scoped_ptr<WatchFile> _keyringP;
+      };
+
+      typedef std::map<Pathname,Cache> CacheMap;
+
+      const std::list<PublicKeyData> & getData( const Pathname & keyring_r ) const
+      {
+       Cache & cache( _cacheMap[keyring_r] );
+       // init new cache entry
+       cache.assertCache( keyring_r );
+       return getData( keyring_r, cache );
+      }
+
+      const std::list<PublicKeyData> & getData( const Pathname & keyring_r, Cache & cache_r ) const
+      {
+       if ( cache_r.hasChanged() )
+       {
+         const char* argv[] =
+         {
+           GPG_BINARY,
+           "--list-public-keys",
+           "--homedir", keyring_r.c_str(),
+           "--no-default-keyring",
+           "--quiet",
+           "--with-colons",
+           "--fixed-list-mode",
+           "--with-fingerprint",
+           "--with-sig-list",
+           "--no-tty",
+           "--no-greeting",
+           "--batch",
+           "--status-fd", "1",
+           NULL
+         };
+
+         PublicKeyScanner scanner;
+         ExternalProgram prog( argv ,ExternalProgram::Discard_Stderr, false, -1, true );
+         for( std::string line = prog.receiveLine(); !line.empty(); line = prog.receiveLine() )
+         {
+           scanner.scan( line );
+         }
+         prog.close();
+
+         cache_r._data.swap( scanner._keys );
+         MIL << "Found keys: " << cache_r._data  << endl;
+       }
+       return cache_r._data;
+      }
+
+      mutable CacheMap _cacheMap;
+    };
+    ///////////////////////////////////////////////////////////////////
+  }
 
-  bool KeyRingReport::askUserToImportKey( const PublicKey &key)
-  { return _keyRingDefaultAccept; }
-  
-  bool KeyRingReport::askUserToAcceptVerificationFailed( const string &file, const PublicKey &key )
-  { return _keyRingDefaultAccept; }
-  
   ///////////////////////////////////////////////////////////////////
   //
   //   CLASS NAME : KeyRing::Impl
@@ -75,279 +185,284 @@ namespace zypp
   /** KeyRing implementation. */
   struct KeyRing::Impl
   {
-    Impl(const Pathname &baseTmpDir)
-    : _trusted_tmp_dir(baseTmpDir, "zypp-trusted-kr")
-   ,  _general_tmp_dir(baseTmpDir, "zypp-general-kr")
-   , _base_dir( baseTmpDir )
-
+    Impl( const Pathname & baseTmpDir )
+    : _trusted_tmp_dir( baseTmpDir, "zypp-trusted-kr" )
+    , _general_tmp_dir( baseTmpDir, "zypp-general-kr" )
+    , _base_dir( baseTmpDir )
     {
+      MIL << "Current KeyRing::DefaultAccept: " << _keyRingDefaultAccept << endl;
     }
 
-    /*
-    Impl( const Pathname &general_kr, const Pathname &trusted_kr )
-    {
-      filesystem::assert_dir(general_kr);
-      filesystem::assert_dir(trusted_kr);
+    void importKey( const PublicKey & key, bool trusted = false );
+    void multiKeyImport( const Pathname & keyfile_r, bool trusted_r = false );
+    void deleteKey( const std::string & id, bool trusted );
 
-      generalKeyRing() = general_kr;
-      trustedKeyRing() = trusted_kr;
-    }
-    */
+    std::string readSignatureKeyId( const Pathname & signature );
 
-    void importKey( const PublicKey &key, bool trusted = false);
-    void deleteKey( const string &id, bool trusted );
-    
-    string readSignatureKeyId( const Pathname &signature );
-    
-    bool isKeyTrusted( const string &id);
-    bool isKeyKnown( const string &id );
-    
-    list<PublicKey> trustedPublicKeys();
-    list<PublicKey> publicKeys();
+    bool isKeyTrusted( const std::string & id )
+    { return bool(publicKeyExists( id, trustedKeyRing() )); }
+    bool isKeyKnown( const std::string & id )
+    { return publicKeyExists( id, trustedKeyRing() ) || publicKeyExists( id, generalKeyRing() ); }
 
-    list<string> trustedPublicKeyIds();
-    list<string> publicKeyIds();
+    std::list<PublicKey> trustedPublicKeys()
+    { return publicKeys( trustedKeyRing() ); }
+    std::list<PublicKey> publicKeys()
+    { return publicKeys( generalKeyRing() ); }
 
-    void dumpPublicKey( const string &id, bool trusted, ostream &stream );
+    const std::list<PublicKeyData> & trustedPublicKeyData()
+    { return publicKeyData( trustedKeyRing() ); }
+    const std::list<PublicKeyData> & publicKeyData()
+    { return publicKeyData( generalKeyRing() ); }
 
-    bool verifyFileSignatureWorkflow( const Pathname &file, const string filedesc, const Pathname &signature);
+    void dumpPublicKey( const std::string & id, bool trusted, std::ostream & stream )
+    { dumpPublicKey( id, ( trusted ? trustedKeyRing() : generalKeyRing() ), stream ); }
+
+    PublicKey exportPublicKey( const PublicKeyData & keyData )
+    { return exportKey( keyData, generalKeyRing() ); }
+    PublicKey exportTrustedPublicKey( const PublicKeyData & keyData )
+    { return exportKey( keyData, trustedKeyRing() ); }
+
+    bool verifyFileSignatureWorkflow(
+        const Pathname & file,
+        const std::string & filedesc,
+        const Pathname & signature,
+        const KeyContext & keycontext = KeyContext());
+
+    bool verifyFileSignature( const Pathname & file, const Pathname & signature )
+    { return verifyFile( file, signature, generalKeyRing() ); }
+    bool verifyFileTrustedSignature( const Pathname & file, const Pathname & signature )
+    { return verifyFile( file, signature, trustedKeyRing() ); }
 
-    bool verifyFileSignature( const Pathname &file, const Pathname &signature);
-    bool verifyFileTrustedSignature( const Pathname &file, const Pathname &signature);
   private:
-    //mutable map<Locale, string> translations;
-    bool verifyFile( const Pathname &file, const Pathname &signature, const Pathname &keyring);
-    void importKey( const Pathname &keyfile, const Pathname &keyring);
-    PublicKey exportKey( string id, const Pathname &keyring);
-    void dumpPublicKey( const string &id, const Pathname &keyring, ostream &stream );
-    void deleteKey( const string &id, const Pathname &keyring );
-    
-    list<PublicKey> publicKeys(const Pathname &keyring);
-    list<string> publicKeyIds(const Pathname &keyring);
-    
-    bool publicKeyExists( string id, const Pathname &keyring);
-
-    const Pathname generalKeyRing() const;
-    const Pathname trustedKeyRing() const;
+    bool verifyFile( const Pathname & file, const Pathname & signature, const Pathname & keyring );
+    void importKey( const Pathname & keyfile, const Pathname & keyring );
+
+    PublicKey exportKey( const std::string & id, const Pathname & keyring );
+    PublicKey exportKey( const PublicKeyData & keyData, const Pathname & keyring );
+
+    void dumpPublicKey( const std::string & id, const Pathname & keyring, std::ostream & stream );
+    filesystem::TmpFile dumpPublicKeyToTmp( const std::string & id, const Pathname & keyring );
+
+    void deleteKey( const std::string & id, const Pathname & keyring );
+
+    std::list<PublicKey> publicKeys( const Pathname & keyring);
+    const std::list<PublicKeyData> & publicKeyData( const Pathname & keyring )
+    { return cachedPublicKeyData( keyring ); }
+
+    /** Get \ref PublicKeyData for ID (\c false if ID is not found). */
+    PublicKeyData publicKeyExists( const std::string & id, const Pathname & keyring );
+
+    const Pathname generalKeyRing() const
+    { return _general_tmp_dir.path(); }
+    const Pathname trustedKeyRing() const
+    { return _trusted_tmp_dir.path(); }
 
     // Used for trusted and untrusted keyrings
-    TmpDir _trusted_tmp_dir;
-    TmpDir _general_tmp_dir;
+    filesystem::TmpDir _trusted_tmp_dir;
+    filesystem::TmpDir _general_tmp_dir;
     Pathname _base_dir;
-  public:
-    /** Offer default Impl. */
-    static shared_ptr<Impl> nullimpl()
-    {
-      static shared_ptr<Impl> _nullimpl( new Impl( TmpPath::defaultLocation() ) );
-      return _nullimpl;
-    }
 
   private:
-    friend Impl * rwcowClone<Impl>( const Impl * rhs );
-    /** clone for RWCOW_pointer */
-    Impl * clone() const
-    { return new Impl( *this ); }
+    /** Functor returning the keyrings data (cached).
+     * \code
+     *  const std::list<PublicKeyData> & cachedPublicKeyData( const Pathname & keyring );
+     * \endcode
+     */
+    CachedPublicKeyData cachedPublicKeyData;
   };
+  ///////////////////////////////////////////////////////////////////
 
 
-  const Pathname KeyRing::Impl::generalKeyRing() const
-  {
-    return _general_tmp_dir.path();
-  }
-
-  const Pathname KeyRing::Impl::trustedKeyRing() const
-  {
-    return _trusted_tmp_dir.path();
-  }
-
-  void KeyRing::Impl::importKey( const PublicKey &key, bool trusted)
+  void KeyRing::Impl::importKey( const PublicKey & key, bool trusted )
   {
-    callback::SendReport<KeyRingSignals> emitSignal;
-
     importKey( key.path(), trusted ? trustedKeyRing() : generalKeyRing() );
-    
+
     if ( trusted )
+    {
+      callback::SendReport<target::rpm::KeyRingSignals> rpmdbEmitSignal;
+      callback::SendReport<KeyRingSignals> emitSignal;
+
+      rpmdbEmitSignal->trustedKeyAdded( key );
       emitSignal->trustedKeyAdded( key );
-    
+    }
   }
 
-  void KeyRing::Impl::deleteKey( const string &id, bool trusted)
+  void KeyRing::Impl::multiKeyImport( const Pathname & keyfile_r, bool trusted_r )
   {
-    deleteKey( id, trusted ? trustedKeyRing() : generalKeyRing() );
+    importKey( keyfile_r, trusted_r ? trustedKeyRing() : generalKeyRing() );
   }
 
-  list<PublicKey> KeyRing::Impl::publicKeys()
+  void KeyRing::Impl::deleteKey( const std::string & id, bool trusted )
   {
-    return publicKeys( generalKeyRing() );
-  }
+    PublicKey key;
 
-  list<PublicKey> KeyRing::Impl::trustedPublicKeys()
-  {
-    return publicKeys( trustedKeyRing() );
-  }
+    if ( trusted )
+    {
+       key = exportKey( id, trustedKeyRing() );
+    }
 
-  list<string> KeyRing::Impl::publicKeyIds()
-  {
-    return publicKeyIds( generalKeyRing() );
-  }
+    deleteKey( id, trusted ? trustedKeyRing() : generalKeyRing() );
 
-  list<string> KeyRing::Impl::trustedPublicKeyIds()
-  {
-    return publicKeyIds( trustedKeyRing() );
-  }
-  
-  bool KeyRing::Impl::verifyFileTrustedSignature( const Pathname &file, const Pathname &signature)
-  {
-    return verifyFile( file, signature, trustedKeyRing() );
-  }
+    if ( trusted )
+    {
+      callback::SendReport<target::rpm::KeyRingSignals> rpmdbEmitSignal;
+      callback::SendReport<KeyRingSignals> emitSignal;
 
-  bool KeyRing::Impl::verifyFileSignature( const Pathname &file, const Pathname &signature)
-  {
-    return verifyFile( file, signature, generalKeyRing() );
+      rpmdbEmitSignal->trustedKeyRemoved( key );
+      emitSignal->trustedKeyRemoved( key );
+    }
   }
 
-  bool KeyRing::Impl::isKeyTrusted( const string &id)
-  {
-    return publicKeyExists( id, trustedKeyRing() );
-  }
-  
-  bool KeyRing::Impl::isKeyKnown( const string &id )
-  {
-    MIL << endl;
-    if ( publicKeyExists( id, trustedKeyRing() ) )
-      return true;
-    else
-      return publicKeyExists( id, generalKeyRing() );
-  }
-  
-  bool KeyRing::Impl::publicKeyExists( string id, const Pathname &keyring)
+  PublicKeyData KeyRing::Impl::publicKeyExists( const std::string & id, const Pathname & keyring )
   {
     MIL << "Searching key [" << id << "] in keyring " << keyring << endl;
-    list<PublicKey> keys = publicKeys(keyring);
-    for (list<PublicKey>::const_iterator it = keys.begin(); it != keys.end(); it++)
+    const std::list<PublicKeyData> & keys( publicKeyData( keyring ) );
+    for_( it, keys.begin(), keys.end() )
     {
       if ( id == (*it).id() )
-        return true;
+      {
+        return *it;
+      }
     }
-    return false;
+    return PublicKeyData();
   }
-  
-  PublicKey KeyRing::Impl::exportKey( string id, const Pathname &keyring)
+
+  PublicKey KeyRing::Impl::exportKey( const PublicKeyData & keyData, const Pathname & keyring )
   {
-    TmpFile tmp_file( _base_dir, "pubkey-"+id+"-" );
-    Pathname keyfile = tmp_file.path();
-    MIL << "Going to export key " << id << " from " << keyring << " to " << keyfile << endl;
-     
-    try {
-      ofstream os(keyfile.asString().c_str());
-      dumpPublicKey( id, keyring, os );
-      os.close();
-      PublicKey key(keyfile);
-      return key;
-    }
-    catch (BadKeyException &e)
-    {
-      ERR << "Cannot create public key " << id << " from " << keyring << " keyring  to file " << e.keyFile() << endl;
-      ZYPP_THROW(Exception("Cannot create public key " + id + " from " + keyring.asString() + " keyring to file " + e.keyFile().asString() ) );
-    }
-    catch (exception &e)
-    {
-      ERR << "Cannot export key " << id << " from " << keyring << " keyring  to file " << keyfile << endl;
-    }
-    return PublicKey();
+    return PublicKey( dumpPublicKeyToTmp( keyData.id(), keyring ), keyData );
   }
 
-  void KeyRing::Impl::dumpPublicKey( const string &id, bool trusted, ostream &stream )
+  PublicKey KeyRing::Impl::exportKey( const std::string & id, const Pathname & keyring )
   {
-     dumpPublicKey( id, ( trusted ? trustedKeyRing() : generalKeyRing() ), stream );
+    PublicKeyData keyData( publicKeyExists( id, keyring ) );
+    if ( keyData )
+      return PublicKey( dumpPublicKeyToTmp( keyData.id(), keyring ), keyData );
+
+    // Here: key not found
+    WAR << "No key " << id << " to export from " << keyring << endl;
+    return PublicKey();
   }
-  
-  void KeyRing::Impl::dumpPublicKey( const string &id, const Pathname &keyring, ostream &stream )
+
+
+  void KeyRing::Impl::dumpPublicKey( const std::string & id, const Pathname & keyring, std::ostream & stream )
   {
     const char* argv[] =
     {
       GPG_BINARY,
+      "-a",
+      "--export",
+      "--homedir", keyring.asString().c_str(),
       "--no-default-keyring",
       "--quiet",
       "--no-tty",
       "--no-greeting",
       "--no-permission-warning",
       "--batch",
-      "--homedir",
-      keyring.asString().c_str(),
-      "-a",
-      "--export",
       id.c_str(),
       NULL
     };
-    ExternalProgram prog(argv,ExternalProgram::Discard_Stderr, false, -1, true);
-    string line;
-    int count;
-    for(line = prog.receiveLine(), count=0; !line.empty(); line = prog.receiveLine(), count++ )
+    ExternalProgram prog( argv,ExternalProgram::Discard_Stderr, false, -1, true );
+    for ( std::string line = prog.receiveLine(); !line.empty(); line = prog.receiveLine() )
     {
       stream << line;
     }
     prog.close();
   }
 
+  filesystem::TmpFile KeyRing::Impl::dumpPublicKeyToTmp( const std::string & id, const Pathname & keyring )
+  {
+    filesystem::TmpFile tmpFile( _base_dir, "pubkey-"+id+"-" );
+    MIL << "Going to export key " << id << " from " << keyring << " to " << tmpFile.path() << endl;
+
+    std::ofstream os( tmpFile.path().c_str() );
+    dumpPublicKey( id, keyring, os );
+    os.close();
+    return tmpFile;
+  }
 
-  bool KeyRing::Impl::verifyFileSignatureWorkflow( const Pathname &file, const string filedesc, const Pathname &signature)
+  bool KeyRing::Impl::verifyFileSignatureWorkflow(
+      const Pathname & file,
+      const std::string & filedesc,
+      const Pathname & signature,
+      const KeyContext & context )
   {
     callback::SendReport<KeyRingReport> report;
-    //callback::SendReport<KeyRingSignals> emitSignal;
-    MIL << "Going to verify signature for " << file << " with " << signature << endl;
+    MIL << "Going to verify signature for " << filedesc << " ( " << file << " ) with " << signature << endl;
 
     // if signature does not exists, ask user if he wants to accept unsigned file.
-    if( signature.empty() || (!PathInfo(signature).isExist()) )
+    if( signature.empty() || (!PathInfo( signature ).isExist()) )
     {
-      bool res = report->askUserToAcceptUnsignedFile( filedesc );
+      bool res = report->askUserToAcceptUnsignedFile( filedesc, context );
       MIL << "User decision on unsigned file: " << res << endl;
       return res;
     }
 
     // get the id of the signature
-    string id = readSignatureKeyId(signature);
+    std::string id = readSignatureKeyId( signature );
 
     // doeskey exists in trusted keyring
-    if ( publicKeyExists( id, trustedKeyRing() ) )
+    PublicKeyData trustedKeyData( publicKeyExists( id, trustedKeyRing() ) );
+    if ( trustedKeyData )
     {
-      PublicKey key = exportKey( id, trustedKeyRing() );
-      
-      MIL << "Key " << id << " " << key.name() << " is trusted" << endl;
+      MIL << "Key is trusted: " << trustedKeyData << endl;
+
+      // lets look if there is an updated key in the
+      // general keyring
+      PublicKeyData generalKeyData( publicKeyExists( id, generalKeyRing() ) );
+      if ( generalKeyData )
+      {
+        // bnc #393160: Comment #30: Compare at least the fingerprint
+        // in case an attacker created a key the the same id.
+        if ( trustedKeyData.fingerprint() == generalKeyData.fingerprint()
+          && trustedKeyData.created() < generalKeyData.created() )
+        {
+          MIL << "Key was updated. Saving new version into trusted keyring: " << generalKeyData << endl;
+          importKey( exportKey( generalKeyData, generalKeyRing() ), true );
+         trustedKeyData = generalKeyData = PublicKeyData(); // invalidated by import.
+       }
+      }
+
+      if ( ! trustedKeyData )  // invalidated by previous import
+       trustedKeyData = publicKeyExists( id, trustedKeyRing() );
+      report->infoVerify( filedesc, trustedKeyData, context );
+
       // it exists, is trusted, does it validates?
       if ( verifyFile( file, signature, trustedKeyRing() ) )
         return true;
       else
-        return report->askUserToAcceptVerificationFailed( filedesc, key );
+      {
+        return report->askUserToAcceptVerificationFailed( filedesc, exportKey( trustedKeyData, trustedKeyRing() ), context );
+      }
     }
     else
     {
-      if ( publicKeyExists( id, generalKeyRing() ) )
+      PublicKeyData generalKeyData( publicKeyExists( id, generalKeyRing() ) );
+      if ( generalKeyData )
       {
-        PublicKey key =  exportKey( id, generalKeyRing());
+        PublicKey key( exportKey( generalKeyData, generalKeyRing() ) );
         MIL << "Exported key " << id << " to " << key.path() << endl;
         MIL << "Key " << id << " " << key.name() << " is not trusted" << endl;
+
         // ok the key is not trusted, ask the user to trust it or not
-        #warning We need the key details passed to the callback
-        if ( report->askUserToTrustKey( key ) )
+        KeyRingReport::KeyTrust reply = report->askUserToAcceptKey( key, context );
+        if ( reply == KeyRingReport::KEY_TRUST_TEMPORARILY ||
+            reply == KeyRingReport::KEY_TRUST_AND_IMPORT )
         {
           MIL << "User wants to trust key " << id << " " << key.name() << endl;
-          //dumpFile(unKey.path());
+          //dumpFile( unKey.path() );
 
-          Pathname which_keyring;
-          if ( report->askUserToImportKey( key ) )
+          Pathname whichKeyring;
+          if ( reply == KeyRingReport::KEY_TRUST_AND_IMPORT )
           {
             MIL << "User wants to import key " << id << " " << key.name() << endl;
             importKey( key, true );
-            which_keyring = trustedKeyRing();
+            whichKeyring = trustedKeyRing();
           }
           else
-          {
-            which_keyring = generalKeyRing();
-          }
+            whichKeyring = generalKeyRing();
 
           // emit key added
-          if ( verifyFile( file, signature, which_keyring ) )
+          if ( verifyFile( file, signature, whichKeyring ) )
           {
             MIL << "File signature is verified" << endl;
             return true;
@@ -355,7 +470,7 @@ namespace zypp
           else
           {
             MIL << "File signature check fails" << endl;
-            if ( report->askUserToAcceptVerificationFailed( filedesc, key ) )
+            if ( report->askUserToAcceptVerificationFailed( filedesc, key, context ) )
             {
               MIL << "User continues anyway." << endl;
               return true;
@@ -377,7 +492,7 @@ namespace zypp
       {
         // unknown key...
         MIL << "File [" << file << "] ( " << filedesc << " ) signed with unknown key [" << id << "]" << endl;
-        if ( report->askUserToAcceptUnknownKey( filedesc, id ) )
+        if ( report->askUserToAcceptUnknownKey( filedesc, id, context ) )
         {
           MIL << "User wants to accept unknown key " << id << endl;
           return true;
@@ -392,194 +507,117 @@ namespace zypp
     return false;
   }
 
-  list<string> KeyRing::Impl::publicKeyIds(const Pathname &keyring)
+  std::list<PublicKey> KeyRing::Impl::publicKeys( const Pathname & keyring )
   {
-    static str::regex rxColons("^([^:]*):([^:]*):([^:]*):([^:]*):([^:]*):([^:]*):([^:]*):([^:]*):([^:]*):([^:]*):([^:]*):([^:]*):\n$");
-    static str::regex rxColonsFpr("^([^:]*):([^:]*):([^:]*):([^:]*):([^:]*):([^:]*):([^:]*):([^:]*):([^:]*):([^:]*):\n$");
+    const std::list<PublicKeyData> & keys( publicKeyData( keyring ) );
+    std::list<PublicKey> ret;
 
-    list<string> ids;
-    
-    const char* argv[] =
-    {
-      GPG_BINARY,
-      "--no-default-keyring",
-      "--quiet",
-      "--list-public-keys",
-      "--with-colons",
-      "--with-fingerprint",
-      "--no-tty",
-      "--no-greeting",
-      "--batch",
-      "--status-fd",
-      "1",
-      "--homedir",
-      keyring.asString().c_str(),
-      NULL
-    };
-    
-    ExternalProgram prog(argv,ExternalProgram::Discard_Stderr, false, -1, true);
-    string line;
-    int count = 0;
-
-    for(line = prog.receiveLine(), count=0; !line.empty(); line = prog.receiveLine(), count++ )
-    {
-      //MIL << line << endl;
-      str::smatch what;
-      if(str::regex_match(line, what, rxColons))
-      {
-        string id;
-        string fingerprint;
-        if ( what[1] == "pub" )
-        {
-          id = what[5];
-          
-          string line2;
-          for(line2 = prog.receiveLine(); !line2.empty(); line2 = prog.receiveLine(), count++ )
-          {
-            str::smatch what2;
-            if (str::regex_match(line2, what2, rxColonsFpr))
-            {
-              if ( (what2[1] == "fpr") && (what2[1] != "pub") && (what2[1] !="sub"))
-              {
-                fingerprint = what2[10];
-                break;
-              }
-            }
-          }
-          
-          ids.push_back(id);
-          MIL << "Found key " << "[" << id << "]" << endl;
-        }
-        //dumpRegexpResults(what);
-      }
-    }
-    prog.close();
-    return ids;
-  }
-  
-  list<PublicKey> KeyRing::Impl::publicKeys(const Pathname &keyring)
-  {
-    
-    list<PublicKey> keys;
-    
-    list<string> ids = publicKeyIds(keyring);
-    
-    for ( list<string>::const_iterator it = ids.begin(); it != ids.end(); ++it )
+    for_( it, keys.begin(), keys.end() )
     {
-      PublicKey key(exportKey( *it, keyring ));
-      keys.push_back(key);
-      MIL << "Found key " << "[" << key.id() << "]" << " [" << key.name() << "]" << " [" << key.fingerprint() << "]" << endl;
+      PublicKey key( exportKey( *it, keyring ) );
+      ret.push_back( key );
+      MIL << "Found key " << key << endl;
     }
-    return keys;
+    return ret;
   }
-    
-  void KeyRing::Impl::importKey( const Pathname &keyfile, const Pathname &keyring)
+
+  void KeyRing::Impl::importKey( const Pathname & keyfile, const Pathname & keyring )
   {
-    if ( ! PathInfo(keyfile).isExist() )
-      ZYPP_THROW(KeyRingException("Tried to import not existant key " + keyfile.asString() + " into keyring " + keyring.asString()));
-    
+    if ( ! PathInfo( keyfile ).isExist() )
+      // TranslatorExplanation first %s is key name, second is keyring name
+      ZYPP_THROW(KeyRingException(boost::str(boost::format(
+          _("Tried to import not existent key %s into keyring %s"))
+          % keyfile.asString() % keyring.asString())));
+
     const char* argv[] =
     {
       GPG_BINARY,
+      "--import",
+      "--homedir", keyring.asString().c_str(),
       "--no-default-keyring",
       "--quiet",
       "--no-tty",
       "--no-greeting",
       "--no-permission-warning",
-      "--status-fd",
-      "1",
-      "--homedir",
-      keyring.asString().c_str(),
-      "--import",
+      "--status-fd", "1",
       keyfile.asString().c_str(),
       NULL
     };
 
-    int code;
-    ExternalProgram prog(argv,ExternalProgram::Discard_Stderr, false, -1, true);
-    code = prog.close();
-
-    //if ( code != 0 )
-    //  ZYPP_THROW(Exception("failed to import key"));
+    ExternalProgram prog( argv,ExternalProgram::Discard_Stderr, false, -1, true );
+    prog.close();
   }
 
-  void KeyRing::Impl::deleteKey( const string &id, const Pathname &keyring )
+  void KeyRing::Impl::deleteKey( const std::string & id, const Pathname & keyring )
   {
     const char* argv[] =
     {
       GPG_BINARY,
+      "--delete-keys",
+      "--homedir", keyring.asString().c_str(),
       "--no-default-keyring",
       "--yes",
       "--quiet",
       "--no-tty",
       "--batch",
-      "--status-fd",
-      "1",
-      "--homedir",
-      keyring.asString().c_str(),
-      "--delete-keys",
+      "--status-fd", "1",
       id.c_str(),
       NULL
     };
 
-    ExternalProgram prog(argv,ExternalProgram::Discard_Stderr, false, -1, true);
+    ExternalProgram prog( argv,ExternalProgram::Discard_Stderr, false, -1, true );
 
     int code = prog.close();
     if ( code )
-      ZYPP_THROW(Exception("Failed to delete key."));
+      ZYPP_THROW(Exception(_("Failed to delete key.")));
     else
       MIL << "Deleted key " << id << " from keyring " << keyring << endl;
   }
 
 
-  string KeyRing::Impl::readSignatureKeyId(const Pathname &signature )
+  std::string KeyRing::Impl::readSignatureKeyId( const Pathname & signature )
   {
-    if ( ! PathInfo(signature).isFile() )
-      ZYPP_THROW(Exception("Signature file " + signature.asString() + " not found"));
+    if ( ! PathInfo( signature ).isFile() )
+      ZYPP_THROW(Exception(boost::str(boost::format(
+          _("Signature file %s not found"))% signature.asString())));
 
     MIL << "Determining key id if signature " << signature << endl;
     // HACK create a tmp keyring with no keys
-    TmpDir dir(_base_dir, "fake-keyring");
-    TmpFile fakeData(_base_dir, "fake-data");
+    filesystem::TmpDir dir( _base_dir, "fake-keyring" );
 
     const char* argv[] =
     {
       GPG_BINARY,
+      "--homedir", dir.path().asString().c_str(),
       "--no-default-keyring",
       "--quiet",
       "--no-tty",
       "--no-greeting",
       "--batch",
-      "--status-fd",
-      "1",
-      "--homedir",
-      dir.path().asString().c_str(),
-      "--verify",
+      "--status-fd", "1",
       signature.asString().c_str(),
-      fakeData.path().asString().c_str(),
       NULL
     };
 
-    ExternalProgram prog(argv,ExternalProgram::Discard_Stderr, false, -1, true);
+    ExternalProgram prog( argv,ExternalProgram::Discard_Stderr, false, -1, true );
 
-    string line;
+    std::string line;
     int count = 0;
 
-    str::regex rxNoKey("^\\[GNUPG:\\] NO_PUBKEY (.+)\n$");
-    string id;
-    for(line = prog.receiveLine(), count=0; !line.empty(); line = prog.receiveLine(), count++ )
+    str::regex rxNoKey( "^\\[GNUPG:\\] NO_PUBKEY (.+)\n$" );
+    std::string id;
+    for( line = prog.receiveLine(), count=0; !line.empty(); line = prog.receiveLine(), count++ )
     {
       //MIL << "[" << line << "]" << endl;
       str::smatch what;
-      if(str::regex_match(line, what, rxNoKey))
+      if( str::regex_match( line, what, rxNoKey ) )
       {
         if ( what.size() >= 1 )
+       {
           id = what[1];
-        //dumpRegexpResults(what);
-      }
-      else
-      {
-        MIL << "'" << line << "'" << endl;
+         break;
+       }
+        //dumpRegexpResults( what );
       }
     }
 
@@ -593,21 +631,19 @@ namespace zypp
     return id;
   }
 
-  bool KeyRing::Impl::verifyFile( const Pathname &file, const Pathname &signature, const Pathname &keyring)
+  bool KeyRing::Impl::verifyFile( const Pathname & file, const Pathname & signature, const Pathname & keyring )
   {
     const char* argv[] =
     {
       GPG_BINARY,
+      "--verify",
+      "--homedir", keyring.asString().c_str(),
       "--no-default-keyring",
       "--quiet",
       "--no-tty",
       "--batch",
       "--no-greeting",
-      "--status-fd",
-      "1",
-      "--homedir",
-      keyring.asString().c_str(),
-      "--verify",
+      "--status-fd", "1",
       signature.asString().c_str(),
       file.asString().c_str(),
       NULL
@@ -623,9 +659,9 @@ namespace zypp
     //     [GNUPG:] ERRSIG A84EDAE89C800ACA 17 2 00 1143618744 9
     //     [GNUPG:] NO_PUBKEY A84EDAE89C800ACA
 
-    ExternalProgram prog(argv,ExternalProgram::Discard_Stderr, false, -1, true);
+    ExternalProgram prog( argv,ExternalProgram::Discard_Stderr, false, -1, true );
 
-    return (prog.close() == 0) ? true : false;
+    return ( prog.close() == 0 ) ? true : false;
   }
 
   ///////////////////////////////////////////////////////////////////
@@ -636,104 +672,66 @@ namespace zypp
   //
   ///////////////////////////////////////////////////////////////////
 
-  ///////////////////////////////////////////////////////////////////
-  //
-  //   METHOD NAME : KeyRing::KeyRing
-  //   METHOD TYPE : Ctor
-  //
-  KeyRing::KeyRing(const Pathname &baseTmpDir)
-  : _pimpl( new Impl(baseTmpDir) )
+  KeyRing::KeyRing( const Pathname & baseTmpDir )
+  : _pimpl( new Impl( baseTmpDir ) )
   {}
 
-  ///////////////////////////////////////////////////////////////////
-  //
-  //   METHOD NAME : KeyRing::KeyRing
-  //   METHOD TYPE : Ctor
-  //
-  //KeyRing::KeyRing( const Pathname &general_kr, const Pathname &trusted_kr )
-  //: _pimpl( new Impl(general_kr, trusted_kr) )
-  //{}
-
-  ///////////////////////////////////////////////////////////////////
-  //
-  //   METHOD NAME : KeyRing::~KeyRing
-  //   METHOD TYPE : Dtor
-  //
   KeyRing::~KeyRing()
   {}
 
-  ///////////////////////////////////////////////////////////////////
-  //
-  // Forward to implementation:
-  //
-  ///////////////////////////////////////////////////////////////////
 
-  
-  void KeyRing::importKey( const PublicKey &key, bool trusted )
-  {
-    _pimpl->importKey( key.path(), trusted );    
-  }
-  
-  string KeyRing::readSignatureKeyId( const Pathname &signature )
-  {
-    return _pimpl->readSignatureKeyId(signature);
-  }
+  void KeyRing::importKey( const PublicKey & key, bool trusted )
+  { _pimpl->importKey( key, trusted ); }
 
-  void KeyRing::deleteKey( const string &id, bool trusted )
-  {
-    _pimpl->deleteKey(id, trusted);
-  }
+  void KeyRing::multiKeyImport( const Pathname & keyfile_r, bool trusted_r )
+  { _pimpl->multiKeyImport( keyfile_r, trusted_r ); }
 
-  list<PublicKey> KeyRing::publicKeys()
-  {
-    return _pimpl->publicKeys();
-  }
+  std::string KeyRing::readSignatureKeyId( const Pathname & signature )
+  { return _pimpl->readSignatureKeyId( signature ); }
 
-  list<PublicKey> KeyRing::trustedPublicKeys()
-  {
-    return _pimpl->trustedPublicKeys();
-  }
+  void KeyRing::deleteKey( const std::string & id, bool trusted )
+  { _pimpl->deleteKey( id, trusted ); }
 
-  list<string> KeyRing::publicKeyIds()
-  {
-    return _pimpl->publicKeyIds();
-  }
+  std::list<PublicKey> KeyRing::publicKeys()
+  { return _pimpl->publicKeys(); }
 
-  list<string> KeyRing::trustedPublicKeyIds()
-  {
-    return _pimpl->trustedPublicKeyIds();
-  }
-  
-  bool KeyRing::verifyFileSignatureWorkflow( const Pathname &file, const string filedesc, const Pathname &signature)
-  {
-    return _pimpl->verifyFileSignatureWorkflow(file, filedesc, signature);
-  }
+  std:: list<PublicKey> KeyRing::trustedPublicKeys()
+  { return _pimpl->trustedPublicKeys(); }
 
-  bool KeyRing::verifyFileSignature( const Pathname &file, const Pathname &signature)
-  {
-    return _pimpl->verifyFileSignature(file, signature);
-  }
+  std::list<PublicKeyData> KeyRing::publicKeyData()
+  { return _pimpl->publicKeyData(); }
 
-  bool KeyRing::verifyFileTrustedSignature( const Pathname &file, const Pathname &signature)
-  {
-    return _pimpl->verifyFileTrustedSignature(file, signature);
-  }
+  std::list<PublicKeyData> KeyRing::trustedPublicKeyData()
+  { return _pimpl->trustedPublicKeyData(); }
 
-  void KeyRing::dumpPublicKey( const string &id, bool trusted, ostream &stream )
-  {
-    _pimpl->dumpPublicKey( id, trusted, stream);
-  }
+  bool KeyRing::verifyFileSignatureWorkflow(
+      const Pathname & file,
+      const std::string filedesc,
+      const Pathname & signature,
+      const KeyContext & keycontext )
+  { return _pimpl->verifyFileSignatureWorkflow( file, filedesc, signature, keycontext ); }
+
+  bool KeyRing::verifyFileSignature( const Pathname & file, const Pathname & signature )
+  { return _pimpl->verifyFileSignature( file, signature ); }
+
+  bool KeyRing::verifyFileTrustedSignature( const Pathname & file, const Pathname & signature )
+  { return _pimpl->verifyFileTrustedSignature( file, signature ); }
+
+  void KeyRing::dumpPublicKey( const std::string & id, bool trusted, std::ostream & stream )
+  { _pimpl->dumpPublicKey( id, trusted, stream ); }
+
+  PublicKey KeyRing::exportPublicKey( const PublicKeyData & keyData )
+  { return _pimpl->exportPublicKey( keyData ); }
+
+  PublicKey KeyRing::exportTrustedPublicKey( const PublicKeyData & keyData )
+  { return _pimpl->exportTrustedPublicKey( keyData ); }
+
+  bool KeyRing::isKeyTrusted( const std::string & id )
+  { return _pimpl->isKeyTrusted( id ); }
+
+  bool KeyRing::isKeyKnown( const std::string & id )
+  { return _pimpl->isKeyKnown( id ); }
 
-  bool KeyRing::isKeyTrusted( const string &id )
-  {
-    return _pimpl->isKeyTrusted(id);
-  }
-     
-  bool KeyRing::isKeyKnown( const string &id )
-  {
-    return _pimpl->isKeyKnown(id);
-  }
-  
   /////////////////////////////////////////////////////////////////
 } // namespace zypp
 ///////////////////////////////////////////////////////////////////