1
0
Fork 0
forgejo/services/actions
Jason Song 47b912cd52
Avoid panic caused by broken payload when creating commit status (#23216)
When creating commit status for Actons jobs, a payload with nil
`HeadCommit` will cause panic.

Reported at:
https://gitea.com/gitea/act_runner/issues/28#issuecomment-732166

Although the `HeadCommit` probably can not be nil after #23215,
`CreateCommitStatus` should protect itself, to avoid being broken in the
future.

In addition, it's enough to print error log instead of returning err
when `CreateCommitStatus` failed.

---------

Co-authored-by: delvh <dev.lh@web.de>
2023-03-04 02:12:37 -05:00
..
clear_tasks.go Avoid panic caused by broken payload when creating commit status (#23216) 2023-03-04 02:12:37 -05:00
commit_status.go Avoid panic caused by broken payload when creating commit status (#23216) 2023-03-04 02:12:37 -05:00
init.go Implement actions (#21937) 2023-01-31 09:45:19 +08:00
job_emitter.go Implement actions (#21937) 2023-01-31 09:45:19 +08:00
job_emitter_test.go Implement actions (#21937) 2023-01-31 09:45:19 +08:00
notifier.go Add context cache as a request level cache (#22294) 2023-02-15 21:37:34 +08:00
notifier_helper.go Avoid panic caused by broken payload when creating commit status (#23216) 2023-03-04 02:12:37 -05:00