mostlymatter

mirrors/mostlymatter

Форкнуть 0

Граф коммитов

Автор	SHA1	Сообщение	Дата
Jesse Hallam	47bfa2b66b	Pass a logger instead of embedding on job (#24650 ) * pass a logger instead of embedding on job * leverage mlog.Millis * use worker logger with HandleJobPanic * rely on existing LogClone instead * guard Job.LogClone against nil Job * s/workername/worker_name * Revert "rely on existing LogClone instead" This reverts commit 17303cbac90d4b01815abca1309b78b97de368fb. * Revert "guard Job.LogClone against nil Job" This reverts commit f1ae22dee58d76f084582857830ffe8d4c546d7e.	2023-10-09 11:04:55 -03:00
Agniva De Sarker	b9c50641db	MM-54580: Fix panic in jobs/base_workers.go (#24631 ) If GetJob fails, we would return a nil job and then try to log with that. To prevent this, we keep a reference to the old job and log with that in the error case. https://mattermost.atlassian.net/browse/MM-54580 ```release-note Fix a panic where a simple worker would crash if if failed to get a job. ```	2023-10-03 07:55:38 +05:30

Автор

SHA1

Сообщение

Дата

Jesse Hallam

47bfa2b66b

Pass a logger instead of embedding on job (#24650 )

* pass a logger instead of embedding on job

* leverage mlog.Millis

* use worker logger with HandleJobPanic

* rely on existing LogClone instead

* guard Job.LogClone against nil Job

* s/workername/worker_name

* Revert "rely on existing LogClone instead"

This reverts commit 17303cbac90d4b01815abca1309b78b97de368fb.

* Revert "guard Job.LogClone against nil Job"

This reverts commit f1ae22dee58d76f084582857830ffe8d4c546d7e.

2023-10-09 11:04:55 -03:00

Agniva De Sarker

b9c50641db

MM-54580: Fix panic in jobs/base_workers.go (#24631 )

If GetJob fails, we would return a nil job and then try to log
with that. To prevent this, we keep a reference to the old job
and log with that in the error case.

https://mattermost.atlassian.net/browse/MM-54580

```release-note
Fix a panic where a simple worker would crash if if failed to get a job.
```

2023-10-03 07:55:38 +05:30

2 Коммитов