@eyedeekay
&zzz
+FreefallHeavens
+R4SAS
+RN
+RN_
+T3s|4
+cumlord
+dr|z3d
+eche|off
+mareki2p
+orignal
+postman
+qend-irc2p
+snex
+wodencafe
Arch
BubbRubb
Chrono
Danny
DeltaOreo
HowardPlayzOfAdmin
Irc2PGuest14159
Irc2PGuest28800
Irc2PGuest93285
Onn4l7h
Over
Sisyphus
Sleepy
SlippyJoe_
Stormycloud_
T3s|4_
Teeed
ac9f
acetone__
ardu
b3t4f4c3__
bak83_
coolbuddy799
dr4wd3
duanin2
duck
eyedeekay_bnc
kaffi
leopold
makoto
marek22k
nilbog
not_bob_afk
poriori
pory
profetikla
r00tobo_BNC
rapidash
shiver_
thetia
u5657
uop23ip
vivid_reader56
w8rabbit
x74a6
xHarr
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