-
lovetox
Zash, MattJ is it somewhere written down what the avatar conversion module logic is
-
lovetox
regarding respecting access controls of pubsub
-
lovetox
when someone querys a vcard-temp
-
MattJ
It essentially translates the vcard request to PEP requests, so the access control is the same
-
MattJ
so if your PEP node(s) are contacts-only, only contacts will be able to query vcard-temp
-
MattJ
for example
-
lovetox
ok for requests this is probably easy
-
lovetox
what about publishing, what if i publish with vcard-temp an avatar
-
lovetox
would that change my pubsub access-model to open?
-
lovetox
i guess not
-
MattJ
https://hg.prosody.im/trunk/file/tip/plugins/mod_vcard_legacy.lua#l157
-
MattJ
If the node doesn't exist, it will be open by default (because that's the traditional behaviour of mod_vcard)
-
MattJ
But if it is configured, the existing settings will not be overwritten
-
lovetox
yeah sound sane
-
lovetox
I ask because i was talking about it with Holger, because ejabberd does not have that logic yet
-
lovetox
so its basically, if a node exists apply access control also to the vcard-temp module
-
lovetox
if not, just do as always
-
MattJ
Yep. It helps to consider that the vcard-temp module is now doing nothing except translating the request to a PEP request
-
MattJ
or a save to a PEP save
-
lovetox
Sounds also easier than maintaining 2 storages and trying to sync them
-
MattJ
Definitely
-
edhelas
So temp is going to not be temp soon
-
Zash
Except now it's used for MUCs as well.
-
Zash
And participant avatars in MUC are weird
-
mathieui
I have a PEP question, would it be possible to expose PEP nodes on a server’s JID, and would it be a protocol violation to allow multiple publishers there (e.g. server admins)?
-
Zash
mathieui, isn't that just plain ol' pubsub then?
-
Zash
or, how much of the automagic subscription stuff do you need from PEP?
-
mathieui
Zash, well, I guess that’s just pubsub, I’m only interested in the +notify
-
mathieui
(and discoverability)
-
Zash
Depending on how much PEP is just pubsub with some sensible defaults... tho a big thing is being attached to your account
-
Zash
Except for +notify, at least with Prosody, you pretty much get that by loading mod_pubsub on a normal host.
-
mathieui
The only thing is that this is something I would like to have easily implemented in clients as a receiver role, so it’s better if it does not require more features
-
mathieui
but I guess that canb be specified without too much of a hassle
-
mathieui
(publishing is another matter, but this could be an adhoc command too for all I care)
-
Zash
> would it be a protocol violation to allow multiple publishers there (e.g. server admins)? no, not a protocol violation, just a matter of how much of pubsub you support
-
Zash
Does PEP even mandate the affiliation management parts of pubsub?
-
Zash
Huh!
-
Zash
> A PEP service MUST: > - Support the "open", "roster", and "whitelist" access models.
-
Zash
I thought 'open' and 'whitelist' were optional.
-
mathieui
another question would be; is a pubsub node on the base server JID something that is possible at all in current implementions?✎ -
mathieui
another question would be; is a pubsub node on the base server JID something possible at all in current implementions? ✏
-
Zash
Prosody can. Ejabberd docs doesn't seem to say you can't, but don't know.