From mboxrd@z Thu Jan 1 00:00:00 1970 From: Peter =?utf-8?q?M=C3=BCller?= To: development@lists.ipfire.org Subject: [PATCH] squid-asnbl: Update to 0.2.5 Date: Mon, 14 Aug 2023 23:14:00 +0000 Message-ID: <315c2f05-a0cd-4488-b5b2-48b6c6aa3d00@ipfire.org> MIME-Version: 1.0 Content-Type: multipart/mixed; boundary="===============5591098575347423102==" List-Id: --===============5591098575347423102== Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: quoted-printable This upstream release incorporates the patch added for resolving #13023. Signed-off-by: Peter M=C3=BCller --- lfs/squid-asnbl | 5 +- ...les_to_make_compatible_with_python_3.patch | 100 ------------------ 2 files changed, 2 insertions(+), 103 deletions(-) delete mode 100644 src/patches/squid/squid-asnbl-0.2.4_initialise_global_var= iables_to_make_compatible_with_python_3.patch diff --git a/lfs/squid-asnbl b/lfs/squid-asnbl index b003d605b..32c109e15 100644 --- a/lfs/squid-asnbl +++ b/lfs/squid-asnbl @@ -25,7 +25,7 @@ =20 include Config =20 -VER =3D 0.2.4 +VER =3D 0.2.5 =20 THISAPP =3D squid-asnbl-$(VER) DL_FILE =3D $(THISAPP).tar.gz @@ -45,7 +45,7 @@ objects =3D $(DL_FILE) =20 $(DL_FILE) =3D $(DL_FROM)/$(DL_FILE) =20 -$(DL_FILE)_BLAKE2 =3D 98bdf0ecff1fc4d4ac0e0eb43d37682acf76900df783bf31b17213= 4ebe31a857594e0a98ee06347da40d97b86a6ac0ce9a63ea7a80a1d73948e338f72d1c3af2 +$(DL_FILE)_BLAKE2 =3D 7206b9fe6c4a0509c9ce0301a5d3553274d005daae2050f23e2605= 6d356c328719821d58d94ed2eac7942a6cafb0ca8de2cb619ce28a0f514b6eef413350e249 =20 install : $(TARGET) =20 @@ -75,7 +75,6 @@ $(subst %,%_BLAKE2,$(objects)) : $(TARGET) : $(patsubst %,$(DIR_DL)/%,$(objects)) @$(PREBUILD) @rm -rf $(DIR_APP) && cd $(DIR_SRC) && tar zvxf $(DIR_DL)/$(DL_FILE) - cd $(DIR_APP) && patch -Np1 -i $(DIR_SRC)/src/patches/squid/squid-asnbl-0.2= .4_initialise_global_variables_to_make_compatible_with_python_3.patch =20 # Install ASNBL helper script cd $(DIR_APP) && install -o root -g root -m 0755 asnbl-helper.py /usr/bin/a= snbl-helper.py diff --git a/src/patches/squid/squid-asnbl-0.2.4_initialise_global_variables_= to_make_compatible_with_python_3.patch b/src/patches/squid/squid-asnbl-0.2.4_= initialise_global_variables_to_make_compatible_with_python_3.patch deleted file mode 100644 index e540d4e76..000000000 --- a/src/patches/squid/squid-asnbl-0.2.4_initialise_global_variables_to_make= _compatible_with_python_3.patch +++ /dev/null @@ -1,100 +0,0 @@ ---- squid-asnbl-0.2.4/asnbl-helper_orig.py -+++ squid-asnbl-0.2.4/asnbl-helper.py -@@ -172,17 +172,19 @@ - return parsedasns -=20 -=20 --def resolve_asn(ipaddr: str, asndb): -- """ Function call: resolve_asn(IP address to be resolved, -- ASN database instance object) -- This function looks up the Autonomous System for the given IP address. = It expects -- an IPFire location database object to be passed as a second parameter, = hence relying -- on another function to set that up. """ -+def resolve_asn(ipaddr: str): -+ """ Function call: resolve_asn(IP address to be resolved) -+ This function looks up the Autonomous System for the given IP address. = """ -+ -+ # Fix for #13023 -+ # Initialize the result variable before it's first use, otherwise pytho= n3 -+ # will sometimes detect a 'mismatch' using global and local variables -+ lookup_result =3D None -=20 - # libloc cannot handle ipaddress objects here, so casting into a string= is necessary - # for good measure, to avoid exceptions here... - try: -- result =3D asndb.lookup(str(ipaddr)) -+ lookup_result =3D ASNDB.lookup(str(ipaddr)) - except BlockingIOError: - # XXX: Prevent likely libloc bug from causing this helper to crash - # (see upstream bug https://bugzilla.ipfire.org/show_bug.cgi?id=3D1= 3023) -@@ -190,21 +192,25 @@ -=20 - # In case nothing was returned above, satisfy result expectation to thi= s function... - try: -- if not result.asn: -+ if not lookup_result.asn: - return 0 - except AttributeError: - return 0 -=20 -- return result.asn -- -- --def asndb_response_tests(testdata: str, asndb): -- """ Function call: asndb_response_tests(response rest data, -- ASN database instance object) -+ return lookup_result.asn -+ -+ -+def asndb_response_tests(testdata: str): -+ """ Function call: asndb_response_tests(response rest data) -=20 - This function asserts the given ASN database to return expected ASNs for - given IP addresses in order to be considered operational. It returns - True if this test succeeds, and False otherwise. """ -+ -+ # Fix for #13023 -+ # Initialize the result variable before it's first use, otherwise pytho= n3 -+ # will sometimes detect a 'mismatch' using global and local variables -+ lookup_result_test =3D None -=20 - tresult =3D True -=20 -@@ -216,13 +222,13 @@ -=20 - for stestdata in ptdata: - LOGIT.debug("Running response test for '%s' against ASNDB '%s' ...", -- stestdata, asndb) -- -- returndata =3D resolve_asn(stestdata[0], asndb) -- -- if returndata !=3D int(stestdata[1]): -+ stestdata, ASNDB) -+ -+ lookup_result_test =3D resolve_asn(stestdata[0]) -+ -+ if lookup_result_test !=3D int(stestdata[1]): - LOGIT.error("Response test failed for ASNDB '%s' (tuple: %s), a= borting", -- asndb, stestdata) -+ ASNDB, stestdata) - tresult =3D False - break -=20 -@@ -428,7 +434,7 @@ - ASNDB =3D set_up_location_database(config["GENERAL"]["ASNDB_PATH"]) -=20 - LOGIT.debug("Running ASN database response tests...") --if asndb_response_tests(config["GENERAL"]["TESTDATA"], ASNDB): -+if asndb_response_tests(config["GENERAL"]["TESTDATA"]): - LOGIT.debug("ASN database operational - excellent. Waiting for input...= ") - else: - LOGIT.error("ASN database response tests failed, aborting") -@@ -490,7 +496,7 @@ - ASNS =3D [] - for singleip in IPS: - # Enumerate ASN for this IP address... -- resolvedasn =3D resolve_asn(singleip, ASNDB) -+ resolvedasn =3D resolve_asn(singleip) -=20 - # In case protection against destinations without public AS announc= ements for their - # IP addresses is desired, the query will be denied in case ASN =3D= 0 appears in an --=20 2.35.3 --===============5591098575347423102==--