diff --git a/package.json b/package.json index 2fe8536..8277234 100644 --- a/package.json +++ b/package.json @@ -1,6 +1,6 @@ { "name": "browser-pool", - "version": "2.0.2", + "version": "2.0.3", "description": "Rotate multiple browsers using popular automation libraries such as Playwright or Puppeteer.", "main": "dist/index.js", "module": "dist/index.mjs", @@ -41,7 +41,6 @@ }, "dependencies": { "@apify/log": "^1.1.1", - "fs-extra": "^10.0.0", "lodash.merge": "^4.6.2", "nanoid": "^3.1.25", "ow": "^0.27.0", @@ -60,6 +59,7 @@ "@typescript-eslint/eslint-plugin": "^4.29.1", "@typescript-eslint/parser": "^4.29.1", "eslint": "^7.32.0", + "fs-extra": "^10.0.0", "gen-esm-wrapper": "^1.1.2", "jest": "^27.0.6", "jest-circus": "^27.0.6", diff --git a/src/abstract-classes/browser-plugin.ts b/src/abstract-classes/browser-plugin.ts index 6758153..06d0505 100644 --- a/src/abstract-classes/browser-plugin.ts +++ b/src/abstract-classes/browser-plugin.ts @@ -1,5 +1,4 @@ import merge from 'lodash.merge'; -import { ensureDir } from 'fs-extra'; import { log } from '../logger'; import { LaunchContext, LaunchContextOptions } from '../launch-context'; import { BrowserController } from './browser-controller'; @@ -143,16 +142,12 @@ export abstract class BrowserPlugin< async launch( launchContext: LaunchContext = this.createLaunchContext(), ): Promise { - const { proxyUrl, useIncognitoPages, userDataDir } = launchContext; + const { proxyUrl } = launchContext; if (proxyUrl) { await this._addProxyToLaunchOptions(launchContext); } - if (!useIncognitoPages) { - await ensureDir(userDataDir); - } - return this._launch(launchContext); } diff --git a/src/launch-context.ts b/src/launch-context.ts index ecad5db..43de880 100644 --- a/src/launch-context.ts +++ b/src/launch-context.ts @@ -1,6 +1,3 @@ -import { join } from 'path'; -import { tmpdir } from 'os'; -import { nanoid } from 'nanoid'; import type { BrowserPlugin, CommonBrowser, CommonLibrary } from './abstract-classes/browser-plugin'; import { UnwrapPromise } from './utils'; @@ -77,7 +74,7 @@ export class LaunchContext< launchOptions, proxyUrl, useIncognitoPages, - userDataDir = join(tmpdir(), nanoid()), + userDataDir = '', } = options; this.id = id; diff --git a/test/browser-plugins/plugins.test.ts b/test/browser-plugins/plugins.test.ts index 686ba83..03d1e46 100644 --- a/test/browser-plugins/plugins.test.ts +++ b/test/browser-plugins/plugins.test.ts @@ -1,6 +1,5 @@ import puppeteer from 'puppeteer'; import playwright from 'playwright'; -import fs from 'fs'; import { PuppeteerPlugin } from '../../src/puppeteer/puppeteer-plugin'; import { PuppeteerController } from '../../src/puppeteer/puppeteer-controller'; @@ -76,18 +75,6 @@ const runPluginTest = < expect(context.useIncognitoPages).toEqual(desiredObject.useIncognitoPages); }); - test('should create userDatadir', async () => { - plugin = new Plugin(library as never, { - useIncognitoPages: false, - }); - - const context = plugin.createLaunchContext(); - browser = await plugin.launch(context as never); - - expect(fs.existsSync(context.userDataDir)).toBeTruthy(); - await browser.close(); - }); - test('should get default launchContext values from plugin options', async () => { const proxyUrl = 'http://apify1234@10.10.10.0:8080/';