@eyedeekay
&eche|on
&zzz
+R4SAS
+RN
+RN_
+StormyCloud
+T3s|4
+acetone
+cumlord
+dr|z3d
+eche|off
+mareki2p
+orignal
+postman
+qend-irc2p
+snex
+wodencafe
Arch
Birdy
BubbRubb
Chrono
Daddy
DeltaOreo
Irc2PGuest49308
Irc2PGuest51170
Irc2PGuest51586
Irc2PGuest64807
Irc2PGuest83827
Irc2PGuest85653
Irc2PGuest96178
Onn4l7h
Over
Sisyphus
Sleepy
SlippyJoe_
T3s|4_
Teeed
aargh2
ac9f
b3t4f4c3___
dr4wd3
duanin2
eyedeekay_bnc
leopold_
makoto
nilbog
nnm--
not_bob_afk
null911
onon_
poriori
profetikla
r00tobo_BNC
rapidash
shiver_
solidx66
u5657
uop23ip
w8rabbit
x74a6
eyedeekay
Thanks zzz. I've just checked in my implementations of a dg2 and dg3 library, primary sessions included. Everything seems to work fine, going to do some benchmarking, look a little closer at the code, but end-to-end tests work across the board
zzz
thanks eyedeekay good job
zzz
surprised you didn't find any bugs on my side
zzz
so what's the state of PR machinery and how do we merge this one (523) ?
eyedeekay
Actually, looks like I might have spoke too soon, might have round-trip delivery issues on dg3, tests keep timing out
eyedeekay
specifically round trip delivery issues on dg3 in primary sessions, seems very specific to that
zzz
hmm
eyedeekay
Will chase it to see where it is, might just be timeouts but something still may be happening
zzz
not sure why dg2 would work and dg3 wouldn't, it's a one-liner difference
zzz
more or less
eyedeekay
probably a timeout thing then, or maybe a hanging socket thing on my test
eyedeekay
Re: merging I am not seeing any conflict issues so once I stare down some logs and figure out if it is my test or the dg3 code I will merge it into master here and push it
eyedeekay
I had a *lot* of inconsistent tests in the old library version, this might be a relic of that
zzz
ok then either bump to -1 or tell me to do it
eyedeekay
Will do
zzz
on the jetty12 PR 512 did you ever get to see my last checkin that attempted to resolve the merge conflict and did it do it?
eyedeekay
Yeah it did
eyedeekay
Doesn't show up in the webUI yet though, still have not figured that out, so I will have to post patches to the PR thread and merge manually
eyedeekay
After we get these PR's done I will pursue fixing webhooks more aggressively
zzz
ok. I can also just merge manually here if you want. I'll have probably 2 or 3 more revs before its ready
eyedeekay
Go for it
zzz
will touch base before I do anything
eyedeekay
Thanks
dr|z3d
welcome back from the rabbit hole, zzz! :)\
zzz
no I haven't spent a month on jetty12.
zzz
it was mostly done months ago, just have to do the migration part.
zzz
home project distractions and procrastinations plus been waiting for gitea fixes
dr|z3d
:)
dr|z3d
how's gitea looking from where you are?
zzz
not looking, just following guidance
dr|z3d
ok
eyedeekay
Last error confirmed to be from my library side, I was using the wrong base64 decoder