Release 1.14

.github/images/centos.Dockerfile

@@ -2,8 +2,8 @@ ARG image=centos/centos:latest
 FROM quay.io/$image
 
 # Install dependencies
-RUN dnf -y update
-RUN dnf -y install autoconf automake gcc libtool make diffutils file gzip
+RUN if command -v yum > /dev/null; then dnf=yum; fi; ${dnf:-dnf} update -y
+RUN if command -v yum > /dev/null; then dnf=yum; fi; ${dnf:-dnf} install -y autoconf automake gcc libtool make diffutils file gzip
 
 # Add source code
 ADD . /src

.github/images/fedora/fedora:38.Dockerfile

@@ -0,0 +1 @@
+../centos.Dockerfile

.github/images/fedora/fedora:39.Dockerfile

@@ -0,0 +1 @@
+../centos.Dockerfile

.github/images/fedora/fedora:42.Dockerfile

@@ -1,17 +0,0 @@
-ARG image=centos/centos:latest
-FROM quay.io/$image
-
-# Install dependencies
-RUN dnf -y update
-RUN dnf -y install autoconf automake gcc libtool make diffutils file gzip awk
-
-# Add source code
-ADD . /src
-WORKDIR /src
-
-# Run steps
-RUN ./bootstrap
-RUN ./configure
-RUN make
-RUN make check
-RUN make distcheck

.github/images/ubuntu:bionic.Dockerfile

@@ -0,0 +1 @@
+debian.Dockerfile

.github/workflows/build-container.yml

@@ -19,7 +19,7 @@ jobs:
 
   build:
     name: Build container
-    runs-on: ubuntu-24.04
+    runs-on: ubuntu-22.04
     needs: test
 
     steps:
@@ -56,7 +56,7 @@ jobs:
           password: ${{ secrets.GITHUB_TOKEN }}
 
       - name: Build and push
-        uses: docker/build-push-action@v6
+        uses: docker/build-push-action@v5
         with:
           file: .github/images/alpine:3.Dockerfile
           context: .

.github/workflows/matrixbuild.yml

@@ -12,22 +12,24 @@ jobs:
           - debian:trixie
           - debian:bookworm
           - debian:bullseye
-          - ubuntu:noble
+          - debian:buster
           - ubuntu:jammy
           - ubuntu:focal
-          - fedora/fedora:42
-          - fedora/fedora:41
+          - ubuntu:bionic
           - fedora/fedora:40
-          - centos/centos:stream10
+          - fedora/fedora:39
+          - fedora/fedora:38
           - centos/centos:stream9
+          - centos/centos:stream8
+          - centos/centos:7
           - rockylinux/rockylinux:9
           - rockylinux/rockylinux:8
           - alpine:edge
-          - alpine:3.21
+          - alpine:3.19
     steps:
     - uses: actions/checkout@v4
     - name: Work around Docker BuildKit regression
       # https://github.com/moby/buildkit/issues/2119: `DOCKER_BUILDKIT=1 docker build` fails if Dockerfile is a symlink
-      run: [ -L .github/images/${{matrix.dockerenv}}.Dockerfile ] && cp --remove-destination $(readlink -f .github/images/${{matrix.dockerenv}}.Dockerfile) .github/images/${{matrix.dockerenv}}.Dockerfile
+      run: cp --remove-destination $(readlink -f .github/images/${{matrix.dockerenv}}.Dockerfile) .github/images/${{matrix.dockerenv}}.Dockerfile
     - name: Run build on ${{matrix.dockerenv}}
       run: docker build . --file .github/images/${{matrix.dockerenv}}.Dockerfile --build-arg image=${{matrix.dockerenv}}

.github/workflows/unit-tests.yml

@@ -8,15 +8,12 @@ on:
       - reopened
       - ready_for_review
       - synchronize
-  push:
-    branches:
-      - main
   workflow_call:
 
 jobs:
   output-unit-tests:
     name: output unit tests
-    runs-on: ubuntu-24.04
+    runs-on: ubuntu-22.04
     steps:
       - name: clone repo
         uses: actions/checkout@v4

CHANGES

@@ -1,6 +1,3 @@
-1.15 (2024-05-15)
-    - Apologies, debug code slipped into the last release
-
 1.14 (2024-05-14)
     - Small performance gain: set TCP_NODELAY on the socket
 

VERSION

@@ -1 +1 @@
-1.15
+1.14

expander.c

