Skip to content
This repository has been archived by the owner on Aug 3, 2023. It is now read-only.

[dev] Worker updates not seen in terminal until restarting dev #1082

Closed
jaymakes11 opened this issue Feb 18, 2020 · 5 comments 路 Fixed by #1117
Closed

[dev] Worker updates not seen in terminal until restarting dev #1082

jaymakes11 opened this issue Feb 18, 2020 · 5 comments 路 Fixed by #1117

Comments

@jaymakes11
Copy link

Mentioned initially in #1078 (comment).

@EverlastingBugstopper

馃悰 Bug Report

Environment

  • operating system: MacOS (Catalina, 10.15.1)
  • output of rustc -V: n/a
  • output of node -v: v10.17.0
  • output of wrangler -V: 1.8.0-rc.1

Steps to reproduce

With dev running, add a console log statement inside the worker. Save. The console log does not show up. Restart dev, the console log is there.

What did you expect to see?

The new console log should show up in the terminal without dev needing to be restarted.

What did you see instead?

The new console log does not show up in the terminal.

@EverlastingBugstopper
Copy link
Contributor

@yet-another-dev - the reload itself can take a bit since it needs to re-build and upload the worker to the cloudflare api (wrangler dev is not offline). Does it work if you wait a bit until you see a Built successfully message?

@jaymakes11
Copy link
Author

Does it work if you wait a bit until you see a Built successfully message?

No, I'm afraid not.

@cpauwels
Copy link

I'm having the same issue here, also with the console.logs.

@EverlastingBugstopper
Copy link
Contributor

Ok - I'm able to reproduce with a Workers Site project, but not with a regular worker. Going to investigate this.

@EverlastingBugstopper
Copy link
Contributor

It seems like this has to do with how make_service_fn works - I need to force it to update quicker than it is. Waiting on a response from some Rust experts 馃槃

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants