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: Wrangler tail fails with "Cannot use 'in' operator to search for 'cron' in null #918

Closed
vlovich opened this issue May 7, 2022 · 2 comments · Fixed by #919 or #909
Closed

Comments

@vlovich
Copy link

vlovich commented May 7, 2022

What version of Wrangler are you using?

0.0.30

What operating system are you using?

Linux

Describe the Bug

Tail died with

node_modules/wrangler/wrangler-dist/cli.js:200

then showing a large amount of minified code (> 1kb) starting with:

Check the render method of %s.`,Ye(su.type)||"Unknown"));var _e=kt(Y,

then:

TypeError: Cannot use 'in' operator to search for 'cron' in null
    at xqe (/home/vlovich/cloudflare/blob-gateway-worker/node_modules/wrangler/wrangler-dist/cli.js:558:1156)
    at _i.GG (/home/vlovich/cloudflare/blob-gateway-worker/node_modules/wrangler/wrangler-dist/cli.js:558:580)
    at _i.emit (node:events:527:28)
    at B9.d9e (/home/vlovich/cloudflare/blob-gateway-worker/node_modules/wrangler/wrangler-dist/cli.js:324:59955)
    at B9.emit (node:events:527:28)
    at B9.dataMessage (/home/vlovich/cloudflare/blob-gateway-worker/node_modules/wrangler/wrangler-dist/cli.js:324:39151)
    at B9.getData (/home/vlovich/cloudflare/blob-gateway-worker/node_modules/wrangler/wrangler-dist/cli.js:324:38466)
    at B9.startLoop (/home/vlovich/cloudflare/blob-gateway-worker/node_modules/wrangler/wrangler-dist/cli.js:324:35220)
    at B9._write (/home/vlovich/cloudflare/blob-gateway-worker/node_modules/wrangler/wrangler-dist/cli.js:324:34573)
    at writeOrBuffer (node:internal/streams/writable:390:12)

Node.js v17.6.0
@vlovich vlovich added the bug label May 7, 2022
@petebacondarwin
Copy link
Contributor

Looks like the event in the eventMessage was null. Is this expected? If so, we need to make the Wrangler code more tolerant here.

threepointone added a commit that referenced this issue May 7, 2022
Sometime the "event" on a tail can be null. This patch makes sure we don't crash when that happens. Fixes #918
petebacondarwin pushed a commit that referenced this issue May 7, 2022
Sometime the "event" on a tail can be null. This patch makes sure we don't crash when that happens. Fixes #918
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
2 participants