mirror of
https://github.com/ArchiveBox/ArchiveBox.git
synced 2026-04-25 17:16:00 +03:00
[GH-ISSUE #1657] Feature Request: More robust export_browser_history.sh #4007
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#4007
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?
Originally created by @pcrockett on GitHub (Feb 16, 2025).
Original GitHub issue: https://github.com/ArchiveBox/ArchiveBox/issues/1657
Originally assigned to: @pirate on GitHub.
What type of suggestion are you making?
Proposing a new feature
What is the problem that your feature request solves?
Looking at available sources, archiving browser history requires running export_browser_history.sh.
However I see a few issues:
What is your proposed solution?
I'm a bit of a Bash nerd and would love to make this work with Linux and Firefox at least. I've already started here. Is this kind of contribution something you would take?
Side notes:
What hacks or alternative solutions have you tried to solve the problem?
Pass the full file name to the script after the
--firefoxargument. But that still fails with a sqlite syntax error.Share the entire output of the
archivebox versioncommand for the current verison you are using.This is on the latest
devbranch. The last time this script was touched was ingithub.com/ArchiveBox/ArchiveBox@aa5533b80fHow badly do you want this new feature?
Mini Survey
@pirate commented on GitHub (Feb 17, 2025):
Before writing any new code, can you try reverting that PR and seeing if https://github.com/ArchiveBox/ArchiveBox/pull/1152/files
Also you should check out the latest ArchiveBox browser extension PR, it adds support for importing from browser history through the extension UI now: https://github.com/ArchiveBox/archivebox-browser-extension/pull/31
@pcrockett commented on GitHub (Feb 17, 2025):
My code is already based on the commit that you linked. That commit fixed one sqlite syntax error, but left another syntax error above it (should be
SELECT '['instead ofSELECT \"[\").The first syntax error probably wasn't caught because the script wasn't using
set -eo pipefail, which is another thing my implementation adds.I will indeed check out that browser extension, thanks.
UPDATE: Checked out the extension. I plan to use it going forward, but this script is more useful to those who want to retroactively import their browser history into ArchiveBox.
@pirate commented on GitHub (Feb 18, 2025):
I opened a PR to track your fixes: #1661. can you check the diff and let me know if it looks ready for review/merge? Thanks!
@pcrockett commented on GitHub (Feb 19, 2025):
Ready for review, with a few comments:
There are probably other things that could be improved, but this is a good step in the right direction and we don't want to overengineer something that's probably a very minor part of the project.