the Observatory appears to be misbehaving. PHP errors on the front page. Jonas', would you mind kicking off that frontend docker hub project, to see if it goes away?
Str4tocasterhas joined
guusdkhas left
jonas’
Guus, I can’t
jonas’
it needs manual iteam intervention, it’s not pulled automatically from docker hub
Guus
uhoh
vaulorhas joined
Guus
ah
jonas’
I’ve tried to get iteam attention for weeks now about this :(
jonas’
but in my mind, I already figured that we need to re-implement this on top of something less awful to maintain
Zashhas joined
guusdkhas joined
Guus
What we see now is not a relatively simple front-end issue?
Guus
where's the code anyway? Docker pulls from a non-existing github repo?
Guus
https://bitbucket.org/horazont/xmppoke-frontend <-- that one?
Guus
err, bitbucket, not github
jonas’
the xmppoke docker stuff itself on https://github.com/horazont/xmppoke-docker
jonas’
the frontend lives here: https://github.com/horazont/xmppoke-frontend-docker
Guus
yes, but line 14? https://github.com/horazont/xmppoke-frontend-docker/blob/master/Dockerfile#L14
edhelas
(I just posted the release note of Movim 0.14 on HN, for those that are interested https://news.ycombinator.com/item?id=18485822)
jonas’
Guus, ha, nice
jonas’
but why
jonas’
I don’t have mit bitbucket credentials at hand to check
Guus
bitbucket never agreed with me
Guus
hopefully you can find out what happened later
lnjhas joined
Str4tocasterhas left
Str4tocasterhas joined
jonas’
Guus, I might forget, feel free to remind me later today (15:00Z+) or on wednesday
Guus
kk
mightyBroccolihas left
matlaghas left
matlaghas joined
j.rhas joined
Str4tocasterhas left
Str4tocasterhas joined
blablahas joined
moparisthebesthas joined
moparisthebesthas joined
mightyBroccolihas joined
lhas joined
404.cityhas left
Str4tocasterhas left
Str4tocasterhas joined
flow
why not put a static page up on xmpp.net, saying that we are looking for help maintaining it
Zash
Sounds good
daniel
I'd volunteer to just rewrite it. But I think someone else is already doing so
Str4tocasterhas left
Str4tocasterhas joined
Kev
It seems to be broken.
Kev
I restarted both poke and the frontend, and the frontend refuses to come up.
vanitasvitaehas left
Andrew Nenakhovhas left
vanitasvitaehas joined
Andrew Nenakhovhas joined
Alexhas left
Guus
Kev, that's probably because the docker instance is being composed partly from a repository that's currently unvailable.
Guus
before the restart, the frontend gave a different error.
Guus
Daniel: what makes you think that someone else is already doing so/
Str4tocasterhas left
daniel
Someone told me
daniel
😏
Str4tocasterhas joined
daniel
Holger mentioned that Matt the jabber.at admin is working on it and also sent me a link that I haven't had time to check out yet
Kev
Guus: It was just starting it, rather than recreating the image.
Holger
https://github.com/mathiasertl/xmpp-test
Holger
daniel: But in the meantime, he said "right now, it's more or less on ice".
daniel
If that doesn't lead anywhere I might start my own. I want a test that then will result in a simple boolean to integrate it into the compliance tester
daniel
And if I write the tooling to create the boolean I already have what it takes to generate a more verbose output
Kev
I suspect that the frontend can't cope with being restarted without being recreated, or something.
Guus
Daniel / Holger: Nice!
Guus
Kev: Odd!
Guus
Until a viable replacement is available, it's probably best not to give up on our old instance though
Kev
It seems like someone's moved things around on the filesystem and broken things, but I don't remember who set this up.
daniel
I think I might have to grow the compliance tester into a more general thing anyway. During the sprint I wrote a thing that tests for cors headers on the HTTP upload
Guus
might take a while before the replacement is ready. Looking at the effort that has gone into the old, but also the new tings (100+ commits) ...
daniel
At some point it will make sense to integrate tls testers
Guus
Kev, might've been me?
Kev
Let's see if I can at least fix the backup script before I go nuclear on it.
lhas joined
Kev
As things stand the backup script is broken, and the README is wrong.
Kev
(And the container won't start)
Kev
Well, at least I've got the backup script working again.
Str4tocasterhas left
Str4tocasterhas joined
Str4tocasterhas left
Str4tocasterhas joined
Str4tocasterhas left
Str4tocasterhas joined
Kev
Oh, the creation scripts only work if the instance is running. If it's not running, they'll refuse to install a new one.
Str4tocasterhas left
Str4tocasterhas joined
alacerhas joined
Kev
Got it back to
Warning: count(): Parameter must be an array or an object that implements Countable in /var/www/html/index.php on line 97
state, at least.
Zashhas left
Holgerhas left
Guus
Thanks Kev
andyhas left
andyhas joined
andyhas left
jonas’
daniel, who is rewriting it?
jonas’
the TLS tests can be easily based on testssl.sh, it has a JSON formatted output
jonas’
that was the plan when I get around to do it, but I didn’t start
andyhas joined
Str4tocasterhas left
matlaghas left
matlaghas joined
Str4tocasterhas joined
guusdkhas left
guusdkhas joined
Str4tocasterhas left
Str4tocasterhas joined
Str4tocasterhas left
Str4tocasterhas joined
Str4tocasterhas left
Str4tocasterhas joined
Str4tocasterhas left
pep.has left
Alexhas left
lorddavidiiihas left
Str4tocasterhas joined
Alexhas joined
alacerhas left
lorddavidiiihas joined
Guus
jonas' : https://github.com/mathiasertl/xmpp-test
Str4tocasterhas left
matlaghas left
matlaghas joined
Andrew Nenakhovhas left
Andrew Nenakhovhas joined
vaulorhas joined
andyhas left
andyhas joined
lnjhas left
Zashhas left
moparisthebesthas joined
vanitasvitaehas left
vanitasvitaehas joined
andrey.ghas left
lovetoxhas left
matlaghas left
matlaghas joined
genofirehas left
Holgerhas left
andyhas left
moparisthebesthas joined
moparisthebesthas joined
genofirehas left
genofirehas left
!xsf_martinhas joined
vanitasvitaehas left
vanitasvitaehas joined
genofirehas left
genofirehas left
!xsf_martinhas left
!xsf_martinhas joined
ThibGhas joined
ThibGhas joined
Guus
Jonas' this is your 15:01Z callback ping 🙂
jonas’
thanks!
jonas’
apparently I changed my username and bitbucket doesn’t do redirects :)
moparisthebest
jonas’: what I'm hearing is you broke it and it's all your fault
jonas’
no
moparisthebest
Lol <3
jonas’
Guus, fix for the URL is pushed
Guus
thanks jonas' !
!xsf_martinhas left
!xsf_martinhas joined
!xsf_martinhas left
!xsf_martinhas joined
tahas left
lhas left
Zashhas left
Nekithas left
Nekithas joined
jjrhhas left
jonas’
isn’t there a general race condition with multiple clients of the same account in a single MUC where when a MUC forwards a groupchat message to one of the clients and the client goes offline, it might get sent by the XMPP server to the other client ("most available resource")?
Guus
jonas' : both index.php as well as list.php don't handle a scenario well where the find_score query doesn't return rows. I'm assuming that's easily fixed. I'm unsure why no scores are returned though. To much work to be processed?
jonas’
I don’t know
Guus
https://bitbucket.org/jssfr/xmppoke-frontend/src/84b48437dca24deaf7557c982c069015cf09a396/index.php <-- line 89 does appear to validate if scores is non-empty, but line 97 does not
lorddavidiiihas left
jonas’
I didn’t write the PHP code, and I don’t want to maintain it.
jjrhhas left
Guus
I'll have a stab at fixing the code to prevent the ugly php error
lorddavidiiihas joined
jjrhhas left
jjrhhas left
Guus
hargh, how do I fork in Bitbucket?
jjrhhas left
lumihas joined
Guus
ah, by creating a new repo
Guus
oh, good, it uses mercurial.
Guusgives up.
Zashhas joined
jonas’
not that hard when you just want to patch a thing
jonas’
hg clone, do your thing, hg commit file1 file2 ...
Guus
sure, but that'll make me install hg.
Guus
I'm doing things in the text editor of bitbucket
sonnyhas left
sonnyhas joined
lovetoxhas joined
marchas joined
Guus
Jonas', I think I just created two PRs against your repo, that _should_ fix the issue
jonas’
Guus, can you try to try it for yourself?
jonas’
the frontend thing contains a docker-compose file which you should be able to use
sonnyhas joined
lhas joined
lskdjfhas joined
waqashas joined
Syndacehas left
SamWhitedhas left
ralphmhas left
mightyBroccolihas left
mightyBroccolihas joined
guusdkhas left
moparisthebesthas joined
mimi89999has joined
lskdjfhas joined
mimi89999has left
ralphmhas left
ralphmhas joined
mightyBroccolihas left
mightyBroccolihas joined
Syndacehas joined
j.rhas joined
sonnyhas joined
jonas’
Guus, hm, I’m really not fond of maintaining this
jonas’
I don’t feel comfortable in reviewing PHP changes, and yet I’m now somewhat of a gatekeeper
Guus
I'm to stupid for docker
Guus
somehow, my changes aren't picked up
jonas’
did you re-build the image? with the correct tag?
marchas left
lnjhas joined
Guus
probably not
mimi89999has joined
j.rhas joined
lovetoxhas left
Guus
ah, got it
lovetoxhas joined
peterhas joined
404.cityhas joined
matlaghas left
Guus
jonas' : my commits fix php-related errors
matlaghas joined
Guus
likely not the underlying issue of the tests not being executed
Guus
but at least, it's failing more gracefully now.
Zashhas left
jonas’
mmm
genofirehas left
goffihas joined
j.rhas joined
Nekithas left
Nekithas joined
Steve Killehas left
Steve Killehas left
Zashhas joined
jjrhhas left
Steve Killehas joined
mimi89999has left
Steve Killehas left
guusdkhas left
mimi89999has left
lumihas left
lumihas joined
labdsfhas left
Nekithas left
Nekithas joined
matlaghas left
matlaghas joined
guusdkhas left
guusdkhas left
guusdkhas joined
matlaghas left
matlaghas joined
Holgerhas left
ralphmhas left
vanitasvitaehas left
vanitasvitaehas left
vanitasvitaehas joined
guusdkhas left
guusdkhas joined
404.cityhas left
andrey.ghas joined
j.rhas joined
lhas left
j.rhas joined
lskdjfhas joined
peterhas left
labdsfhas joined
peterhas joined
pep.has left
404.cityhas joined
404.cityhas left
labdsfhas left
labdsfhas joined
vaulorhas joined
goffihas joined
goffihas left
goffihas joined
MattJ
jonas’ [15:20]:
> isn’t there a general race condition with multiple clients of the same account in a single MUC where when a MUC forwards a groupchat message to one of the clients and the client goes offline, it might get sent by the XMPP server to the other client ("most available resource")?
jonas’
no, there isn’t, I figured it out by now
jonas’
due to:
8.5.3.2.1. Message
For a message stanza of type "normal", "groupchat", or "headline",
the server MUST either (a) silently ignore the stanza or (b) return
an error stanza to the sender, which SHOULD be <service-
unavailable/>.
jonas’
sorry for the noise
MattJ
Groupchat messages are not redirected to other resources
MattJ
Sorry, great :)
peterhas left
!xsf_martinhas joined
Zashhas left
Zashhas joined
guusdkhas left
guusdkhas joined
j.rhas left
guusdkhas left
guusdkhas joined
pep.has joined
guusdkhas left
guusdkhas joined
Ge0rG
Also "silently ignore the stanza"
MattJ
Silently ignore that option
Ge0rG
Implementation defined behavior.
jonas’
IB!
jonas’
at least not UB!
MattJ
Silently ignore buggy implementations
Ge0rG
MattJ: they are not buggy if they are compliant
MattJ
You think so? :)
Ge0rG
MattJ: by your definition, mod_carbons and mod_mam are buggy as well
Ge0rGshortly considered writing "as hell"
MattJ
They are
Zashhas left
Ge0rG
And the XEPs are buggy too. And the RFC.
MattJ
TCP/IP is buggy, but (mostly) follows spec
MattJ
Abandon hope
Zash
Humans are buggy
jonas’
it’s all buggy
MattJ
Not all
MattJ
The stuff we haven't written yet is perfect
Ge0rG
It's buggy all the way down. A fractal of bad protocol design
MattJ
Ge0rG: great name for a client!
Ge0rG
MattJ: "Buggy"? 😀
lskdjfhas joined
moparisthebest
Maybe you should just start from scratch, call it matrix or something
Zash
Bug is a kind of dance?
moparisthebest
I'm sure you won't hit the same problems :)
Ge0rG
> have you tried the Buggy messenger?
All of them!
Zash
-y as suffix is hip again?
Zash
hippy
labdsfhas left
j.rhas joined
Ge0rG
Buggy, the zimpy messenger.
Ge0rG
My phone auto corrected zimpy to "gimpy"
Alexhas left
labdsfhas joined
lovetox
when was the last XEP that made it to DRAFT, just out of interest?
jonas’
commit c58d3bbf5ed18e95b374e3bc1db9a85de40f2661
Author: Jonas Wielicki <j.wielicki@sotecware.net>
Date: Thu Jan 25 08:54:39 2018 +0100
XEP-0387: Move to draft
jonas’
that’s a good candidate
Zash
1806, it was a cold winter that year
jonas’
ha, the compliance suites
lovetox
lol :D
lovetox
ok and now a real xep
lovetox
please
Zash
must have been in the dark ages
jonas’
I’d h ave to do more magic than that, but it might be May 2017
jonas’
or March?
Marandahas joined
jonas’
march
jonas’
back when Sam was still editoring much -- it’s not my fault, I swear!!
Ge0rG
Getting compliance Suite into draft was hard enough!
Marandahas left
jjrhhas left
jjrhhas left
jonas’
yes, XEP-0233 was the last one moving to draft before that, that was in Mar 2017
Alexhas joined
mimi89999has left
Zashhas left
lskdjfhas joined
tuxhas left
lorddavidiiihas left
lorddavidiiihas joined
marchas joined
Zashhas left
Zashhas left
peterhas joined
lorddavidiiihas left
lorddavidiiihas joined
peterhas left
SamWhited
Speaking of compliance suites, someone should do some for this upcoming year (even if they're basically the same) so that they don't appear out of date
mimi89999has joined
mimi89999has joined
jonas’
yeah, I’ve got that on my todo
Yagizahas left
Ge0rG
If it's the same, can we just update the year?
jonas’
next council will decide
jonas’
(but I’m in favour)
mimi89999has joined
SamWhitedhas left
mimi89999has left
Ge0rG
I think I was missing one XEP when explaining what's relevant for mobile to haeckse last weekend