FIX error when doing sed on non existing privoxy config file#64
FIX error when doing sed on non existing privoxy config file#64pubkey wants to merge 1 commit intodperson:masterfrom
Conversation
|
Are you sure that this is a fix? Because you'll get an empty config for |
|
It works for me at least. |
They're not filling the file. Did you check it's content after the build? |
skobkin
left a comment
There was a problem hiding this comment.
You can probably bring the old /etc/privoxy/config to the repository and add it using COPY. But you'll need to check that it works well first.
Currently the image will built successfully, but only Tor part of it will be working as intended. So there is no point in such changes, you can just use image which only contains Tor itself.
| apk --no-cache --no-progress add bash curl privoxy shadow tini tor tzdata&&\ | ||
| apk --no-cache --no-progress add bash curl privoxy shadow tini tor tzdata && \ | ||
| file='/etc/privoxy/config' && \ | ||
| touch $file && \ |
There was a problem hiding this comment.
This command just creates an empty file.
The following sed commands are doing nothing after that. So after this fix privoxy will not function as intended by the maintainer.
|
Installing privoxy nowadays adds '.new' to its default config files under /etc/privoxy. I maintain an up to date version under https://github.com/SimonHaas/torproxy |
When doing a full build from the

Dockerfilein the repo, I got this error because theRUNcommand tries to runsedon a file that does not exist.This commit ensures that the privoxy config file exists, before working with it.