Closed szmarczak closed 2 years ago
This currently uses Manifest V2. Manifest V2 is scheduled to be turned off in January 2023. For enterprise June 2023.
I tried using Manifest V3 - that means the cookie headers need be modified via CDP (not using Playwright because it turns off caching). That works BUT unfortunately due to https://crbug.com/1292450 and https://crbug.com/1200844 it's not possible to modify the cookie headers. Instead, I removed the cookies and set them manually. So, at the point of sending request, there were no cookies for that particular website in the browser - this way Chromium accepted modified cookie headers. Unfortunately for some reason Gmail (it's a good test) doesn't like that. I tried shifting cookies instead (delete other cookies -> rename cookies for the current session -> send request -> restore old cookies), but that is buggy as hell - I could write more details about this but it has so many bugs that I'd say those cookies were living their own life :P
tl;dr: Manifest V2 it works like a charm. Let's hope those two issues get fixed in 6 months.
Btw I dont think we want to merge this here, should go to the crawlee monorepo instead.
(we can, but generally this repo will be deprecated, and @crawlee/browser-pool
will be the place where future development will happen)
Is SDK v2 gonna use the crawlee one as well?
Nope, we wont be maintaining v2 after crawlee is out, maybe some critical fixes, but not features.
(it would be also BC, as crawlee packages require node 16, while SDK v2 supports node 15 too)
Makes sense. Thanks for clarifying. I'll open a PR in the crawlee monorepo then.
Not sure why CI fails, I guess unrelated breaking
playwright
change...Result:
It works! Headless as well! Draft because I need to put the above into a Jest test yet.