main icon indicating copy to clipboard operation
main copied to clipboard

Psi+ doesn't show message errors

Open mdosch opened this issue 5 years ago • 7 comments

Version: 1.4.554 (debian)

When a message is not delivered, Psi+ doesn't show the error therefore you don't know that the message was not delivered (e.g. because it's rejected).

<message to='REDACTED/Psi+' type='error' id='aac9a' from='[email protected]' xml:lang='en'><error type='cancel'><not-allowed xmlns='urn:ietf:params:xml:ns:xmpp-stanzas'/><text xm
lns='urn:ietf:params:xml:ns:xmpp-stanzas'>Error handling test</text></error></message>

In this case I would expect to get an indication that the message delivery failed and the message Error handling test should be shown to the user. But nothing is shown:

2020-10-30-115151_scrot

You can use xmpp:[email protected] for testing as it will also reply with an error.

mdosch avatar Oct 30 '20 11:10 mdosch

This is strange. As far as I see XEP-0086 is supported in Psi+: https://github.com/psi-im/psi/wiki/Supported-XEPs

tehnick avatar Oct 30 '20 11:10 tehnick

Error codes are deprecated and stanza errors are defined in RFC6120.

mdosch avatar Oct 30 '20 11:10 mdosch

Yeh, Psi+ supports a lot of deprecated things and does not support some of new ones...

tehnick avatar Oct 30 '20 11:10 tehnick

@Ri0n: Can you look this good @mdosch ticket?

In more, there is "reply with nonexistent account" too, specified in profanity-im ticket:

  • https://github.com/profanity-im/profanity/issues/1435

It is more a Psi ticket ^^

Neustradamus avatar Apr 25 '24 22:04 Neustradamus

да. ошибки не выводит

lao-mang avatar May 07 '24 16:05 lao-mang

There are two errors in this case:

<iq id='ID1' from='[email protected]' to='[email protected]' type='get'>
<pubsub xmlns='http://jabber.org/protocol/pubsub'>
<items node='eu.siacs.conversations.axolotl.devicelist'/>
</pubsub>
</iq>

<iq xml:lang="en" to="[email protected]/client" from="[email protected]" type="error" id="ID1">
<error type="auth">
<forbidden xmlns="urn:ietf:params:xml:ns:xmpp-stanzas"/>
</error>
</iq>

<message to="[email protected]" type="chat" id="ID2">
<body>test</body>
<active xmlns="http://jabber.org/protocol/chatstates"/>
<request xmlns="urn:xmpp:receipts"/>
<nick xmlns="http://jabber.org/protocol/nick">Neustradamus</nick>
<origin-id xmlns="urn:xmpp:sid:0" id="ID2"/>
</message>

<message xml:lang="en" to="[email protected]/client" from="[email protected]" type="error" id="ID2">
<error type="cancel">
<not-allowed xmlns="urn:ietf:params:xml:ns:xmpp-stanzas"/>
<text xmlns="urn:ietf:params:xml:ns:xmpp-stanzas">Error handling test</text>
</error>
</message>

Other errors are here:

  • https://github.com/psi-im/psi/issues/609

Neustradamus avatar May 14 '24 09:05 Neustradamus

@mdosch: Your original error is now showed with this commit:

  • https://github.com/psi-im/psi/commit/90539a9c595f50c6e9190190095b0746c0cfbc7e
Not allowed.
The recipient or server does not allow any entity to perform the action.
Error handling test
------

Psi+ 1.5.1889:

  • https://github.com/psi-plus/psi-plus-snapshots/commit/79f5e477c6dcdab477439f3d07a74540fd0b1660

@Ri0n: But not the PubSub error :/

<iq id='ID1' from='[email protected]' to='[email protected]' type='get'>
<pubsub xmlns='http://jabber.org/protocol/pubsub'>
<items node='eu.siacs.conversations.axolotl.devicelist'/>
</pubsub>
</iq>

<iq xml:lang="en" to="[email protected]/client" from="[email protected]" type="error" id="ID1">
<error type="auth">
<forbidden xmlns="urn:ietf:params:xml:ns:xmpp-stanzas"/>
</error>
</iq>

Neustradamus avatar May 15 '24 11:05 Neustradamus