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]: Invalid URL /v1/acquire thrown from PuppeteerWorkers.launch #50

Closed
hmnd opened this issue Jun 7, 2024 · 6 comments · Fixed by #51
Closed

[Bug]: Invalid URL /v1/acquire thrown from PuppeteerWorkers.launch #50

hmnd opened this issue Jun 7, 2024 · 6 comments · Fixed by #51
Labels
bug Something isn't working

Comments

@hmnd
Copy link

hmnd commented Jun 7, 2024

Bug description

Steps to reproduce the problem:

  1. Create worker following the guide here.
  2. Try to make a request to the worker.

Puppeteer version

0.0.10

Node.js version

20.13.1

npm version

10.5.2

What operating system are you seeing the problem on?

Linux

Relevant log output

✘ [ERROR] Uncaught (in response) TypeError: Invalid URL: /v1/acquire

      at PuppeteerWorkers.launch
@hmnd hmnd added the bug Something isn't working label Jun 7, 2024
@meddulla
Copy link
Collaborator

meddulla commented Jun 7, 2024

Hi @hmnd Can't reproduce and the API hasn't changed. Can you provide your wrangler.toml + js/ts file in a gist?

@hmnd
Copy link
Author

hmnd commented Jun 7, 2024

@meddulla thanks for the quick response. Here you go :) https://gist.github.com/hmnd/5f6e7810c6e1feb8b5b66dab57769116

@sunneydev
Copy link

@meddulla seems like base url for the fetch is not defined. i also get type errors with current latest versions on everything.

its for some reason colliding with undici-types, even though i don't have @types/node.
image
image
image

@meddulla
Copy link
Collaborator

meddulla commented Jun 9, 2024

@hmnd @sunneydev Thanks. Can reproduce now. The issue seems to be related to the compatibility_date. Can you both try and set the compatibility_date to 2024-06-02 or less?
@sunneydev If the types issues persist, can you share a full package.json, wrangler.toml and index.js/ts in a gist that reproduces the issue?

@abdullah
Copy link

abdullah commented Jun 9, 2024

Updating compatibility_date works for me 🚀

@meddulla
Copy link
Collaborator

meddulla commented Jun 10, 2024

Latest version0.0.11 fixes this. Thanks for your input.

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.

4 participants