mirror of
https://github.com/ArchiveBox/ArchiveBox.git
synced 2026-04-25 09:06:02 +03:00
[PR #936] [MERGED] Fix bin_version: set LANG=C when calling executables to avoid parsing localized output #4311
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#4311
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/936
Author: @pellaeon
Created: 2/24/2022
Status: ✅ Merged
Merged: 3/5/2022
Merged by: @pirate
Base:
dev← Head:pr-binversion📝 Commits (1)
5e9d054Fix bin_version: set LANG=C when calling executables to avoid parsing📊 Changes
1 file changed (+1 additions, -1 deletions)
View changed files
📝
archivebox/config.py(+1 -1)📄 Description
Summary
bin_versionis used to set user agent string intitleextractor.bin_versioncalls external executables with--versionand parses its output. The resulting version string is used to set user agent string.Executables might output in localized languages, however, user agent strings can only be in
latin-1, resulting in this error:This problem is fixed by running executables with environment variable LANG=C.
Example of running
wget --versionwhenLANG="zh_TW.UTF-8":Related issues
Changes these areas
🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.