Guusthe 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
Guusuhoh
vaulorhas joined
Guusah
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
GuusWhat we see now is not a relatively simple front-end issue?
Guuswhere's the code anyway? Docker pulls from a non-existing github repo?
Guushttps://bitbucket.org/horazont/xmppoke-frontend <-- that one?
Guuserr, bitbucket, not github
jonas’the xmppoke docker stuff itself on https://github.com/horazont/xmppoke-docker
Guusyes, 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
Guusbitbucket never agreed with me
Guushopefully 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
Guuskk
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
flowwhy not put a static page up on xmpp.net, saying that we are looking for help maintaining it
ZashSounds good
danielI'd volunteer to just rewrite it. But I think someone else is already doing so
Str4tocasterhas left
Str4tocasterhas joined
KevIt seems to be broken.
KevI 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
GuusKev, that's probably because the docker instance is being composed partly from a repository that's currently unvailable.
Guusbefore the restart, the frontend gave a different error.
GuusDaniel: what makes you think that someone else is already doing so/
Str4tocasterhas left
danielSomeone told me
daniel😏
Str4tocasterhas joined
danielHolger 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
KevGuus: It was just starting it, rather than recreating the image.
Holgerhttps://github.com/mathiasertl/xmpp-test
Holgerdaniel: But in the meantime, he said "right now, it's more or less on ice".
danielIf 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
danielAnd if I write the tooling to create the boolean I already have what it takes to generate a more verbose output
KevI suspect that the frontend can't cope with being restarted without being recreated, or something.
GuusDaniel / Holger: Nice!
GuusKev: Odd!
GuusUntil a viable replacement is available, it's probably best not to give up on our old instance though
KevIt seems like someone's moved things around on the filesystem and broken things, but I don't remember who set this up.
danielI 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
Guusmight 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) ...
danielAt some point it will make sense to integrate tls testers
GuusKev, might've been me?
KevLet's see if I can at least fix the backup script before I go nuclear on it.
lhas joined
KevAs things stand the backup script is broken, and the README is wrong.
Kev(And the container won't start)
KevWell, at least I've got the backup script working again.
Str4tocasterhas left
Str4tocasterhas joined
Str4tocasterhas left
Str4tocasterhas joined
Str4tocasterhas left
Str4tocasterhas joined
KevOh, 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
KevGot 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
GuusThanks 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
jonas’apparently I changed my username and bitbucket doesn’t do redirects :)
moparisthebestjonas’: what I'm hearing is you broke it and it's all your fault
jonas’no
moparisthebestLol <3
jonas’Guus, fix for the URL is pushed
Guusthanks 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")?
Guusjonas' : 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
Guushttps://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
GuusI'll have a stab at fixing the code to prevent the ugly php error
lorddavidiiihas joined
jjrhhas left
jjrhhas left
Guushargh, how do I fork in Bitbucket?
jjrhhas left
lumihas joined
Guusah, by creating a new repo
Guusoh, 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 ...
Guussure, but that'll make me install hg.
GuusI'm doing things in the text editor of bitbucket
sonnyhas left
sonnyhas joined
lovetoxhas joined
marchas joined
GuusJonas', 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
GuusI'm to stupid for docker
Guussomehow, my changes aren't picked up
jonas’did you re-build the image? with the correct tag?
marchas left
lnjhas joined
Guusprobably not
mimi89999has joined
j.rhas joined
lovetoxhas left
Guusah, got it
lovetoxhas joined
peterhas joined
404.cityhas joined
matlaghas left
Guusjonas' : my commits fix php-related errors
matlaghas joined
Guuslikely not the underlying issue of the tests not being executed
Guusbut 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
MattJjonas’ [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
MattJGroupchat messages are not redirected to other resources
MattJSorry, 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
Ge0rGAlso "silently ignore the stanza"
MattJSilently ignore that option
Ge0rGImplementation defined behavior.
jonas’IB!
jonas’at least not UB!
MattJSilently ignore buggy implementations
Ge0rGMattJ: they are not buggy if they are compliant
MattJYou think so? :)
Ge0rGMattJ: by your definition, mod_carbons and mod_mam are buggy as well
Ge0rGshortly considered writing "as hell"
MattJThey are
Zashhas left
Ge0rGAnd the XEPs are buggy too. And the RFC.
MattJTCP/IP is buggy, but (mostly) follows spec
MattJAbandon hope
ZashHumans are buggy
jonas’it’s all buggy
MattJNot all
MattJThe stuff we haven't written yet is perfect
Ge0rGIt's buggy all the way down. A fractal of bad protocol design
MattJGe0rG: great name for a client!
Ge0rGMattJ: "Buggy"? 😀
lskdjfhas joined
moparisthebestMaybe you should just start from scratch, call it matrix or something
ZashBug is a kind of dance?
moparisthebestI'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?
Zashhippy
labdsfhas left
j.rhas joined
Ge0rGBuggy, the zimpy messenger.
Ge0rGMy phone auto corrected zimpy to "gimpy"
Alexhas left
labdsfhas joined
lovetoxwhen 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
Zash1806, it was a cold winter that year
jonas’ha, the compliance suites
lovetoxlol :D
lovetoxok and now a real xep
lovetoxplease
Zashmust 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!!
Ge0rGGetting 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
SamWhitedSpeaking 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
Ge0rGIf 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
Ge0rGI think I was missing one XEP when explaining what's relevant for mobile to haeckse last weekend
Ge0rGOh yes, it's missing 0184
Ge0rGI don't even understand how we could miss that.