You cannot select more than 25 topics Topics must start with a letter or number, can include dashes ('-') and can be up to 35 characters long.
gitea/services
Earl Warren fab73e4a3a
Actually recover from a panic in cron task (#28409)
- Currently there's code to recover gracefully from panics that happen
within the execution of cron tasks. However this recover code wasn't
being run, because `RunWithShutdownContext` also contains code to
recover from any panic and then gracefully shutdown Forgejo. Because
`RunWithShutdownContext` registers that code as last, that would get run
first which in this case is not behavior that we want.
- Move the recover code to inside the function, so that is run first
before `RunWithShutdownContext`'s recover code (which is now a noop).

Fixes: https://codeberg.org/forgejo/forgejo/issues/1910

Co-authored-by: Gusted <postmaster@gusted.xyz>
1 year ago
..
actions Use db.Find instead of writing methods for every object (#28084) 1 year ago
agit Penultimate round of `db.DefaultContext` refactor (#27414) 1 year ago
asymkey Use db.Find instead of writing methods for every object (#28084) 1 year ago
attachment Even more `db.DefaultContext` refactor (#27352) 2 years ago
auth Use db.Find instead of writing methods for every object (#28084) 1 year ago
automerge Improve queue and logger context (#24924) 2 years ago
context Another round of `db.DefaultContext` refactor (#27103) 2 years ago
convert Fix package webhook (#27839) 1 year ago
cron Actually recover from a panic in cron task (#28409) 1 year ago
externalaccount Final round of `db.DefaultContext` refactor (#27587) 1 year ago
feed More `db.DefaultContext` refactor (#27265) 2 years ago
forms Fix required error for token name (#28267) 1 year ago
gitdiff Even more `db.DefaultContext` refactor (#27352) 2 years ago
indexer Fix missing issue search index update when changing status (#28325) 1 year ago
issue Penultimate round of `db.DefaultContext` refactor (#27414) 1 year ago
lfs Remove GetByBean method because sometimes it's danger when query condition parameter is zero and also introduce new generic methods (#28220) 1 year ago
mailer Fix missing mail reply address (#27997) 1 year ago
markup make writing main test easier (#27270) 2 years ago
migrations Fix migration panic due to an empty review comment diff (#28334) 1 year ago
mirror Even more `db.DefaultContext` refactor (#27352) 2 years ago
notify Update status and code index after changing the default branch (#27018) 2 years ago
org Delete repos of org when purge delete user (#27273) 1 year ago
packages Fix RPM/Debian signature key creation (#28352) 1 year ago
pull Remove GetByBean method because sometimes it's danger when query condition parameter is zero and also introduce new generic methods (#28220) 1 year ago
release Fix comment permissions (#28213) 1 year ago
repository Also sync DB branches on push if necessary (#28361) 1 year ago
secrets Use db.Find instead of writing methods for every object (#28084) 1 year ago
task Fix incorrect ctx usage in defer function (#27740) 1 year ago
uinotification Penultimate round of `db.DefaultContext` refactor (#27414) 1 year ago
user Use db.Find instead of writing methods for every object (#28084) 1 year ago
webhook Use db.Find instead of writing methods for every object (#28084) 1 year ago
wiki Even more `db.DefaultContext` refactor (#27352) 2 years ago