Script 'mail_helper' called by obssrc
Hello community,

here is the log from the commit of package iprange for openSUSE:Factory checked 
in at 2026-04-09 16:09:04
++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++
Comparing /work/SRC/openSUSE:Factory/iprange (Old)
 and      /work/SRC/openSUSE:Factory/.iprange.new.21863 (New)
++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++

Package is "iprange"

Thu Apr  9 16:09:04 2026 rev:5 rq:1345204 version:2.0.0

Changes:
--------
--- /work/SRC/openSUSE:Factory/iprange/iprange.changes  2026-04-07 
16:48:49.344325420 +0200
+++ /work/SRC/openSUSE:Factory/.iprange.new.21863/iprange.changes       
2026-04-09 16:21:52.951453549 +0200
@@ -1,0 +2,10 @@
+Wed Apr  8 01:59:06 UTC 2026 - Mia Herkt <[email protected]>
+
+- Add firehol-dns-handle-EAI_AGAIN.patch
+  Adds an error message for when the maximum number of retries
+  due to temporary name resolution failure is reached.
+  Fixes a test case that was failing on OBS due to lack of
+  Internet connectivity.
+  gh#firehol/iprange#42
+
+-------------------------------------------------------------------

New:
----
  firehol-dns-handle-EAI_AGAIN.patch

----------(New B)----------
  New:
- Add firehol-dns-handle-EAI_AGAIN.patch
  Adds an error message for when the maximum number of retries
----------(New E)----------

++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++

Other differences:
------------------
++++++ iprange.spec ++++++
--- /var/tmp/diff_new_pack.H9hQDA/_old  2026-04-09 16:21:53.459474394 +0200
+++ /var/tmp/diff_new_pack.H9hQDA/_new  2026-04-09 16:21:53.463474558 +0200
@@ -23,6 +23,8 @@
 License:        GPL-2.0-or-later
 URL:            https://firehol.org/
 Source:         
https://github.com/firehol/iprange/releases/download/v%{version}/iprange-%{version}.tar.xz
+# PATCH-FIX-UPSTREAM -- Prints error message when DNS fails after retries, 
fixing a test case -- https://github.com/firehol/iprange/issues/42
+Patch0:         
https://github.com/firehol/iprange/commit/60f054fcf93759b88d6f752376b08610ea6580a9.patch#/firehol-dns-handle-EAI_AGAIN.patch
 BuildRequires:  autoconf
 BuildRequires:  automake
 
@@ -30,11 +32,7 @@
 This tool manages IP address ranges for FireHOL.
 
 %prep
-%autosetup
-# Test expects permanent failure but no network connection means we
-# only get temporary name resolution errors. Reported upstream:
-# https://github.com/firehol/iprange/issues/42
-rm -r tests.d/64-dns-failure-exit-status
+%autosetup -p1
 
 %build
 %configure

++++++ firehol-dns-handle-EAI_AGAIN.patch ++++++
>From 60f054fcf93759b88d6f752376b08610ea6580a9 Mon Sep 17 00:00:00 2001
From: Costa Tsaousis <[email protected]>
Date: Tue, 7 Apr 2026 23:05:01 +0300
Subject: [PATCH] dns: print failure message when EAI_AGAIN retries are
 exhausted
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

When getaddrinfo returns EAI_AGAIN ("Temporary failure in name
resolution"), iprange retries the request up to 20 times. After the last
retry, the request was silently completed without any final diagnostic.

On systems without network connectivity (e.g. distribution package build
VMs), this causes the user to see twenty "will be retried" lines and
then nothing, even though the command exits non-zero. It also breaks
tests.d/64-dns-failure-exit-status, which greps stderr for "failed
permanently" — a string only emitted on the EAI_NONAME path.

Mirror the EAI_NONAME branch and emit a "failed permanently after
retries" message before completing the request, so the user (and the
test) get a clear final error regardless of whether the resolver reached
a server.

Fixes #42
---
 src/ipset_dns.c | 2 ++
 1 file changed, 2 insertions(+)

diff --git a/src/ipset_dns.c b/src/ipset_dns.c
index 2dadf65..80711af 100644
--- a/src/ipset_dns.c
+++ b/src/ipset_dns.c
@@ -147,6 +147,8 @@ static void dns_request_failed(DNSREQ *d, int added, int 
gai_error)
                 dns_unlock_requests();
                 return;
             }
+            if(!dns_silent)
+                fprintf(stderr, "%s: DNS: '%s' failed permanently after 
retries: %s\n", PROG, d->hostname, gai_strerror(gai_error));
             dns_request_done(d, added);
             return;
 

Reply via email to