From mboxrd@z Thu Jan  1 00:00:00 1970
From: Michael Tremer <michael.tremer@ipfire.org>
To: development@lists.ipfire.org
Subject: Re: [PATCH] tor: Fix tor permissions if presant via update.sh
Date: Mon, 10 Jun 2019 15:31:16 +0100
Message-ID: <EF99C8E3-1BBB-41D5-B87B-1E22967D35A7@ipfire.org>
In-Reply-To: <20190610113652.11536-1-ummeegge@ipfire.org>
MIME-Version: 1.0
Content-Type: multipart/mixed; boundary="===============1563583316831010080=="
List-Id: <development.lists.ipfire.org>

--===============1563583316831010080==
Content-Type: text/plain; charset="utf-8"
Content-Transfer-Encoding: quoted-printable

Hi,

Thanks for sending in that patch.

But I think instead of doing this in the Core Update, this change should be a=
pplied in the tor update scripts.

-Michael

> On 10 Jun 2019, at 12:36, Erik Kapfer <ummeegge(a)ipfire.org> wrote:
>=20
> Signed-off-by: Erik Kapfer <ummeegge(a)ipfire.org>
> ---
> config/rootfiles/core/133/update.sh | 5 +++++
> 1 file changed, 5 insertions(+)
>=20
> diff --git a/config/rootfiles/core/133/update.sh b/config/rootfiles/core/13=
3/update.sh
> index a05ad0741..3ecb5651b 100644
> --- a/config/rootfiles/core/133/update.sh
> +++ b/config/rootfiles/core/133/update.sh
> @@ -71,6 +71,11 @@ sudo -u nobody /srv/web/ipfire/cgi-bin/vpnmain.cgi
> /etc/init.d/squid start
> /etc/init.d/collectd restart
>=20
> +# Set new permissions for tor
> +if [ -d "/var/lib/tor" ]; then
> +    chown -R tor:tor /var/lib/tor
> +fi
> +
> # Finish
> /etc/init.d/fireinfo start
> sendprofile
> --=20
> 2.12.2
>=20


--===============1563583316831010080==--