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] calamaris: Bug fix for proxy reports staying empty after
 Core 136 upgrade
Date: Thu, 14 Nov 2019 18:22:23 +0000
Message-ID: <874B7B20-2739-4188-95A6-51800B9DBC89@ipfire.org>
In-Reply-To: <20191114180346.2010-1-matthias.fischer@ipfire.org>
MIME-Version: 1.0
Content-Type: multipart/mixed; boundary="===============1366529805563359769=="
List-Id: <development.lists.ipfire.org>

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

Reviewed-by: Michael Tremer <michael.tremer(a)ipfire.org>

> On 14 Nov 2019, at 18:03, Matthias Fischer <matthias.fischer(a)ipfire.org> =
wrote:
>=20
> After upgrading to Core 136, 'calamaris' "Proxy reports" stayed empty.
> GUI always show "No reports available".
>=20
> Tested manually on console stops and throws an error:
>=20
> ...
> root(a)ipfire: ~ # /usr/bin/perl /var/ipfire/proxy/calamaris/bin/mkreport
> 1 0 2019 8 10 2019 -d 10 -P 30 -t 10 -D 2 -u -r -1 -R 100 -s
> Can't use 'defined(%hash)' (Maybe you should just omit the defined()?)
> at /var/ipfire/proxy/calamaris/bin/calamaris line 2609.
> ...
>=20
> Line 2609 was changed and reports are built again.
>=20
> Signed-off-by: Matthias Fischer <matthias.fischer(a)ipfire.org>
> ---
> lfs/calamaris                                        |  5 ++++-
> .../01_calamaris_cant_use_defined_hash.patch         | 12 ++++++++++++
> 2 files changed, 16 insertions(+), 1 deletion(-)
> create mode 100644 src/patches/calamaris/01_calamaris_cant_use_defined_hash=
.patch
>=20
> diff --git a/lfs/calamaris b/lfs/calamaris
> index 2022f84f8..32858526a 100644
> --- a/lfs/calamaris
> +++ b/lfs/calamaris
> @@ -1,7 +1,7 @@
> ###########################################################################=
####
> #                                                                          =
   #
> # IPFire.org - A linux based firewall                                      =
   #
> -# Copyright (C) 2007-2018  IPFire Team  <info(a)ipfire.org>               =
      #
> +# Copyright (C) 2007-2019  IPFire Team  <info(a)ipfire.org>               =
      #
> #                                                                          =
   #
> # This program is free software: you can redistribute it and/or modify     =
   #
> # it under the terms of the GNU General Public License as published by     =
   #
> @@ -70,6 +70,9 @@ $(subst %,%_MD5,$(objects)) :
> $(TARGET) : $(patsubst %,$(DIR_DL)/%,$(objects))
> 	@$(PREBUILD)
> 	@rm -rf $(DIR_APP) && cd $(DIR_SRC) && tar zxf $(DIR_DL)/$(DL_FILE)
> +
> +	cd $(DIR_APP) && patch -Np1 -i $(DIR_SRC)/src/patches/calamaris/01_calama=
ris_cant_use_defined_hash.patch
> +
> 	cd $(DIR_APP) && cp -f calamaris $(DIR_SRC)/config/calamaris/mkreport \
> 			/var/ipfire/proxy/calamaris/bin/
> 	chmod 755 /var/ipfire/proxy/calamaris/bin/{calamaris,mkreport}
> diff --git a/src/patches/calamaris/01_calamaris_cant_use_defined_hash.patch=
 b/src/patches/calamaris/01_calamaris_cant_use_defined_hash.patch
> new file mode 100644
> index 000000000..64d66276c
> --- /dev/null
> +++ b/src/patches/calamaris/01_calamaris_cant_use_defined_hash.patch
> @@ -0,0 +1,12 @@
> +diff U3 a/calamaris b/calamaris
> +--- a/calamaris	Sun Jun  6 18:26:14 2004
> ++++ b/calamaris	Wed Nov 13 19:59:15 2019
> +@@ -2606,7 +2606,7 @@
> + 	    100, 100 * $tcp_hit / $tcp );
> +   }
> +   outstop(10);
> +-  if ( defined(%tcp_content) ) {
> ++  if (%tcp_content) {
> +     outstart(11);
> +     if ( $tcp =3D=3D 0 ) {
> +       outline( 11, 'no matching requests' );
> --=20
> 2.18.0
>=20


--===============1366529805563359769==--