-
Guus
intosi: we are in need of your magic powers once more!
-
Guus
The website is missing two resources, but it is in /extensions/ which is outside of my range of influence
-
Guus
https://xmpp.org/extensions/xep.ent should serve the file in https://github.com/xsf/xeps/blob/master/xep.ent https://xmpp.org/extensions/xep.xsl should serve the file in https://github.com/xsf/xeps/blob/master/xep.xsl
-
Guus
(I will "pretty please" if it helps)
-
intosi
Consider it done if you put sugar on top.
-
Guus
pretty please with sugar on top
- Guus brings out the whipped cream and starts shaking.
-
intosi
I'll undo the fix if you dare ;)
-
Guus
0;-)
-
Guus
Does anyone know what this issue is about? https://github.com/xsf/xmpp.org/issues/43 ralphm?
-
Guus
intosi: now it complains about a missing xep.dtd, which can be found in the same place. Could you do that one too, please?
-
Guus
(relates to https://github.com/xsf/xmpp.org/issues/196)
-
intosi
Sure
-
Guus
right, now it shows somethign
-
Guus
intosi: two new 404's: pretty.css and pretty.js - these shouldgo in xmpp,org/ (root) though, not in xmpp.org/extensions/
-
Tobias
Guus, or we change the XEP html to have them in extensions? instead of requiring tons of URL rewrites in the server
-
intosi
Or that.
-
Guus
Tobias: I am fine with any improvement, but currently, I'm pushing for the easy fix, which is something that we can do *now*.
-
intosi
pretty.js? Don't you mean prettify.js?
-
Guus
prettify, indeed
-
Guus
both css as well as js
-
intosi
And where are these referenced from?
-
Guus
You'll notice the 404's whenever you open a url like this one: https://xmpp.org/extensions/xep-0300.xml
-
Guus
the xml is loading the xsl, which in turn references the dtd, css and js, I suppose.
-
intosi
Guus: I don't, in fact. I see 200 responses there.
-
ralphm
Guus: even though we do have a page at that URL now, the original tech pages were more detailed: http://web.archive.org/web/20150313010246/http://xmpp.org/about-xmpp/technology-overview
-
intosi
Because they load /extensions/prettify.js
-
intosi
;)
-
Guus
wuh? it works for me now
-
jonasw
Guus: I get "undefined entity" &LEGALNOTICE;
-
Guus
weird caching issue, I think
-
intosi
Yeah, after I added redirects.
-
Guus
ah, ok
-
intosi
But I'd rather fix the stylesheets in the longer run.
-
Guus
jonasw: feel free to suggest improvements :)
-
Guus
intosi: agreed: but at least now, we have a non-empty page
-
intosi
Right. Back to $DAYJOB
-
Guus
likewise
-
Guus
thanks for the help
-
intosi
np
-
intosi
yw
-
jonasw
Guus: I’m not sure what the cause is. According to debugging tools, all resources are fetched freshly, but then again, it doesn’t even try to load xep.{dtd,ent}. I blame Firefox’ XML/XSL support.
-
jonasw
force-clearing the cache doesn’t help either
-
Guus
jonasw: tried another browser?
-
jonasw
on my way
-
jonasw
works with chromium
-
Guus
If you ever figure out what makes it work on firefox, please open an issue on github
-
jonasw
I’ve never used the XML versions, so I’m not sure if that’s a new issue
-
Guus
it likely is not new. Does not mean we shouldn't improve if we can :)
-
intosi
It never worked in Firefox for me.
-
intosi
FWIW
-
jonasw
is it sensible to have a version of the XEPs around which don’t work in a major browser?
-
intosi
Does in Safari and Chrome now.
-
Guus
Prettify.js throws an exception on line 31: Uncaught DOMException: Failed to set the 'innerHTML' property on 'Element': The provided markup is invalid XML, and therefore cannot be inserted into an XML document.
-
jonasw
(although I’m not sure if firefox is still a major browser…)
-
Guus
not sure if that's related
-
jonasw
Guus: firefox doesn’t even get to the point where JS or anything is loaded
-
jonasw
it fails XML parsing
-
Guus
jonasw: that might be presentation only. Did you try to look at the source of that page?
-
intosi
Due to missing entity references. Doesn't try to load the ent file.
-
jonasw
intosi: or the DTD for that matter
-
intosi
Or that.
-
Guus
Chromium warns about wrong mime-types too. the xsl is transferred as application/octet-stream
-
Guus
anyways: day job.
-
SamWhited
There's a feature request open for Firefox but right now it doesn't support loading external resources in an XML file.
-
Ge0rG
XML resource redirection is an old vulnerability vector
-
Guus
We are down to a one-page issue list for the website :)
-
Guus
(no, this is not intended as a challenge to get it longer again)
-
Guus
Feel free to resolve another few though
-
Zash
Guus: You don't want any further issues reported eh?
-
Guus
No. The website is perfect and flawless as is.
-
Guus
People that report issues are spreading fake news
-
Guus
sad.
-
Guus
Oh wow. Our lower house has just voted to legalize growing of cannabis.
-
Guus
Contrary to popular believe, that was illegal here. You could legally sell it, but not grow or buy it in bulk (nice, no?)
-
Guus
s/was/is/
-
Zash
As in, the non-industrial kind?
-
Guus
the industrial kind.
-
Guus
(coffee shops in Amsterdam got to get their produce from somewhere, right?)
-
Guus
There has been a weird non-enforcement policy. The violent crime related to cannabis cultivation is pretty high here.
-
Zash
(and by industrial I mean the kind with less psychoactive stuff, grown in some places at an industrial scale for the fibers)
-
Guus
ah, no, I was referring to commercially growns stuff (vs home-grown) for recreational purposes.
-
Lance
Given all of the discussion about bind2 and preserving client info for debugging, should we revisit and advance https://xmpp.org/extensions/xep-0232.html ?
-
Zash
> <identity category='client' name='Exodus' type='pc'/>
-
Zash
Lance: What they want is a persistent identifier for the device itself, not more verbose client description (As I understand it)
-
Zash
client or device
-
Zash
I'm wondering about adding an attribute on the opening stream header for easier tracing
-
Zash
Then it can be visible even before you auth/bind
-
Ge0rG
Zash: what kind of attribute?
-
Zash
Ge0rG: <stream:stream [...] xmlns:trace="urn:example:xmpp:trace" trace:id="yaxim-xxxxxx">
-
daniel
Zash: +1
-
daniel
And get rid of client assigned resources (even in parts) once and for all
-
MattJ
+1
-
MattJ
If we get rid of client-assigned resources, I'll do my best to prevent Ge0rG from ever seeing another UUID
-
Zash
I see no need in changing the current spec however
-
daniel
Zash: current as in regular bind?
-
MattJ
daniel, btw, just to confirm I saw your post about 313 and MUC <x> elements - going to have a look at that tomorrow. I remember we chatted about it a while back
-
daniel
Yes regular bind can stay
-
Ge0rG
daniel: aren't you the one with the broken client-generated resource implementation? :D
-
Zash
Ge0rG: I believe you can do the above Right Now if you wanted, pretty sure it shows up in debug logs. No idea about servers other than Prosody however.
-
Zash
Ge0rG: And it would work earlier.
-
Zash
As in, before bind/auth
-
Ge0rG
Zash: I want that trace:id to be in all log lines affecting that client please. even MUC debug messages on the MUC service.
-
Zash
daniel: I mean the rules (client can suggest something, server picks something). I'm sure bind2 can improve the protocol and make things better.
-
daniel
Ge0rG: i belive Conversations is one of the few clients that show the disco name when it comes to resource selection (for jingle or otr)
-
daniel
Zash: I don't see the benefit in a client suggesting something
-
Zash
daniel: Don't suggest something then.
-
Ge0rG
daniel: how is that an answer to my question? And how does it work with multiple instances of the same client?
-
Ge0rG
daniel: you can just admit it and pretend to be ashamed, and I'll leave for the night ☺️
-
Zash
Outside of client developers, how many have multiple instances of the same client?
-
Zash
Suppose you can show timestamp of last stanza received from each resource.
-
Zash
And other presence details
-
daniel
Well I guess the ultimate goal is to never show that resource selection at all
-
daniel
And instead use message initiation for jingle