qt: Allow specifying an import filter when importing keys

* lang/qt/src/importjob.cpp (struct ImportJobPrivate): Add member
m_importFilter.
* lang/qt/src/importjob.cpp, lang/qt/src/importjob.h (class ImportJob):
Add member functions setImportFilter and importFilter.
* lang/qt/src/qgpgmeimportjob.cpp (import_qba): Add arg importFilter
and adjust the callers. Set import filter context flag.

* lang/qt/tests/t-import.cpp (ImportTest): Add member function
testImportWithImportFilter.
--

GnuPG-bug-id: 5739
This commit is contained in:
Ingo Klöcker 2021-12-15 12:23:26 +01:00
parent 3e81a4a336
commit 619579bb17
4 changed files with 79 additions and 8 deletions

View File

@ -53,11 +53,24 @@ struct ImportJobPrivate : public JobPrivate
~ImportJobPrivate() override = default; ~ImportJobPrivate() override = default;
QString m_importFilter;
Key::Origin m_keyOrigin = Key::OriginUnknown; Key::Origin m_keyOrigin = Key::OriginUnknown;
QString m_keyOriginUrl; QString m_keyOriginUrl;
}; };
} }
void QGpgME::ImportJob::setImportFilter(const QString &filter)
{
const auto d = jobPrivate<ImportJobPrivate>(this);
d->m_importFilter = filter;
}
QString QGpgME::ImportJob::importFilter() const
{
const auto d = jobPrivate<ImportJobPrivate>(this);
return d->m_importFilter;
}
void ImportJob::setKeyOrigin(GpgME::Key::Origin origin, const QString &url) void ImportJob::setKeyOrigin(GpgME::Key::Origin origin, const QString &url)
{ {
const auto d = jobPrivate<ImportJobPrivate>(this); const auto d = jobPrivate<ImportJobPrivate>(this);

View File

@ -72,6 +72,9 @@ protected:
public: public:
~ImportJob() override; ~ImportJob() override;
void setImportFilter(const QString &filter);
QString importFilter() const;
void setKeyOrigin(GpgME::Key::Origin origin, const QString &url = {}); void setKeyOrigin(GpgME::Key::Origin origin, const QString &url = {});
GpgME::Key::Origin keyOrigin() const; GpgME::Key::Origin keyOrigin() const;
QString keyOriginUrl() const; QString keyOriginUrl() const;

View File

@ -40,11 +40,9 @@
#include "dataprovider.h" #include "dataprovider.h"
#include "context.h" #include <gpgme++/context.h>
#include "data.h" #include <gpgme++/data.h>
#include "key.h" #include <gpgme++/key.h>
#include <cassert>
using namespace QGpgME; using namespace QGpgME;
using namespace GpgME; using namespace GpgME;
@ -72,8 +70,12 @@ static const char *originToString(Key::Origin origin)
return (it != std::end(mapping)) ? it->second : nullptr; return (it != std::end(mapping)) ? it->second : nullptr;
} }
static QGpgMEImportJob::result_type import_qba(Context *ctx, const QByteArray &certData, Key::Origin keyOrigin, const QString &keyOriginUrl) static QGpgMEImportJob::result_type import_qba(Context *ctx, const QByteArray &certData, const QString &importFilter,
Key::Origin keyOrigin, const QString &keyOriginUrl)
{ {
if (!importFilter.isEmpty()) {
ctx->setFlag("import-filter", importFilter.toStdString().c_str());
}
if (keyOrigin != Key::OriginUnknown) { if (keyOrigin != Key::OriginUnknown) {
if (const auto origin = originToString(keyOrigin)) { if (const auto origin = originToString(keyOrigin)) {
std::string value{origin}; std::string value{origin};
@ -96,13 +98,13 @@ static QGpgMEImportJob::result_type import_qba(Context *ctx, const QByteArray &c
Error QGpgMEImportJob::start(const QByteArray &certData) Error QGpgMEImportJob::start(const QByteArray &certData)
{ {
run(std::bind(&import_qba, std::placeholders::_1, certData, keyOrigin(), keyOriginUrl())); run(std::bind(&import_qba, std::placeholders::_1, certData, importFilter(), keyOrigin(), keyOriginUrl()));
return Error(); return Error();
} }
GpgME::ImportResult QGpgME::QGpgMEImportJob::exec(const QByteArray &keyData) GpgME::ImportResult QGpgME::QGpgMEImportJob::exec(const QByteArray &keyData)
{ {
const result_type r = import_qba(context(), keyData, keyOrigin(), keyOriginUrl()); const result_type r = import_qba(context(), keyData, importFilter(), keyOrigin(), keyOriginUrl());
resultHook(r); resultHook(r);
return mResult; return mResult;
} }

View File

@ -69,6 +69,59 @@ private Q_SLOTS:
qputenv("GNUPGHOME", tempGpgHome.path().toLocal8Bit()); qputenv("GNUPGHOME", tempGpgHome.path().toLocal8Bit());
} }
void testImportWithImportFilter()
{
if (GpgME::engineInfo(GpgME::GpgEngine).engineVersion() < "2.1.14") {
QSKIP("gpg does not yet support the --import-filter option");
}
// pub ed25519 2021-12-15 [SC]
// E7A0841292ACC9465D3142652FB3A6F51FBF28A2
// uid [ultimate] importWithImportFilter@example.com
// uid [ultimate] importWithImportFilter@example.net
// sub cv25519 2021-12-15 [E]
static const char keyFpr[] = "E7A0841292ACC9465D3142652FB3A6F51FBF28A2";
static const char keyData[] =
"-----BEGIN PGP PUBLIC KEY BLOCK-----\n"
"\n"
"mDMEYbm2PhYJKwYBBAHaRw8BAQdACzxBWtNNsmJ6rzpZkjh1yBe+Ajsk9NR8umEu\n"
"Da3HLgG0ImltcG9ydFdpdGhJbXBvcnRGaWx0ZXJAZXhhbXBsZS5uZXSIlAQTFgoA\n"
"PBYhBOeghBKSrMlGXTFCZS+zpvUfvyiiBQJhubY+AhsDBQsJCAcCAyICAQYVCgkI\n"
"CwIEFgIDAQIeBwIXgAAKCRAvs6b1H78oosRgAQCc/ke6q076nvzIE2UzT83JK/B6\n"
"lxSV7Fb8bKltOMpvsAD+Phap3EzA8jdMyKoO0FM926bw5lX7QROfeZ/JBYqyPwC0\n"
"ImltcG9ydFdpdGhJbXBvcnRGaWx0ZXJAZXhhbXBsZS5jb22IlAQTFgoAPBYhBOeg\n"
"hBKSrMlGXTFCZS+zpvUfvyiiBQJhubZlAhsDBQsJCAcCAyICAQYVCgkICwIEFgID\n"
"AQIeBwIXgAAKCRAvs6b1H78oohPWAQC/u9UXzkxRkrB2huaTZCsyimWEGZIMmxWd\n"
"tE+vN9/IvQD/Yzia+xRS6yca3Yz6iW8xS844ZqRxvkUEHjtJXSOzagm4OARhubY+\n"
"EgorBgEEAZdVAQUBAQdANQFjmDctY3N0/ELPZtj9tapwFs4vrmTVpx/SCfZmihkD\n"
"AQgHiHgEGBYKACAWIQTnoIQSkqzJRl0xQmUvs6b1H78oogUCYbm2PgIbDAAKCRAv\n"
"s6b1H78oovGyAP41ySzvvDpV7XDJBOAFxvWLmywa5IcO7Lrg7y1efoWj0AD+Kk/B\n"
"s7jGLdoG51h670h50MMoYCANB6MwAdSP+qZUlQg=\n"
"=/3O0\n"
"-----END PGP PUBLIC KEY BLOCK-----\n";
auto *job = openpgp()->importJob();
job->setImportFilter(QLatin1String{"keep-uid=mbox = importWithImportFilter@example.net"});
connect(job, &ImportJob::result, this,
[this](ImportResult result, QString, Error)
{
QVERIFY(!result.error());
QVERIFY(!result.imports().empty());
QVERIFY(result.numImported());
Q_EMIT asyncDone();
});
job->start(QByteArray{keyData});
QSignalSpy spy (this, SIGNAL(asyncDone()));
QVERIFY(spy.wait());
auto ctx = Context::createForProtocol(GpgME::OpenPGP);
GpgME::Error err;
const auto key = ctx->key(keyFpr, err, false);
QVERIFY(!key.isNull());
QCOMPARE(key.numUserIDs(), 1);
QCOMPARE(key.userID(0).id(), "importWithImportFilter@example.net");
}
void testImportWithKeyOrigin() void testImportWithKeyOrigin()
{ {
if (GpgME::engineInfo(GpgME::GpgEngine).engineVersion() < "2.1.22") { if (GpgME::engineInfo(GpgME::GpgEngine).engineVersion() < "2.1.22") {