Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[Bug]: updateProgress in Sandboxed processors not update job.progress proprialy #2213

Closed
1 task done
x-etienne opened this issue Oct 4, 2023 · 1 comment · Fixed by #2214
Closed
1 task done
Labels
bug Something isn't working

Comments

@x-etienne
Copy link

Version

v4.12.0

Platform

NodeJS

What happened?

Hi buddies, thanks in advance

When I update job progress in a Sandboxed processors

export default async function (job: Job) {
  await job.updateProgress(42);
 // here job.progress != 42, it remains 0
}

the 'progress' property of job instance not update

while if I do the same things in normal worker, the 'progress' property is updated as expected

const worker = new Worker(queueName, async (job: Job) => {
  await job.updateProgress(42);
// here job.progress == 42, it change as expected
});

How to reproduce.

No response

Relevant log output

No response

Code of Conduct

  • I agree to follow this project's Code of Conduct
@x-etienne x-etienne added the bug Something isn't working label Oct 4, 2023
@manast
Copy link
Contributor

manast commented Oct 4, 2023

Thanks for the report. I guess we overlooked updating the progress property for the sandbox case.

github-actions bot pushed a commit that referenced this issue Oct 5, 2023
## [4.12.2](v4.12.1...v4.12.2) (2023-10-05)

### Bug Fixes

* **sandbox:** update progress value on job instance ([#2214](#2214)) fixes [#2213](#2213) ([3d0f36a](3d0f36a))
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants