SamuelScheit / puppeteer-stream

A Library for puppeteer to retrieve audio and/or video streams
MIT License
358 stars 114 forks source link

ability to use puppeteer-extra plugins like stealth #48

Closed ganganimaulik closed 2 years ago

ganganimaulik commented 2 years ago

47

I had to reload extension on browser load for some reason when we provide puppeteer-extra package. not sure why extension is not getting detected without reloading here: https://github.com/ganganimaulik/puppeteer-stream/blob/main/src/PuppeteerStream.ts#L79

anyway it's working fine as per example added: https://github.com/ganganimaulik/puppeteer-stream/blob/feat/extra-plugin/examples/puppeteer-extra.js

SamuelScheit commented 2 years ago

Thank you very much for your contribution!

ganganimaulik commented 2 years ago

Thank you very much for your contribution!

you're welcome :)

thomask commented 2 years ago

This change introduces a BC break

image

Is this intentional?

Also arg1 naming is not clear

ganganimaulik commented 2 years ago

This change introduces a BC break

image

Is this intentional?

Also arg1 naming is not clear

made pr to make second arg optional for ts #51 -> cant think of variable name feel free to suggest, make pr. lol

thomask commented 2 years ago

That was fast 😀

However (without having tested) I think the BC break is still there, because my opts, which were/are passed as the first argument, are not being passed to puppeteer.launch.

Perhaps you meant to launch with the first argument in case Puppeteer is not passed as the first argument?

ganganimaulik commented 2 years ago
  const browser = await launch({
    defaultViewport: null,
    devtools: true,
  });

if I try to pass arguments like this, it's working fine for me - shows devtools (without new pr). not sure what you're talking about.

thomask commented 2 years ago

My bad, I had missed these lines