diff options
| author | Melvin Keskin <melvo@olomono.de> | 2022-04-07 18:10:42 +0200 |
|---|---|---|
| committer | Linus Jahn <lnj@kaidan.im> | 2022-04-09 22:27:12 +0200 |
| commit | a42c11570b9bba3465fb79dc936de6cb3c7c48cb (patch) | |
| tree | 0d5a417e9fa1972fd89797c074bbbef3113fd51f | |
| parent | d306ed08aa9d681a0b51ebbf2ec2a00b70b61018 (diff) | |
| download | qxmpp-a42c11570b9bba3465fb79dc936de6cb3c7c48cb.tar.gz | |
PubSubManager: Add requestFeatures()
| -rw-r--r-- | src/client/QXmppPubSubManager.cpp | 59 | ||||
| -rw-r--r-- | src/client/QXmppPubSubManager.h | 18 | ||||
| -rw-r--r-- | tests/qxmpppubsubmanager/tst_qxmpppubsubmanager.cpp | 103 |
3 files changed, 180 insertions, 0 deletions
diff --git a/src/client/QXmppPubSubManager.cpp b/src/client/QXmppPubSubManager.cpp index 8e2a1b6f..00d537db 100644 --- a/src/client/QXmppPubSubManager.cpp +++ b/src/client/QXmppPubSubManager.cpp @@ -79,6 +79,13 @@ using namespace QXmpp::Private; /// /// +/// \typedef QXmppPubSubManager::FeaturesResult +/// +/// Type containing service discovery features, InvalidServiceType if the service is not of the +/// desired type or the returned IQ error (QXmppStanza::Error). +/// + +/// /// \typedef QXmppPubSubManager::NodesResult /// /// Type containing a list of node names or the returned IQ error @@ -171,6 +178,58 @@ QXmppPubSubManager::~QXmppPubSubManager() } /// +/// Requests all features of a pubsub service and checks the identities via service discovery. +/// +/// This uses a \xep{0030, Service Discovery} info request to get the service +/// identities and features. +/// +/// The features are only returned if the service is of type serviceType, +/// otherwise InvalidServiceType is returned. +/// +/// \warning THIS API IS NOT FINALIZED YET! +/// +/// \param serviceJid JID of the entity hosting the pubsub service +/// \param serviceType type of service to retrieve features for +/// +QFuture<QXmppPubSubManager::FeaturesResult> QXmppPubSubManager::requestFeatures(const QString &serviceJid, ServiceType serviceType) +{ + QXmppDiscoveryIq request; + request.setType(QXmppIq::Get); + request.setQueryType(QXmppDiscoveryIq::InfoQuery); + request.setTo(serviceJid); + + return chainIq(client()->sendIq(std::move(request)), this, [=](QXmppDiscoveryIq &&iq) -> FeaturesResult { + const auto identities = iq.identities(); + + const auto isPubSubServiceFound = std::any_of(identities.cbegin(), identities.cend(), [=](const QXmppDiscoveryIq::Identity &identity) { + if (identity.category() == QStringLiteral("pubsub")) { + const auto identityType = identity.type(); + + switch (serviceType) { + case PubSubOrPep: + return identityType == QStringLiteral("service") || identityType == QStringLiteral("pep"); + case PubSub: + return identityType == QStringLiteral("service"); + case Pep: + return identityType == QStringLiteral("pep"); + } + } + return false; + }); + + if (isPubSubServiceFound) { +#if QT_VERSION >= QT_VERSION_CHECK(6, 0, 0) + return iq.features(); +#else + return iq.features().toVector(); +#endif + } + + return InvalidServiceType(); + }); +} + +/// /// Requests all listed nodes of a pubsub service via service discovery. /// /// This uses a \xep{0030, Service Discovery} items request to get a list of diff --git a/src/client/QXmppPubSubManager.h b/src/client/QXmppPubSubManager.h index 8e02f801..8c7a014b 100644 --- a/src/client/QXmppPubSubManager.h +++ b/src/client/QXmppPubSubManager.h @@ -26,12 +26,28 @@ class QXMPP_EXPORT QXmppPubSubManager : public QXmppClientExtension public: /// + /// Type of PubSub service + /// + enum ServiceType { + PubSubOrPep, ///< PubSub service or PEP service + PubSub, ///< PubSub service only + Pep ///< PEP service only + }; + + /// /// Pre-defined ID of a PubSub item /// enum StandardItemId { Current ///< Item of a singleton node }; + /// + /// Used to indicate a service type mismatch. + /// + struct InvalidServiceType + { + }; + template<typename T> struct Items { @@ -40,6 +56,7 @@ public: }; using Result = std::variant<QXmpp::Success, QXmppStanza::Error>; + using FeaturesResult = std::variant<QVector<QString>, InvalidServiceType, QXmppStanza::Error>; using NodesResult = std::variant<QVector<QString>, QXmppStanza::Error>; using InstantNodeResult = std::variant<QString, QXmppStanza::Error>; template<typename T> @@ -58,6 +75,7 @@ public: ~QXmppPubSubManager(); // Generic PubSub (the PubSub service is the given entity) + QFuture<FeaturesResult> requestFeatures(const QString &serviceJid, ServiceType serviceType = PubSubOrPep); QFuture<NodesResult> fetchNodes(const QString &jid); QFuture<Result> createNode(const QString &jid, const QString &nodeName); QFuture<Result> createNode(const QString &jid, const QString &nodeName, const QXmppPubSubNodeConfig &config); diff --git a/tests/qxmpppubsubmanager/tst_qxmpppubsubmanager.cpp b/tests/qxmpppubsubmanager/tst_qxmpppubsubmanager.cpp index 602e9c39..11c41dd2 100644 --- a/tests/qxmpppubsubmanager/tst_qxmpppubsubmanager.cpp +++ b/tests/qxmpppubsubmanager/tst_qxmpppubsubmanager.cpp @@ -26,6 +26,9 @@ using PSManager = QXmppPubSubManager; using Affiliation = QXmppPubSubAffiliation; using AffiliationType = QXmppPubSubAffiliation::Affiliation; +const char *ns_pubsub = "http://jabber.org/protocol/pubsub"; +const char *ns_pubsub_auto_create = "http://jabber.org/protocol/pubsub#auto-create"; + class TestEventManager : public QXmppPubSubEventManager { public: @@ -67,6 +70,7 @@ class tst_QXmppPubSubManager : public QObject private: Q_SLOT void testDiscoFeatures(); + Q_SLOT void testRequestFeatures(); Q_SLOT void testFetchNodes(); Q_SLOT void testFetchPepNodes(); Q_SLOT void testCreateNodes_data(); @@ -115,6 +119,105 @@ void tst_QXmppPubSubManager::testDiscoFeatures() QCOMPARE(manager.discoveryFeatures(), QStringList { "http://jabber.org/protocol/pubsub#rsm" }); } +void tst_QXmppPubSubManager::testRequestFeatures() +{ + auto [test, psManager] = Client(); + + auto future = psManager->requestFeatures("pubsub.shakespeare.lit"); + test.expect(QStringLiteral("<iq id='qxmpp1' to='pubsub.shakespeare.lit' type='get'>" + "<query xmlns='http://jabber.org/protocol/disco#info'/>" + "</iq>")); + test.inject(QStringLiteral("<iq id='qxmpp1' from='pubsub.shakespeare.lit' to='francisco@denmark.lit/barracks' type='result'>" + "<query xmlns='http://jabber.org/protocol/disco#info'>" + "<feature var='http://jabber.org/protocol/pubsub'/>" + "<feature var='http://jabber.org/protocol/pubsub#auto-create'/>" + "</query></iq>")); + + expectFutureVariant<QXmppPubSubManager::InvalidServiceType>(future); + + future = psManager->requestFeatures("pubsub.shakespeare.lit"); + test.expect(QStringLiteral("<iq id='qxmpp1' to='pubsub.shakespeare.lit' type='get'>" + "<query xmlns='http://jabber.org/protocol/disco#info'/>" + "</iq>")); + test.inject(QStringLiteral("<iq id='qxmpp1' from='pubsub.shakespeare.lit' to='francisco@denmark.lit/barracks' type='result'>" + "<query xmlns='http://jabber.org/protocol/disco#info'>" + "<identity category='pubsub' type='service'/>" + "<feature var='http://jabber.org/protocol/pubsub'/>" + "<feature var='http://jabber.org/protocol/pubsub#auto-create'/>" + "</query></iq>")); + + auto features = expectFutureVariant<QVector<QString>>(future); + QCOMPARE(features, (QVector<QString> { ns_pubsub, ns_pubsub_auto_create })); + + future = psManager->requestFeatures("juliet@capulet.lit"); + test.expect(QStringLiteral("<iq id='qxmpp1' to='juliet@capulet.lit' type='get'>" + "<query xmlns='http://jabber.org/protocol/disco#info'/>" + "</iq>")); + test.inject(QStringLiteral("<iq id='qxmpp1' from='juliet@capulet.lit' to='juliet@capulet.lit/balcony' type='result'>" + "<query xmlns='http://jabber.org/protocol/disco#info'>" + "<identity category='pubsub' type='pep'/>" + "<feature var='http://jabber.org/protocol/pubsub'/>" + "<feature var='http://jabber.org/protocol/pubsub#auto-create'/>" + "</query></iq>")); + + features = expectFutureVariant<QVector<QString>>(future); + QCOMPARE(features, (QVector<QString> { ns_pubsub, ns_pubsub_auto_create })); + + future = psManager->requestFeatures("juliet@capulet.lit", QXmppPubSubManager::PubSub); + test.expect(QStringLiteral("<iq id='qxmpp1' to='juliet@capulet.lit' type='get'>" + "<query xmlns='http://jabber.org/protocol/disco#info'/>" + "</iq>")); + test.inject(QStringLiteral("<iq id='qxmpp1' from='juliet@capulet.lit' to='juliet@capulet.lit/balcony' type='result'>" + "<query xmlns='http://jabber.org/protocol/disco#info'>" + "<identity category='pubsub' type='pep'/>" + "<feature var='http://jabber.org/protocol/pubsub'/>" + "<feature var='http://jabber.org/protocol/pubsub#auto-create'/>" + "</query></iq>")); + + expectFutureVariant<QXmppPubSubManager::InvalidServiceType>(future); + + future = psManager->requestFeatures("pubsub.shakespeare.lit", QXmppPubSubManager::PubSub); + test.expect(QStringLiteral("<iq id='qxmpp1' to='pubsub.shakespeare.lit' type='get'>" + "<query xmlns='http://jabber.org/protocol/disco#info'/>" + "</iq>")); + test.inject(QStringLiteral("<iq id='qxmpp1' from='pubsub.shakespeare.lit' to='francisco@denmark.lit/barracks' type='result'>" + "<query xmlns='http://jabber.org/protocol/disco#info'>" + "<identity category='pubsub' type='service'/>" + "<feature var='http://jabber.org/protocol/pubsub'/>" + "<feature var='http://jabber.org/protocol/pubsub#auto-create'/>" + "</query></iq>")); + + features = expectFutureVariant<QVector<QString>>(future); + QCOMPARE(features, (QVector<QString> { ns_pubsub, ns_pubsub_auto_create })); + + future = psManager->requestFeatures("pubsub.shakespeare.lit", QXmppPubSubManager::Pep); + test.expect(QStringLiteral("<iq id='qxmpp1' to='pubsub.shakespeare.lit' type='get'>" + "<query xmlns='http://jabber.org/protocol/disco#info'/>" + "</iq>")); + test.inject(QStringLiteral("<iq id='qxmpp1' from='pubsub.shakespeare.lit' to='francisco@denmark.lit/barracks' type='result'>" + "<query xmlns='http://jabber.org/protocol/disco#info'>" + "<identity category='pubsub' type='service'/>" + "<feature var='http://jabber.org/protocol/pubsub'/>" + "<feature var='http://jabber.org/protocol/pubsub#auto-create'/>" + "</query></iq>")); + + expectFutureVariant<QXmppPubSubManager::InvalidServiceType>(future); + + future = psManager->requestFeatures("juliet@capulet.lit", QXmppPubSubManager::Pep); + test.expect(QStringLiteral("<iq id='qxmpp1' to='juliet@capulet.lit' type='get'>" + "<query xmlns='http://jabber.org/protocol/disco#info'/>" + "</iq>")); + test.inject(QStringLiteral("<iq id='qxmpp1' from='juliet@capulet.lit' to='juliet@capulet.lit/balcony' type='result'>" + "<query xmlns='http://jabber.org/protocol/disco#info'>" + "<identity category='pubsub' type='pep'/>" + "<feature var='http://jabber.org/protocol/pubsub'/>" + "<feature var='http://jabber.org/protocol/pubsub#auto-create'/>" + "</query></iq>")); + + features = expectFutureVariant<QVector<QString>>(future); + QCOMPARE(features, (QVector<QString> { ns_pubsub, ns_pubsub_auto_create })); +} + void tst_QXmppPubSubManager::testFetchNodes() { auto [test, psManager] = Client(); |
