Remove (unused?) PEP restriction for XEP-0084

Description

A code fragment, which appears to be part of the original PEP implementation, attempts to prevent PEP being used for XEP-0084 (User Avatar), at least for uploading an avatar.

The way that this is implemented (checking the node ID for a specific value) won’t work

It is implemented as follows in IQPepHandler:

This check should be removed, as it’s not doing anything.

There’s an argument to be made that XEP-0084 Avatar uploads do not interact with the existing VCardManager infrastructure. This is something to address, possibly with help of XEP-0398 based conversions.

Environment

None

Activity

Show:
Fixed

Details

Assignee

Reporter

Components

Fix versions

Priority

Created August 21, 2023 at 10:01 PM
Updated August 30, 2023 at 6:36 PM
Resolved August 30, 2023 at 6:36 PM