-
Sam
XSF folks at FOSSY: anyone want to grab food/beer?
-
singpolyma
Sam: ossguy and I are at the sustain it's event at hey love✎ -
singpolyma
Sam: ossguy and I are at the sustain oss event at hey love ✏
-
Sam
I'm not familiar with them, but I think they're the folks doing an interview with me tomorrow morning
-
Link Mauve
Hi, what happened to this ProtoXEP? Did it ever get voted to get a number, or rejected, or what? https://xmpp.org/extensions/inbox/muc-avatars.html
-
Kev
2018-10-05 6) AOB Link Mauve queries the status of the MUC Avatars proto-XEP (https://xmpp.org/extensions/inbox/muc-avatars.html); Kev notes that it had three +1s, and Sam vetoed on-list.
-
Link Mauve
Kev, no, this is after 2023-02-15.
-
Link Mauve
Now that it is on the Historical track.
-
Link Mauve
Following https://github.com/xsf/xeps/pull/1269
-
Kev
Ah. So needs to go back to Council then.
-
Link Mauve
Speaking of which, Sam, do you agree with this PR? https://github.com/xsf/xeps/pull/1279
-
Link Mauve
Council for https://github.com/xsf/xeps/pull/1278
-
Sam
Link Mauve: oops, missed that somehow. Yah, that's fine by me. Thanks.
-
lovetox
Very interested in the veto
-
lovetox
Gajim implements the muc avatar xep and it's the right solution in my opinion
-
singpolyma
I'm not sure anything based on vcard-temp can be "the right solution" but maybe an expedient one sometimes
-
singpolyma
I find it curious this xep says you need to be joined to get a muc avatar in the currently deployed way. Can vcard-temp not be fetched from outside? I would expect it could
-
Link Mauve
singpolyma, MUC is based on presence, you can’t send an iq to a “participant” who isn’t currently joined (they aren’t a participant in that case), nor if you aren’t joined yourself (unless the MUC is configured to let you do so).
-
singpolyma
Link Mauve: this is for the muc itself though, not a participant?
-
lovetox
singpolyma: only the upload mechanism is vcars temp and for that it is good.
-
lovetox
This xep allows exactly what you asked to fetch the avatar while not jpined✎ -
lovetox
This xep allows exactly what you asked to fetch the avatar while not joined, like all other MUC infos ✏
-
singpolyma
lovetox: but I think we already can with what we do today?
-
lovetox
No
-
singpolyma
The xep claims we can't
-
singpolyma
But I can't think of any reason why
-
Link Mauve
singpolyma, where does it say that?
-
singpolyma
Link Mauve: in the intro when talking about why this xep exists
-
Link Mauve
Section 2 says “Let users discover the avatar even when not present in the MUC.”
-
lovetox
singpolyma: are you saying you can invent a not documented protocol which would allow you to?
-
singpolyma
Anyway. In either case if we were going to change current behaviour why would we not change to pep avatar rather than speccing a custom thing for MUC?
-
Link Mauve
singpolyma, ah, third paragraph of section 1, that’s the limitation of using presence instead of disco#info.
-
singpolyma
lovetox: I'm talking about the currently in-use protocol for muc vcard
-
lovetox
Therr is no documented protocol in use
-
Link Mauve
singpolyma, the historical track is for documenting existing protocols, which haven’t been through the standards process but are widely used.
-
singpolyma
You claim vcard-temp is undocumented?
-
Link Mauve
singpolyma, usage of vCard-temp on MUC is documented only in some Ejabberd tutorial.
-
singpolyma
And in the vcard-temp xep
-
Link Mauve
Is it?
-
singpolyma
How is it not? Is the protocol different in some way I have missed?
-
lovetox
singpolyma: it's not about vcard temp storage
-
lovetox
It's about the distrbution
-
lovetox
The disocvery
-
Link Mauve
singpolyma, the only mention in XEP-0153 is that you can query other participants’ vCard-temps.
-
Link Mauve
It doesn’t mention anything about the rooms themselves.
-
Link Mauve
Which is what this ProtoXEP aims to fix.
-
singpolyma
You can query the vcard-temp on any jid. A MUC has a jid. The fact that it's a MUC shouldn't affect the protocol?
-
lovetox
It's not about the query again
-
Link Mauve
singpolyma, and if you read my ProtoXEP that’s exactly what it describes.
-
singpolyma
Link Mauve: but then why does the protoxep say we need this new protocol in order to get the avatar for a room we have not joined? That was my question
-
lovetox
And if you read the XEP it does not change anything about the query
-
Link Mauve
singpolyma, it isn’t about joined or not joined, is it?
-
Link Mauve
If you disco#info the room you get the hash of the avatar, so you know whether you support it already or not, because otherwise you have to always fetch the avatar of every room you come across whether they changed or not.
-
Link Mauve
It can make a lot of traffic on each client’s startup if the user has a lot of rooms.
-
singpolyma
Ok, I think I see what is misunderstood > Some implementations recently chose to advertise those avatars using the existing vCard-Based Avatars (XEP-0153) [2] extension in <presence/>, but it exposed issues in other implementations, and was only available when the user is already present in the room, not before joining it (for example when listing all available rooms). Sounded to me like "was only available" was referring to the avatar, but in fact is referring to the extension.✎ -
singpolyma
Ok, I think I see what I misunderstood > Some implementations recently chose to advertise those avatars using the existing vCard-Based Avatars (XEP-0153) [2] extension in <presence/>, but it exposed issues in other implementations, and was only available when the user is already present in the room, not before joining it (for example when listing all available rooms). Sounded to me like "was only available" was referring to the avatar, but in fact is referring to the extension. ✏
-
Link Mauve
singpolyma, it refers to section 5.2.
-
Link Mauve
Any suggestion as to how it could be formulated better?
-
singpolyma
I am also curious how a new protocol nt yet in use can be "Historical" but I don't care much about that and I certainly agree if anyone implements it it should be considered almost as though historical so I kinda get it
-
Link Mauve
singpolyma, Prosody already implements it, under the {http://modules.prosody.im/mod_vcard_muc}avatar#sha1 field var instead of the proposed muc#roominfo_avatarhash though.
-
singpolyma
Link Mauve: maybe it could say "exposed issues in other implementations, and this extra metadata was only available after joining the room" I think something like this is more clear than "it"
-
Zash
Prosody and ejabberd also implement the xep-0153 way that caused trouble for some clients, but if you're thinking "a jid is a jid" then that's that and no further xep is needed, and the prosody ways is not needed.
-
lovetox
It's about the discovery of the hash
-
lovetox
Of course we need a mechanism
-
lovetox
And presence is a bad one for a muc
-
Zash
Right you only get it by joining with the 153-only method
-
lovetox
We need to disco mucs anyway, and the best way is to add it there
-
Zash
Plus MUC alrady had that notification message for configuration change
-
nicoco
advertising the room avatar hash in the disco info of the room, I didn't know this was a thing in prosody. any clients use that yet?
-
Zash
Gajim?
-
lovetox
Yes you can test it on the muc join dialog nicoco
-
nicoco
cool, so something like `<field var='{http://modules.prosody.im/mod_vcard_muc}avatar#sha1'><value>thehash` in the extended disco features?
-
lovetox
Yes you can check disco info of the prosody muc
-
Zash
Right here too. :)
-
arc
Zash, missing you at Fossy. Will you go next year?
-
Zash
Across the pond? Unlikely
-
arc
Ah