-
edhelas
One small debugging test for you guys
-
edhelas
https://upload.movim.eu/files/9d94237298995552fa13436420195fbca436dce7/jveoIhOIeLvd/SPOILER_SPOILER_876d8fd6ad580a1b.webm
-
edhelas
I had to fix my CSS to prevent some layout issues :D
-
Martin
Low quality vid of a dancing cat?
-
edhelas
More the fact that the video size is not stable
-
meson
On Android 13: Simple Gallery's built in video player doesn't play it at all. Google Fotos fails hard too. VLC plays fine up to the first second, then the video freezes but the sound continues. Aves is able to play it; :)✎ -
meson
On Android 13: Simple Gallery's built in video player doesn't play it at all. Google Fotos fails hard too. VLC plays fine up to the first second, then the video freezes but the sound continues. Aves is able to play it! :) ✏
-
lovetox
its a webm, it opens in firefox on my desktop, and plays fine
-
Menel
Can confirm what meson said. Also cheogram doesn't even download it. "ungültige Datei" That's worse then a file that's compleatly unknown. They would be downloaded generally. So... Someting is broken with it?
-
jonas’
firefox also seems unhappy, though I'm not sure if that's due to Content-Disposition: attachment
-
MSavoritias (fae,ve)
icecat plays it fine
-
MSavoritias (fae,ve)
as fine as it can be with changing the size all the time
-
jonas’
ah when I open it from disk firefox plays it, so it's content-disposition: attachment indeed
-
pep.
jonas’, yeah, edhelas added it on movim.eu and indeed it makes Firefox download every single attachment instead of displaying them
-
theTedd
edhelas, opening it in VLC, playing on loop, and viewing Tools->Codec Information, the resolution bounces between large and small; I'd guess it's meant to be some kind of zoom in & out effect, but it doesn't work correctly for most players since they expect a constant size within the same stream
-
Menel
I even see rhat effect for one or two seconds in vlc... Then it freezes
-
jonas’
pep., edhelas, why though?
-
edhelas
jonas’ it was a quick fix for some security issues, prevent some JS/CSS to be executed in the browser
-
pep.
The proper fix here is to restrict the input mimetype
-
edhelas
NOPE
-
edhelas
Forget it
-
pep.
Not to add a weird header
-
edhelas
mimetype can be easily workedaround
-
jonas’
edhelas, https://github.com/horazont/xmpp-http-upload/blob/master/xhu.py#L172-L174
-
jonas’
what about these three?
-
edhelas
Maybe
-
edhelas
As I said it was just a quickfix to fix the security issue
-
jonas’
pep., edhelas is right about constraining content-type not being enough; some browsers will bypass that and guess based on the content (hence the nosniff)
-
edhelas
jonas’ thanks for the link, I'll try it out
-
pep.
Well restricting input mimetype is still necessary, otherwise the client needs to guess as well
-
edhelas
I don't want to forbid users to upload JS or CSS files
-
pep.
Then what's the point of the header if you're going to interpret the file as given to you anyway
-
pep.
UX-wise outside of movim anyway it's a pita
-
edhelas
> Then what's the point of the header if you're going to interpret the file as given to you anyway No that is exactly the point of the header ↺
-
edhelas
I don't want an uploaded JS file to be executed inside the Movim page
-
edhelas
I had a CVE for that https://cve.mitre.org/cgi-bin/cvename.cgi?name=CVE-2023-2849
-
edhelas
Not released yet
-
jonas’
11:58:50 pep.> Then what's the point of the header if you're going to interpret the file as given to you anyway ask edge/IE
-
pep.
No but seriously you don't find this a pain? jonas’ you also use poezio+browser. I guess you don't come by movim.eu links often
-
jonas’
yeah, I do find this to be a pain, and I generally don't get in contact with movim.eu links indeed
-
edhelas
pep. listen, I want to be clear, I __understood your problem with it__ and I WILL take care of it
-
lovetox
is it possible to extend the roster with custom elements?
-
lovetox
or let me ask the question differently, say i have a company with 100.000 employees
-
lovetox
how would we support that use case with xmpp?
-
lovetox
if i want to start a conversation i want to find people, and if i find them i want meta infos, like department, phone number etc (basically vcard stuff)
-
lovetox
now i need the presence status, so they need to be all in my roster
-
Kev
Sounds like 55
👍️ 1 -
lovetox
is this problematic? with 100.000 ? probably not because the full roster is only downloaded at first connect to the server
-
lovetox
but i get a lot of presence info i never need
-
Kev
You wouldn't put 100,000 people in your roster, you'd use 55 for it. Or I would, anyawy.✎ -
Zash
I'd probably loook at XEP-0055 and presence probes
-
Kev
You wouldn't put 100,000 people in your roster, you'd use 55 for it. Or I would, anyway. ✏
-
Zash
For some corporate server it'd probably make some sense to allow and respond to presence probes inside the server/company.
-
lovetox
so 0055, and it returns all kind of metainfo, in essence its a dataform so it can return anything
-
Zash
On the other hand, I'm no fan of the multiple results part of XEP-0004 ./✎ -
Zash
On the other hand, I'm no fan of the multiple results part of XEP-0004 :/ ✏
-
Kev
It's not the nicest thing, but there are probably better oceans to boil.
-
lovetox
it could also return the current presence data
-
lovetox
and then i open a conversation, and then i presence probe? and this tells the server i want updates?
-
Zash
And of course where you already have them in your roster, you can use presence from that
-
lovetox
either way, there are some XEPs that lay some groundwork, but sounds all very much like it needs special server customizatuons✎ -
lovetox
either way, there are some XEPs that lay some groundwork, but sounds all very much like it needs special server customizations ✏
-
Kev
Depends how much you care about the presence part. People we've dealt with haven't.
-
Kev
Without presence, 55 works out of the box.
-
lovetox
you met people which in a company setting, dont care what show (online, dnd, away) a other user has?
-
Kev
When searching for people outside their preset roster groups, right.
-
Zash
You could add them.
-
Kev
Indeed.
-
lovetox
ok so you would put some people in the roster, like people working in the same country, but others not, which are less likely to be contacted
-
Zash
Didn't we talk about moving presence stuff into PEP?
-
Zash
That would help with e.g. "in a meeting" kind of statuses
-
Kev
> Didn't we talk about moving presence stuff into PEP? Yes. Really should get around to actually make that happen.
-
Zash
User Activity :)
-
lovetox
but how is this better? pep sends you also a notification per user
-
lovetox
its the same as you it were in your roster
-
Zash
lovetox, easier to query without presence subscription
-
Zash
as in, would already work with existing servers
-
lovetox
i thought the goal of pep is that i dont need to query something, and get notification on change
-
lovetox
i dont see that scale, the same as presence
-
Zash
It's pubsub. You can query pubsub without a subscription.
-
Zash
If you're opening a conversation with a few non-contacts, it's just a few queries?
-
lovetox
im not sure i understand, i need a mechanisms that updates me on changes. Of course i can query the info when i open a chat, but i need to get the info when it changes
-
Kev
Which is exactly what pubsub is about, no?
-
lovetox
so when i open a chat, i subscribe to some public node of the user
-
lovetox
and it will update me, until i unsubscribe?
-
lovetox
all without having presence subscription
-
Kev
Right.
-
lovetox
i never dealt with anything but PEP, so i never did that subscribe thing manually, but if this works that way, yes i think that would work
-
lovetox
and this bascially does not need any support from the server
-
lovetox
i could implement this right now, if my client is the only thing the company uses
-
lovetox
the 0055 would need a small server module i guess
-
Kev
Most (all?) the usual servers support 55.
-
Kev
But yes, the server does need to support 55 and 60/163 for it.
-
lovetox
55 is a generic search protocol
-
lovetox
filling it with useful information, is a server customization
-
lovetox
as i said, to make it really good, it should return much more info
-
lovetox
department, status, show, phonenumber etc