fix: Clean up tmp profile dirs when browser is closed #8580
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
What kind of change does this PR introduce?
Bugfix: Puppeteer is not deleting tmp profiles after browser is closed.
Did you add tests for your changes?
Yes, I actually first created the test to verify that it was actually broken the way I thought.
If relevant, did you update the documentation?
No, this is a small bugfix, but let me know if I should update anything.
Summary
Puppeteer has a feature in which if you don't specify a user data dir (
--user-data-dir
) it automatically creates one in a tmp folder, and then it's deleted when the browser is closed.It seems that this feature got broken here, as it is now setting
isTempUserDataDir
to false in any case.In my particular case, I instantiate Puppeteer multiple times, leading to an infinite number of dirs in the /tmp folder which were only deleted when I restarted the node.
Does this PR introduce a breaking change?
No. (unless for any reason somebody is expecting for tmp profiles to be kept)