mirror of
https://github.com/ArchiveBox/ArchiveBox.git
synced 2026-04-26 01:26:00 +03:00
[GH-ISSUE #615] Bugfix: Cookies file causes wget failures #1891
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#1891
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 @winteriscariot on GitHub (Jan 12, 2021).
Original GitHub issue: https://github.com/ArchiveBox/ArchiveBox/issues/615
Describe the bug
Steps to reproduce
On Arch Linux,
Screenshots or log output
Software versions
@pirate commented on GitHub (Jan 13, 2021):
Please post the full output of
archivebox versionwith no redactions.@zapp-brannigan commented on GitHub (Jan 18, 2021):
Hi. I have the same Problem since 0.5.3:
archivebox version:
@pirate commented on GitHub (Jan 18, 2021):
Looks like you have
COOKIES_FILEdisabled in that output, can you post the value you're using for it and exactly how you set it. e.g. via config file,archivebox configcommand, env var, etc.@zapp-brannigan commented on GitHub (Jan 18, 2021):
I've just commented it out in my config:
Long time ago i used this command to enable cookies:
@pirate commented on GitHub (Jan 18, 2021):
Ah I think it's because it's read as a
pathlib.Pathin the config loading process, and needs to be converted to astrbefore being added to the wget command during the extraction processCMD = ['wget', ...].@pirate commented on GitHub (Jan 21, 2021):
Fixed:
ef7711f