@eyedeekay
&zzz
+R4SAS
+RN
+RN_
+StormyCloud
+acetone
+dr|z3d
+eche|off
+hk
+postman
+qend-irc2p
+snex
+wodencafe
Arch
BravoOreo
BubbRubb
Chrono
Daddy
Danny
FreeB
FreefallHeavens
HowardPlayzOfAdmin
Irc2PGuest22674
Irc2PGuest28862
Irc2PGuest52684
Irc2PGuest52770
Irc2PGuest63688
Onn4l7h
Onn4|7h
Over
Rogueone
Sisyphus
Sleepy
St1nt
T3s|4_
Teeed
aargh
aisle
ardu
b3t4f4c3__
cumlord
dr4wd3
duanin2
eyedeekay_bnc
kafficecco
nilbog-
nnm--
not_bob_afk
ohThuku1
orignal_
poriori_
pory_
profetikla
r00tobo
rapidash
shiver_
solidx66
thetia
u5657
w8rabbit
weko_
wew-
x74a6
dr|z3d
eyedeekay: any reason PRs on gitea are behind a login?
eyedeekay
Because calling the PR page triggers a git diff which triggers a git cat-file which is the thing that's always hanging and causing the OOMs
eyedeekay
`REQUIRE_SIGNIN_VIEW = expensive`
dr|z3d
what, to view PRs? surely not? :)
eyedeekay
Significantly reduced the crashes to do so
eyedeekay
And it corresponds to the constant crawling. It doesn't happen for every single diff, only like, specific ones, timeouts don't help, I can kill them with a cronjob but that's like... nasty and affects everybody not just the hung git processes
dr|z3d
here's a different idea. all the non-official repos, make them either private or login required. that way you're minimizing the exposed surface without sacrificing access to the main project stuff.
eyedeekay
Well there's a problem with that. Remember how I said it's *specific* diffs?
dr|z3d
odd. well you can limit the size of diffs that are presented without required to expand them, dunno if that's any help.
dr|z3d
*being required
eyedeekay
I already tried that, does not help
dr|z3d
hmm. something jank in the land of gitea/idk :)
dr|z3d
rate limiting in nginx, dunno if that might mitigate things.
dr|z3d
shouldn't impact normal users, just spiders and whatever else automated.
eyedeekay
see dms
anonymousmaybe
@eyedeekay
anonymousmaybe
got it reproduced on your side?