aboutsummaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorLinus Jahn <lnj@kaidan.im>2023-01-28 17:58:01 +0100
committerLinus Jahn <lnj@kaidan.im>2023-01-28 17:58:01 +0100
commit8283bcff0b142e2f40daa82a452b002b61952000 (patch)
tree2e70406c07564de9f8388dead6a157af564c9300
parent0c8209919cf13691fd9e2a1306a28259c1ca1657 (diff)
downloadqxmpp-8283bcff0b142e2f40daa82a452b002b61952000.tar.gz
Client: Rename send/sendUnencrypted to sendSensitive/send
-rw-r--r--src/client/QXmppAtmManager.cpp2
-rw-r--r--src/client/QXmppClient.cpp10
-rw-r--r--src/client/QXmppClient.h2
-rw-r--r--src/client/QXmppRosterManager.cpp4
-rw-r--r--src/omemo/QXmppOmemoManager_p.cpp2
-rw-r--r--tests/qxmppclient/tst_qxmppclient.cpp8
-rw-r--r--tests/qxmppomemomanager/tst_qxmppomemomanager.cpp4
7 files changed, 16 insertions, 16 deletions
diff --git a/src/client/QXmppAtmManager.cpp b/src/client/QXmppAtmManager.cpp
index e6df2679..8ec0fdc6 100644
--- a/src/client/QXmppAtmManager.cpp
+++ b/src/client/QXmppAtmManager.cpp
@@ -492,5 +492,5 @@ QXmppTask<QXmpp::SendResult> QXmppAtmManager::sendTrustMessage(const QString &en
QXmppSendStanzaParams params;
params.setAcceptedTrustLevels(TrustLevel::Authenticated);
- return client()->send(std::move(message), params);
+ return client()->sendSensitive(std::move(message), params);
}
diff --git a/src/client/QXmppClient.cpp b/src/client/QXmppClient.cpp
index 891de237..747e7200 100644
--- a/src/client/QXmppClient.cpp
+++ b/src/client/QXmppClient.cpp
@@ -399,7 +399,7 @@ bool QXmppClient::sendPacket(const QXmppNonza &packet)
///
/// If stream management is enabled, the future continues to be active until the
/// server acknowledges the packet. On success, QXmpp::SendSuccess with
-/// acknowledged == true is reported and the future finishes.
+/// acknowledged == true is reported.
///
/// If connection errors occur, the packet is resent if possible. If
/// reconnecting is not possible, an error is reported.
@@ -410,7 +410,7 @@ bool QXmppClient::sendPacket(const QXmppNonza &packet)
///
/// \since QXmpp 1.5
///
-QXmppTask<QXmpp::SendResult> QXmppClient::send(QXmppStanza &&stanza, const std::optional<QXmppSendStanzaParams> &params)
+QXmppTask<QXmpp::SendResult> QXmppClient::sendSensitive(QXmppStanza &&stanza, const std::optional<QXmppSendStanzaParams> &params)
{
const auto sendEncrypted = [this](QXmppTask<MessageEncryptResult> &&future) {
QXmppPromise<QXmpp::SendResult> interface;
@@ -454,7 +454,7 @@ QXmppTask<QXmpp::SendResult> QXmppClient::send(QXmppStanza &&stanza, const std::
///
/// \since QXmpp 1.5
///
-QXmppTask<QXmpp::SendResult> QXmppClient::sendUnencrypted(QXmppStanza &&stanza, const std::optional<QXmppSendStanzaParams> &)
+QXmppTask<QXmpp::SendResult> QXmppClient::send(QXmppStanza &&stanza, const std::optional<QXmppSendStanzaParams> &)
{
return d->stream->send(stanza);
}
@@ -473,9 +473,9 @@ QXmppTask<QXmpp::SendResult> QXmppClient::reply(QXmppStanza &&stanza, const std:
// This should pick the right e2ee manager as soon as multiple encryptions
// in parallel are supported.
if (e2eeMetadata) {
- return send(std::move(stanza), params);
+ return sendSensitive(std::move(stanza), params);
}
- return sendUnencrypted(std::move(stanza), params);
+ return send(std::move(stanza), params);
}
///
diff --git a/src/client/QXmppClient.h b/src/client/QXmppClient.h
index ca33afb6..31a99030 100644
--- a/src/client/QXmppClient.h
+++ b/src/client/QXmppClient.h
@@ -219,8 +219,8 @@ public:
State state() const;
QXmppStanza::Error::Condition xmppStreamError();
+ QXmppTask<QXmpp::SendResult> sendSensitive(QXmppStanza &&, const std::optional<QXmppSendStanzaParams> & = {});
QXmppTask<QXmpp::SendResult> send(QXmppStanza &&, const std::optional<QXmppSendStanzaParams> & = {});
- QXmppTask<QXmpp::SendResult> sendUnencrypted(QXmppStanza &&, const std::optional<QXmppSendStanzaParams> & = {});
QXmppTask<QXmpp::SendResult> reply(QXmppStanza &&stanza, const std::optional<QXmppE2eeMetadata> &e2eeMetadata, const std::optional<QXmppSendStanzaParams> & = {});
QXmppTask<IqResult> sendIq(QXmppIq &&, const std::optional<QXmppSendStanzaParams> & = {});
QXmppTask<IqResult> sendSensitiveIq(QXmppIq &&, const std::optional<QXmppSendStanzaParams> & = {});
diff --git a/src/client/QXmppRosterManager.cpp b/src/client/QXmppRosterManager.cpp
index 04b74c43..c02d2211 100644
--- a/src/client/QXmppRosterManager.cpp
+++ b/src/client/QXmppRosterManager.cpp
@@ -341,7 +341,7 @@ QXmppTask<QXmpp::SendResult> QXmppRosterManager::subscribeTo(const QString &bare
packet.setTo(QXmppUtils::jidToBareJid(bareJid));
packet.setType(QXmppPresence::Subscribe);
packet.setStatusText(reason);
- return client()->send(std::move(packet));
+ return client()->sendSensitive(std::move(packet));
}
///
@@ -358,7 +358,7 @@ QXmppTask<QXmpp::SendResult> QXmppRosterManager::unsubscribeFrom(const QString &
packet.setTo(QXmppUtils::jidToBareJid(bareJid));
packet.setType(QXmppPresence::Unsubscribe);
packet.setStatusText(reason);
- return client()->send(std::move(packet));
+ return client()->sendSensitive(std::move(packet));
}
///
diff --git a/src/omemo/QXmppOmemoManager_p.cpp b/src/omemo/QXmppOmemoManager_p.cpp
index dea38c13..4e9d7840 100644
--- a/src/omemo/QXmppOmemoManager_p.cpp
+++ b/src/omemo/QXmppOmemoManager_p.cpp
@@ -3629,7 +3629,7 @@ QXmppTask<QXmpp::SendResult> ManagerPrivate::sendEmptyMessage(const QString &rec
message.addHint(QXmppMessage::Store);
message.setOmemoElement(omemoElement);
- auto future = q->client()->sendUnencrypted(std::move(message));
+ auto future = q->client()->send(std::move(message));
future.then(q, [=](QXmpp::SendResult result) mutable {
interface.finish(std::move(result));
});
diff --git a/tests/qxmppclient/tst_qxmppclient.cpp b/tests/qxmppclient/tst_qxmppclient.cpp
index a18fa0d2..c51fe047 100644
--- a/tests/qxmppclient/tst_qxmppclient.cpp
+++ b/tests/qxmppclient/tst_qxmppclient.cpp
@@ -129,14 +129,14 @@ void tst_QXmppClient::testE2eeExtension()
EncryptionExtension encrypter;
client.setEncryptionExtension(&encrypter);
- auto result = client.send(QXmppMessage("me@qxmpp.org", "somebody@qxmpp.org", "Hello"));
+ auto result = client.sendSensitive(QXmppMessage("me@qxmpp.org", "somebody@qxmpp.org", "Hello"));
QVERIFY(encrypter.messageCalled);
QVERIFY(!encrypter.iqCalled);
QCoreApplication::processEvents();
expectFutureVariant<QXmppError>(result.toFuture(this));
encrypter.messageCalled = false;
- result = client.send(QXmppPresence(QXmppPresence::Available));
+ result = client.sendSensitive(QXmppPresence(QXmppPresence::Available));
QVERIFY(!encrypter.messageCalled);
QVERIFY(!encrypter.iqCalled);
@@ -148,11 +148,11 @@ void tst_QXmppClient::testE2eeExtension()
return request;
};
- client.send(createRequest());
+ client.sendSensitive(createRequest());
QVERIFY(encrypter.iqCalled);
encrypter.iqCalled = false;
- client.sendUnencrypted(createRequest());
+ client.send(createRequest());
QVERIFY(!encrypter.iqCalled);
encrypter.iqCalled = false;
diff --git a/tests/qxmppomemomanager/tst_qxmppomemomanager.cpp b/tests/qxmppomemomanager/tst_qxmppomemomanager.cpp
index 85a4792b..d276a626 100644
--- a/tests/qxmppomemomanager/tst_qxmppomemomanager.cpp
+++ b/tests/qxmppomemomanager/tst_qxmppomemomanager.cpp
@@ -328,7 +328,7 @@ void tst_QXmppOmemoManager::testSendMessage()
if (const auto optionalOmemoElement = message.omemoElement(); optionalOmemoElement && optionalOmemoElement.value().payload().isEmpty()) {
isEmptyOmemoMessageReceivedByAlice1 = true;
- auto future = m_alice1.client.send(std::move(message2), QXmppSendStanzaParams());
+ auto future = m_alice1.client.sendSensitive(std::move(message2), QXmppSendStanzaParams());
future.then(this, [=, &isSecondMessageSentByAlice1](QXmpp::SendResult result) {
if (std::get_if<QXmpp::SendSuccess>(&result)) {
isSecondMessageSentByAlice1 = true;
@@ -344,7 +344,7 @@ void tst_QXmppOmemoManager::testSendMessage()
connect(m_alice1.manager, &QXmppOmemoManager::deviceAdded, &context, [=, &isFirstMessageSentByAlice1](const QString &jid, uint32_t) mutable {
if (jid == m_alice2.client.configuration().jidBare()) {
if (!isFirstMessageSentByAlice1) {
- auto future = m_alice1.client.send(std::move(message1), QXmppSendStanzaParams());
+ auto future = m_alice1.client.sendSensitive(std::move(message1), QXmppSendStanzaParams());
future.then(this, [=, &isFirstMessageSentByAlice1](QXmpp::SendResult result) {
if (std::get_if<QXmpp::SendSuccess>(&result)) {
isFirstMessageSentByAlice1 = true;