@@ -1035,7 +1035,7 @@ bgpq_expand(struct bgpq_expander *b)
 	struct addrinfo 	 hints, *res = NULL, *rp;
 	struct linger		 sl;
 	struct asn_entry	*asne;
-	int			 fd = -1, err, ret, aquery = 0, nodelay = 1;
+	int			 fd = -1, err, ret, aquery = 0, nodelay = 0;
 	int			 slen;
 
 	sl.l_onoff = 1;
@@ -1076,11 +1076,19 @@ bgpq_expand(struct bgpq_expander *b)
 			continue;
 		}
 
+		socklen_t len = sizeof(nodelay);
+		getsockopt(fd, IPPROTO_TCP, TCP_NODELAY, &nodelay, &len);
+		SX_DEBUG(debug_expander, "TCP_NODELAY set to %i\n", nodelay);
+
+		nodelay = 0;
 		if (setsockopt(fd, IPPROTO_TCP, TCP_NODELAY, &nodelay,
 		    sizeof(nodelay)) == -1)
 			SX_DEBUG(debug_expander, "Unable to set TCP_NODELAY on"
 			    " socket: %s\n", strerror(errno));
 
+		getsockopt(fd, IPPROTO_TCP, TCP_NODELAY, &nodelay, &len);
+		SX_DEBUG(debug_expander, "TCP_NODELAY set to %i\n", nodelay);
+
 		break;
 	}
 

main.c

@@ -604,14 +604,6 @@ main(int argc, char* argv[])
 		exit(1);
 	}
 
-	if (expander.vendor == V_ARISTA
-	    && expander.generation == T_EACL
-	    && expander.family == AF_INET6) {
-		sx_report(SX_FATAL, "Sorry, extended access-lists is not compatible "
-		    "with Arista EOS and IPv6\n");
-		exit(1);
-	}
-	
 	if (expander.sequence
 	    && (expander.vendor != V_CISCO && expander.vendor != V_ARISTA)) {
 		sx_report(SX_FATAL, "Sorry, prefix-lists sequencing (-s) supported"

printer.c

@@ -300,14 +300,11 @@ bgpq4_print_juniper_aslist(FILE *f, struct bgpq_expander *b)
 
 	RB_FOREACH(asne, asn_tree, &b->asnlist) {
 		if (!nc) {
-			fprintf(f, "  as-list a%u members [",
-			    lineNo);
-		}
-
-		// Filter out AS 0
-		// "error: RPD Policy: Invalid AS 0"
-		if (asne->asn != 0)
+			fprintf(f, "  as-list a%u members [ %u",
+			    lineNo, asne->asn);
+		} else {
 			fprintf(f," %u", asne->asn);
+		}
 
 		nc++;
 
@@ -1165,7 +1162,7 @@ bgpq4_print_eprefix(struct sx_radix_node *n, void *ff)
 
 	sx_prefix_snprintf(n->prefix, prefix, sizeof(prefix));
 
-	snprintf(seqno, sizeof(seqno), "seq %i", seq++);
+	snprintf(seqno, sizeof(seqno), " seq %i", seq++);
 
 	if (n->isAggregate) {
 		if (n->aggregateLow > n->prefix->masklen) {
@@ -1868,7 +1865,7 @@ bgpq4_print_k7prefix(struct sx_radix_node *n, void *ff)
 
 	if (n->isAggregate)
 		fprintf(f,"/routing filter rule add chain=\""
-		    "%s-%s\" rule=\"if (dst in %s && dst-len in %d-%d) {accept}\"\n",
+		    "%s-%s\"  rule=\"if (dst in %s && dst-len in %d-%d) {accept}\"\n",
 		    bname ? bname : "NN",
 		    n->prefix->family == AF_INET ? "V4" : "V6",
 		    prefix, n->aggregateLow, n->aggregateHi);

tests/reference/eos--4.txt

@@ -1,4 +1,4 @@
 no ip prefix-list NN
 ip prefix-list NN
-   seq 1 permit 192.31.196.0/24
-   seq 2 permit 192.175.48.0/24
+    seq 1 permit 192.31.196.0/24
+    seq 2 permit 192.175.48.0/24

tests/reference/eos--6.txt

@@ -1,4 +1,4 @@
 no ipv6 prefix-list NN
 ipv6 prefix-list NN
-   seq 1 permit 2001:4:112::/48
-   seq 2 permit 2620:4f:8000::/48
+    seq 1 permit 2001:4:112::/48
+    seq 2 permit 2620:4f:8000::/48