mirror of
https://github.com/ArchiveBox/ArchiveBox.git
synced 2026-04-26 01:26:00 +03:00
[PR #1219] [MERGED] After a timeout, chrome will leave behind a SingletonLock, which prev… #1355
Labels
No labels
expected: maybe someday
expected: next release
expected: release after next
expected: unlikely unless contributed
good first ticket
help wanted
pull-request
scope: all users
scope: windows users
size: easy
size: hard
size: medium
size: medium
status: backlog
status: blocked
status: done
status: idea-phase
status: needs followup
status: wip
status: wontfix
touches: API/CLI/Spec
touches: configuration
touches: data/schema/architecture
touches: dependencies/packaging
touches: docs
touches: js
touches: views/replayers/html/css
why: correctness
why: functionality
why: performance
why: security
No milestone
No project
No assignees
1 participant
Notifications
Due date
No due date set.
Dependencies
No dependencies set.
Reference
starred/ArchiveBox#1355
Loading…
Add table
Add a link
Reference in a new issue
No description provided.
Delete branch "%!s()"
Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?
📋 Pull Request Information
Original PR: https://github.com/ArchiveBox/ArchiveBox/pull/1219
Author: @spresse1
Created: 8/28/2023
Status: ✅ Merged
Merged: 8/29/2023
Merged by: @pirate
Base:
dev← Head:chrome-cleanup📝 Commits (1)
603ce7eAfter a timeout, chrome will leave behind a SingletonLock, which prevents future instances of chrome from starting. When an extractor fails due to a timeout, remove this file.📊 Changes
4 files changed (+18 additions, -0 deletions)
View changed files
📝
archivebox/extractors/dom.py(+2 -0)📝
archivebox/extractors/pdf.py(+2 -0)📝
archivebox/extractors/screenshot.py(+2 -0)📝
archivebox/util.py(+12 -0)📄 Description
Summary
After a timeout in the pdf, screenshot, or dom extractor inside a docker container, chrome will leave behind a file at ~/.config/chromium/SingletonLock. This stops all three extractors from functioning until the docker container is completely torn down and regenerated.
This code adds removal of this file after a timeout only when running in a docker container. There is no behavior change outside a docker container. This was a deliberate choice on my part, as I don't want to interfere with a user's running chrome sessions when not running in a docker container.
Related issues
#1181
Changes these areas
🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.
aria2with bittorrents link/FTP/SFTP/Metalink download support #2339aria2with bittorrents link/FTP/SFTP/Metalink download support #3849