fix(config): let command-line options override the config file

`Opts::merge` copied every value present in the config file over the
parsed command line, so e.g. `-r 8600-8650` was silently replaced by the
config's `range` and the requested ports were never scanned (`-p` only
survived because `PortStrategy::pick` prefers `ports` over `range`).
The same happened to `-a`, `-b`, `-t`, `--tries`, `-g`, `--scripts`,
`--scan-order`, the trailing nmap command and every other merged option.
This contradicts the documented behaviour: "Any arguments passed
directly to the binary will override the default provided in this
config file".

`Opts::parse()` cannot tell a value typed by the user from a clap
default, so `Opts::read()` now parses through `ArgMatches` and records
the ids of the arguments whose `value_source` is `CommandLine` in a new
skipped `cli_args` field. `merge_required`/`merge_optional` leave those
fields alone; options not given on the command line are still filled in
from the config file. `--top` and `--no-config` behave as before.

Add pure parsing tests (no sockets) covering both directions.

Fixes #722

.github/dependabot.yml

@@ -3,15 +3,15 @@ updates:
 - package-ecosystem: "github-actions"
   directory: "/"
   schedule:
-    interval: "monthly"
+    interval: "daily"
 - package-ecosystem: "docker"
   directory: "/"
   schedule:
-    interval: "monthly"
+    interval: "daily"
 - package-ecosystem: cargo
   directory: "/"
   schedule:
-    interval: monthly
+    interval: daily
   open-pull-requests-limit: 10
   ignore:
   - dependency-name: anyhow

.github/scan-benchmark/macos_tcp_memory.py

@@ -1,57 +0,0 @@
-"""Provision TCP memory for repeated scans on a disposable macOS CI runner."""
-
-import ctypes
-import json
-import os
-import subprocess
-
-
-class Statistics(ctypes.Structure):
-    # XNU's bsd/sys/mem_acct_private.h, struct memacct_statistics.
-    _fields_ = [("peak", ctypes.c_uint64), ("allocated", ctypes.c_int64),
-                ("softlimit", ctypes.c_uint64), ("hardlimit", ctypes.c_uint64),
-                ("name", ctypes.c_char * 16)]
-
-
-def tcp_memory(new_limit: int | None = None) -> dict:
-    """Read the TCP accounting object, optionally raising its hard limit."""
-    libc = ctypes.CDLL(None, use_errno=True)
-    libc.sysctlnametomib.argtypes = [ctypes.c_char_p, ctypes.POINTER(ctypes.c_int),
-                                   ctypes.POINTER(ctypes.c_size_t)]
-    libc.sysctl.argtypes = [ctypes.POINTER(ctypes.c_int), ctypes.c_uint,
-                           ctypes.c_void_p, ctypes.POINTER(ctypes.c_size_t),
-                           ctypes.c_void_p, ctypes.c_size_t]
-    mib = (ctypes.c_int * 24)()
-    count = ctypes.c_size_t(22)
-    if libc.sysctlnametomib(b"kern.memacct", mib, ctypes.byref(count)):
-        raise OSError(ctypes.get_errno(), os.strerror(ctypes.get_errno()))
-
-    def query(operation: int, index: int, output, replacement=None):
-        mib[count.value], mib[count.value + 1] = operation, index
-        length = ctypes.c_size_t(ctypes.sizeof(output))
-        if libc.sysctl(mib, count.value + 2, ctypes.byref(output), ctypes.byref(length),
-                       ctypes.byref(replacement) if replacement is not None else None,
-                       ctypes.sizeof(replacement) if replacement is not None else 0):
-            raise OSError(ctypes.get_errno(), os.strerror(ctypes.get_errno()))
-        return length.value
-
-    names = ctypes.create_string_buffer(8 * 16)
-    length = query(5, 0, names)  # MEM_ACCT_SUBSYSTEMS
-    index = next(i for i in range(length // 16)
-                 if names.raw[i * 16:(i + 1) * 16].rstrip(b"\0") == b"TCP")
-    stats = Statistics()
-    query(6, index, stats)  # MEM_ACCT_ALL_SUBSYSTEM_STATISTICS
-    previous_limit = stats.hardlimit
-    if new_limit is not None and 0 < previous_limit < new_limit:
-        query(3, index, ctypes.c_uint64(), ctypes.c_uint64(new_limit))
-        query(6, index, stats)
-    return {"previous_hardlimit": previous_limit,
-            **{name: getattr(stats, name) for name in ("peak", "allocated", "softlimit", "hardlimit")}}
-
-
-if __name__ == "__main__":
-    # The default TCP budget is 1/32 of physical memory. Repeated 10,000-socket
-    # batches hit ENOBUFS on Darwin 25.6 even on master. Give both builds 1/8
-    # before timing; retain every listener check and performance threshold.
-    memory = int(subprocess.check_output(["sysctl", "-n", "hw.memsize"], text=True))
-    print(json.dumps(tcp_memory(memory // 8)))

.github/scan-benchmark/scan_bench.py

@@ -1,984 +0,0 @@
-#!/usr/bin/env python3
-"""Compare RustScan builds by scanning listeners on the loopback interface.
-
-CI-only tooling for ``.github/workflows/scan-benchmark.yml``; it is not part
-of the test suite. It port-scans 127.0.0.1, so only run it on a disposable CI
-runner, never on a developer machine or a shared host.
-
-``run`` starts ``serve`` (TCP listeners and UDP responders on 127.0.0.1) in a
-separate process, runs every scenario with every build (interleaving the
-builds so that drift on the runner affects them all alike), checks that all
-builds report the same open ports, and writes the raw measurements as JSON
-plus a Markdown summary of the medians. On Linux the workflow also sets up a
-host behind an artificial delay (a network namespace on the runner, with its
-own ``serve``), so that answers arrive after a round trip, as they do from a
-remote host, instead of while the probe is being sent.
-
-Measured per run:
-
-* scan time: RustScan's own "Portscan" timer (``RUST_LOG=rustscan=info``),
-  reported as throughput (sockets per second);
-* wall time of the whole process, and the time until the first ``Open`` line
-  appears on stdout (latency to the first result);
-* peak RSS and CPU time (user + system) of the process;
-* peak number of open file descriptors (Windows: handles) and of threads,
-  sampled every few milliseconds while the scan runs.
-"""
-
-from __future__ import annotations
-
-import argparse
-import json
-import os
-import platform
-import re
-import selectors
-import signal
-import socket
-import statistics
-import subprocess
-import sys
-import threading
-import time
-from dataclasses import asdict, dataclass, field
-
-LOOPBACK = "127.0.0.1"
-
-# Ports `serve` listens on. They stay below 32768, i.e. below the ephemeral
-# port ranges of Linux (32768-60999), macOS and Windows (49152-65535): a scan
-# of a port inside the ephemeral range can occasionally connect a socket to
-# itself and report a bogus open port.
-TCP_PORTS = tuple(range(1024, 32768, 101))  # 315 open TCP ports
-UDP_PORTS = tuple(range(1031, 32768, 997))  # 32 open UDP ports
-SWEEP = (1, 32767)
-
-SAMPLE_INTERVAL = 0.005  # seconds between fd/thread samples
-RUN_TIMEOUT = 300  # seconds before a run is killed and counted as failed
-
-
-# --------------------------------------------------------------------------
-# Listener process
-# --------------------------------------------------------------------------
-
-
-def serve(addresses: list[str]) -> int:
-    """Accepts (and closes) TCP connections and answers every UDP datagram."""
-    selector = selectors.DefaultSelector()
-    bound: dict[str, dict[str, list[int]]] = {"tcp": {}, "udp": {}, "busy": {}}
-
-    for address in addresses:
-        for kind, ports in (("tcp", TCP_PORTS), ("udp", UDP_PORTS)):
-            sock_type = socket.SOCK_STREAM if kind == "tcp" else socket.SOCK_DGRAM
-            for port in ports:
-                sock = socket.socket(socket.AF_INET, sock_type)
-                if os.name != "nt":
-                    sock.setsockopt(socket.SOL_SOCKET, socket.SO_REUSEADDR, 1)
-                try:
-                    sock.bind((address, port))
-                except OSError:
-                    sock.close()
-                    bound["busy"].setdefault(f"{kind} {address}", []).append(port)
-                    continue
-                if kind == "tcp":
-                    sock.listen(1024)
-                sock.setblocking(False)
-                selector.register(sock, selectors.EVENT_READ, kind)
-                bound[kind].setdefault(address, []).append(port)
-
-    print(json.dumps(bound), flush=True)
-
-    while True:
-        for key, _ in selector.select(timeout=1.0):
-            sock = key.fileobj
-            if key.data == "tcp":
-                while True:
-                    try:
-                        conn, _ = sock.accept()
-                    except OSError:  # BlockingIOError once the queue is empty
-                        break
-                    conn.close()
-            else:
-                while True:
-                    try:
-                        _, peer = sock.recvfrom(4096)
-                    except ConnectionResetError:
-                        # Windows reports an earlier ICMP error here; skip it.
-                        continue
-                    except OSError:  # BlockingIOError once the queue is empty
-                        break
-                    try:
-                        sock.sendto(b"rustscan-benchmark\n", peer)
-                    except OSError:
-                        pass
-
-
-def start_server(addresses: list[str]) -> tuple[subprocess.Popen, dict]:
-    args = [arg for address in addresses for arg in ("--address", address)]
-    proc = subprocess.Popen(
-        [sys.executable, os.path.abspath(__file__), "serve", *args],
-        stdout=subprocess.PIPE,
-        stdin=subprocess.DEVNULL,
-        text=True,
-    )
-    line = proc.stdout.readline()
-    if not line:
-        raise SystemExit("the listener process exited before it was ready")
-    return proc, json.loads(line)
-
-
-# --------------------------------------------------------------------------
-# Scenarios
-# --------------------------------------------------------------------------
-
-
-@dataclass(frozen=True)
-class Scenario:
-    name: str
-    title: str
-    addresses: tuple[str, ...]
-    ranges: tuple[tuple[int, int], ...] = ()
-    ports: tuple[int, ...] = ()
-    batch: int = 4500
-    timeout_ms: int = 1500
-    udp: bool = False
-    repeat: int = 1  # extra samples for very short scenarios
-    excluded_ports: tuple[int, ...] = ()
-
-    @property
-    def sockets(self) -> int:
-        span = sum(end - start + 1 for start, end in self.ranges)
-        excluded = sum(self.covers(port) for port in set(self.excluded_ports))
-        return len(self.addresses) * (len(self.ports) + span - excluded)
-
-    def covers(self, port: int) -> bool:
-        return port in self.ports or any(s <= port <= e for s, e in self.ranges)
-
-    def args(self) -> list[str]:
-        args = ["-a", ",".join(self.addresses)]
-        if self.ports:
-            args += ["-p", ",".join(map(str, self.ports))]
-        else:
-            args += ["-r", ",".join(f"{s}-{e}" for s, e in self.ranges)]
-        args += ["-b", str(self.batch), "-t", str(self.timeout_ms)]
-        if self.udp:
-            args.append("--udp")
-        if self.excluded_ports:
-            args += ["--exclude-ports", ",".join(map(str, self.excluded_ports))]
-        return args
-
-    def expected(self, listeners: dict) -> set[str]:
-        by_address = listeners["udp" if self.udp else "tcp"]
-        return {
-            f"{address}:{port}"
-            for address in self.addresses
-            for port in by_address.get(address, [])
-            if self.covers(port) and port not in self.excluded_ports
-        }
-
-    def describe(self) -> str:
-        hosts = f"{len(self.addresses)} addresses" if len(self.addresses) > 1 else self.addresses[0]
-        return f"{self.title}: `{' '.join(self.args()[2:])}` on {hosts}"
-
-
-def build_scenarios(
-    system: str, filtered: tuple[int, int] | None, delayed: str | None, delay_ms: int
-) -> list[Scenario]:
-    lo = (LOOPBACK,)
-    if system == "Windows":
-        # Windows retries a refused connection for about a second, so every
-        # closed port there takes as long as a filtered one: sweep fewer.
-        sweep, small_sweep = ((1, 8191),), ((1, 2047),)
-    else:
-        sweep = small_sweep = (SWEEP,)
-
-    def ports(ranges: tuple[tuple[int, int], ...]) -> str:
-        return f"{sum(e - s + 1 for s, e in ranges):,} ports"
-
-    scenarios = [
-        Scenario("tcp-1-port", "TCP, 1 open port", lo, ports=(TCP_PORTS[0],), repeat=3),
-        Scenario("tcp-open", "TCP, open listeners only", lo, ports=TCP_PORTS, repeat=3),
-        Scenario("tcp-sweep", f"TCP, {ports(sweep)}, default batch", lo, sweep),
-        Scenario(
-            "tcp-sweep-excluded",
-            f"TCP, {ports(sweep)}, 1,024 excluded ports",
-            lo,
-            sweep,
-            excluded_ports=tuple(range(1024, 2048)),
-        ),
-        Scenario(
-            "tcp-sweep-b500", f"TCP, {ports(small_sweep)}, small batch", lo, small_sweep, batch=500
-        ),
-        Scenario("tcp-sweep-b10000", f"TCP, {ports(sweep)}, large batch", lo, sweep, batch=10000),
-    ]
-    if system == "Linux":
-        # Linux answers on all of 127.0.0.0/8; other systems only on 127.0.0.1.
-        hosts = tuple(f"127.0.0.{i}" for i in range(1, 9))
-        scenarios.append(
-            Scenario("tcp-8-hosts", "TCP, 8 hosts x 4,096 ports", hosts, ((1, 4096),))
-        )
-    if delayed:
-        scenarios.append(
-            Scenario("tcp-delayed", f"TCP, {ports(sweep)}, {delay_ms} ms away", (delayed,), sweep)
-        )
-    if filtered:
-        dropped = (filtered,)
-        scenarios += [
-            Scenario(
-                "tcp-filtered", "TCP, 2,000 filtered ports", lo, dropped, batch=500, timeout_ms=250
-            ),
-            Scenario(
-                "tcp-mixed",
-                f"TCP, {ports(sweep)} + 2,000 filtered ports",
-                lo,
-                sweep + dropped,
-                timeout_ms=500,
-            ),
-        ]
-    scenarios.append(
-        Scenario("udp-sweep", f"UDP, {ports(sweep)}", lo, sweep, timeout_ms=500, udp=True)
-    )
-    scenarios.append(
-        Scenario(
-            "udp-open", "UDP, echo responders only", lo, ports=UDP_PORTS,
-            timeout_ms=500, udp=True, repeat=9,
-        )
-    )
-    if delayed:
-        scenarios.append(
-            Scenario(
-                "udp-delayed",
-                f"UDP, {ports(sweep)}, {delay_ms} ms away",
-                (delayed,),
-                sweep,
-                timeout_ms=500,
-                udp=True,
-            )
-        )
-    if filtered:
-        scenarios.append(
-            Scenario(
-                "udp-filtered",
-                "UDP, 2,000 filtered ports",
-                lo,
-                (filtered,),
-                batch=500,
-                timeout_ms=250,
-                udp=True,
-            )
-        )
-    return scenarios
-
-
-# --------------------------------------------------------------------------
-# Running one scan
-# --------------------------------------------------------------------------
-
-
-@dataclass
-class RunResult:
-    build: str
-    scenario: str
-    sockets: int = 0
-    ok: bool = True
-    error: str = ""
-    returncode: int | None = None
-    wall_s: float | None = None
-    scan_s: float | None = None
-    first_open_s: float | None = None
-    rss_kb: int | None = None
-    cpu_s: float | None = None
-    peak_fds: int | None = None
-    peak_threads: int | None = None
-    open: list[str] = field(default_factory=list)
-
-    @property
-    def rate(self) -> float | None:
-        return self.sockets / self.scan_s if self.scan_s else None
-
-
-SCAN_TIME = re.compile(r"Portscan\s*\|\s*([0-9.]+)\s*s")
-OPEN_LINE = re.compile(r"^Open (\S+)$")
-SUMMARY_LINE = re.compile(r"^(\S+) -> \[([0-9,]*)\]$")
-
-
-class Sampler(threading.Thread):
-    """Samples the open descriptors and the threads of a running process."""
-
-    def __init__(self, proc: subprocess.Popen):
-        super().__init__(daemon=True)
-        self.proc = proc
-        self.done = threading.Event()
-        self.peak_fds: int | None = None
-        self.peak_threads: int | None = None
-        self.system = platform.system()
-        self.libproc = darwin_libproc() if self.system == "Darwin" else None
-
-    def sample(self) -> tuple[int | None, int | None]:
-        pid = self.proc.pid
-        if self.system == "Linux":
-            fds = len(os.listdir(f"/proc/{pid}/fd"))
-            threads = None
-            with open(f"/proc/{pid}/status", encoding="ascii", errors="replace") as status:
-                for line in status:
-                    if line.startswith("Threads:"):
-                        threads = int(line.split()[1])
-                        break
-            return fds, threads
-        if self.libproc is not None:
-            return darwin_fds_and_threads(self.libproc, pid)
-        if self.system == "Windows":
-            return windows_handle_count(self.proc), None
-        return None, None
-
-    def run(self) -> None:
-        while not self.done.is_set():
-            try:
-                fds, threads = self.sample()
-            except (OSError, ValueError):
-                return  # the process is gone
-            if fds is not None:
-                self.peak_fds = max(self.peak_fds or 0, fds)
-            if threads is not None:
-                self.peak_threads = max(self.peak_threads or 0, threads)
-            self.done.wait(SAMPLE_INTERVAL)
-
-
-def darwin_libproc():
-    import ctypes
-
-    try:
-        libproc = ctypes.CDLL("/usr/lib/libproc.dylib")
-    except OSError:
-        return None
-    libproc.proc_pidinfo.argtypes = [
-        ctypes.c_int,
-        ctypes.c_int,
-        ctypes.c_uint64,
-        ctypes.c_void_p,
-        ctypes.c_int,
-    ]
-    libproc.proc_pidinfo.restype = ctypes.c_int
-    return libproc
-
-
-def darwin_fds_and_threads(libproc, pid: int) -> tuple[int | None, int | None]:
-    import ctypes
-
-    proc_pidlistfds, proc_pidtaskinfo = 1, 4
-    size = libproc.proc_pidinfo(pid, proc_pidlistfds, 0, None, 0)
-    if size <= 0:
-        raise OSError("process is gone")
-    buf = ctypes.create_string_buffer(size)
-    used = libproc.proc_pidinfo(pid, proc_pidlistfds, 0, ctypes.addressof(buf), size)
-    if used <= 0:
-        raise OSError("process is gone")
-    fds = used // 8  # sizeof(struct proc_fdinfo)
-
-    class TaskInfo(ctypes.Structure):  # struct proc_taskinfo
-        _fields_ = [
-            *((name, ctypes.c_uint64) for name in ("vsize", "rsize", "user", "system", "tu", "ts")),
-            *(
-                (name, ctypes.c_int32)
-                for name in (
-                    "policy",
-                    "faults",
-                    "pageins",
-                    "cow_faults",
-                    "messages_sent",
-                    "messages_received",
-                    "syscalls_mach",
-                    "syscalls_unix",
-                    "csw",
-                    "threadnum",
-                    "numrunning",
-                    "priority",
-                )
-            ),
-        ]
-
-    info = TaskInfo()
-    got = libproc.proc_pidinfo(
-        pid, proc_pidtaskinfo, 0, ctypes.addressof(info), ctypes.sizeof(info)
-    )
-    return fds, (info.threadnum if got == ctypes.sizeof(info) else None)
-
-
-def windows_handle_count(proc: subprocess.Popen) -> int | None:
-    import ctypes
-    from ctypes import wintypes
-
-    count = wintypes.DWORD()
-    handle = wintypes.HANDLE(int(proc._handle))  # Popen keeps the handle open
-    if not ctypes.windll.kernel32.GetProcessHandleCount(handle, ctypes.byref(count)):
-        raise OSError("process is gone")
-    return count.value
-
-
-def windows_memory_and_cpu(proc: subprocess.Popen) -> tuple[int | None, float | None]:
-    import ctypes
-    from ctypes import wintypes
-
-    class Counters(ctypes.Structure):  # PROCESS_MEMORY_COUNTERS
-        _fields_ = [
-            ("cb", wintypes.DWORD),
-            ("PageFaultCount", wintypes.DWORD),
-            ("PeakWorkingSetSize", ctypes.c_size_t),
-            ("WorkingSetSize", ctypes.c_size_t),
-            ("QuotaPeakPagedPoolUsage", ctypes.c_size_t),
-            ("QuotaPagedPoolUsage", ctypes.c_size_t),
-            ("QuotaPeakNonPagedPoolUsage", ctypes.c_size_t),
-            ("QuotaNonPagedPoolUsage", ctypes.c_size_t),
-            ("PagefileUsage", ctypes.c_size_t),
-            ("PeakPagefileUsage", ctypes.c_size_t),
-        ]
-
-    handle = wintypes.HANDLE(int(proc._handle))
-    counters = Counters()
-    counters.cb = ctypes.sizeof(counters)
-    rss_kb = None
-    if ctypes.windll.psapi.GetProcessMemoryInfo(handle, ctypes.byref(counters), counters.cb):
-        rss_kb = counters.PeakWorkingSetSize // 1024
-
-    times = [wintypes.FILETIME() for _ in range(4)]  # creation, exit, kernel, user
-    cpu_s = None
-    if ctypes.windll.kernel32.GetProcessTimes(handle, *(ctypes.byref(t) for t in times)):
-        ticks = sum((t.dwHighDateTime << 32) | t.dwLowDateTime for t in times[2:])
-        cpu_s = ticks / 1e7  # 100 ns units
-    return rss_kb, cpu_s
-
-
-def run_once(name: str, binary: str, scenario: Scenario) -> RunResult:
-    result = RunResult(build=name, scenario=scenario.name, sockets=scenario.sockets)
-    cmd = [
-        binary,
-        *scenario.args(),
-        "--scripts",
-        "none",
-        "--accessible",
-        "--no-banner",
-        "--no-config",
-    ]
-    env = dict(os.environ, RUST_LOG="rustscan=info")
-    stdout_lines: list[str] = []
-    stderr_chunks: list[bytes] = []
-
-    start = time.perf_counter()
-    proc = subprocess.Popen(
-        cmd, stdout=subprocess.PIPE, stderr=subprocess.PIPE, stdin=subprocess.DEVNULL, env=env
-    )
-
-    def read_stdout() -> None:
-        for raw in proc.stdout:
-            line = raw.decode("utf-8", "replace").rstrip("\r\n")
-            if result.first_open_s is None and line.startswith("Open "):
-                result.first_open_s = time.perf_counter() - start
-            stdout_lines.append(line)
-
-    readers = [
-        threading.Thread(target=read_stdout, daemon=True),
-        threading.Thread(target=lambda: stderr_chunks.append(proc.stderr.read()), daemon=True),
-    ]
-    for reader in readers:
-        reader.start()
-    sampler = Sampler(proc)
-    sampler.start()
-    # On Unix the child is reaped with os.wait4 below, so the watchdog must
-    # not go through Popen (which would try to reap it as well).
-    kill = proc.kill if os.name == "nt" else lambda: os.kill(proc.pid, signal.SIGKILL)
-    watchdog = threading.Timer(RUN_TIMEOUT, kill)
-    watchdog.start()
-
-    try:
-        if hasattr(os, "wait4"):
-            _, status, usage = os.wait4(proc.pid, 0)
-            result.wall_s = time.perf_counter() - start
-            proc.returncode = os.waitstatus_to_exitcode(status)
-            maxrss = usage.ru_maxrss  # KiB on Linux, bytes on macOS
-            result.rss_kb = maxrss // 1024 if platform.system() == "Darwin" else maxrss
-            result.cpu_s = usage.ru_utime + usage.ru_stime
-        else:
-            proc.wait()
-            result.wall_s = time.perf_counter() - start
-            result.rss_kb, result.cpu_s = windows_memory_and_cpu(proc)
-    finally:
-        watchdog.cancel()
-        sampler.done.set()
-        sampler.join()
-        for reader in readers:
-            reader.join()
-
-    result.returncode = proc.returncode
-    result.peak_fds = sampler.peak_fds
-    result.peak_threads = sampler.peak_threads
-    stderr = b"".join(stderr_chunks).decode("utf-8", "replace")
-
-    opened: set[str] = set()
-    summarised: set[str] = set()
-    for line in stdout_lines:
-        if match := OPEN_LINE.match(line):
-            opened.add(match.group(1))
-        elif match := SUMMARY_LINE.match(line):
-            summarised.update(f"{match.group(1)}:{p}" for p in match.group(2).split(",") if p)
-    result.open = sorted(opened)
-
-    if scan_time := SCAN_TIME.search(stderr):
-        result.scan_s = float(scan_time.group(1))
-
-    problems = []
-    if proc.returncode != 0:
-        problems.append(f"exit code {proc.returncode}")
-    if result.scan_s is None:
-        problems.append("no 'Portscan' timer in the log")
-    if summarised != opened:
-        problems.append(f"'Open' lines and the summary disagree: {sorted(summarised ^ opened)[:5]}")
-    if problems:
-        result.ok = False
-        tail = "\n".join(stderr.strip().splitlines()[-15:])
-        result.error = "; ".join(problems) + (f"\nstderr tail:\n{tail}" if tail else "")
-    return result
-
-
-# --------------------------------------------------------------------------
-# Environment
-# --------------------------------------------------------------------------
-
-
-def sysctl(*names: str) -> list[str]:
-    return subprocess.run(
-        ["sysctl", "-n", *names], capture_output=True, text=True, check=True
-    ).stdout.split("\n")
-
-
-def ephemeral_range() -> tuple[int, int] | None:
-    system = platform.system()
-    try:
-        if system == "Linux":
-            with open("/proc/sys/net/ipv4/ip_local_port_range", encoding="ascii") as f:
-                low, high = map(int, f.read().split())
-            return low, high
-        if system == "Darwin":
-            first, last = sysctl("net.inet.ip.portrange.first", "net.inet.ip.portrange.last")[:2]
-            return int(first), int(last)
-    except (OSError, ValueError, subprocess.CalledProcessError):
-        return None
-    return (49152, 65535) if system == "Windows" else None
-
-
-def diagnose_tcp(binary: str, scenario: Scenario, listeners: dict) -> dict:
-    """Untimed failure capture on a disposable macOS runner, using debug logs."""
-    from macos_tcp_memory import tcp_memory
-
-    def states() -> dict[str, int]:
-        snapshot = subprocess.run(
-            ["netstat", "-an", "-p", "tcp"], capture_output=True, text=True, check=True
-        )
-        counts: dict[str, int] = {}
-        for line in snapshot.stdout.splitlines():
-            if line.startswith("tcp"):
-                state = line.split()[-1]
-                counts[state] = counts.get(state, 0) + 1
-        return counts
-
-    before = states()
-    memory_before = tcp_memory()
-    probe = subprocess.run(
-        [binary, *scenario.args(), "--scripts", "none", "--accessible", "--no-banner", "--no-config"],
-        env=dict(os.environ, RUST_LOG="rustscan=info,rustscan::scanner=debug"),
-        capture_output=True, text=True, check=True,
-    )
-    opened = {
-        match.group(1) for line in probe.stdout.splitlines()
-        if (match := OPEN_LINE.match(line))
-    }
-    return {
-        "before": before, "after": states(),
-        "memory_before": memory_before, "memory_after": tcp_memory(),
-        "missing": sorted(scenario.expected(listeners) - opened),
-        "errors": [line for line in probe.stderr.splitlines()
-                   if "Typical socket connection errors" in line],
-    }
-
-
-def environment() -> dict[str, object]:
-    system = platform.system()
-    info: dict[str, object] = {
-        "system": system,
-        "release": platform.release(),
-        "machine": platform.machine(),
-        "cpus": os.cpu_count(),
-        "cpu": platform.processor(),
-        "python": platform.python_version(),
-        "ephemeral_ports": ephemeral_range(),
-    }
-    try:
-        if system == "Linux":
-            with open("/proc/cpuinfo", encoding="ascii", errors="replace") as f:
-                for line in f:
-                    if line.lower().startswith(("model name", "cpu model")):
-                        info["cpu"] = line.split(":", 1)[1].strip()
-                        break
-        elif system == "Darwin":
-            info["cpu"] = sysctl("machdep.cpu.brand_string")[0].strip()
-    except (OSError, subprocess.CalledProcessError):
-        pass
-    try:
-        import resource
-
-        info["open_file_limit"] = resource.getrlimit(resource.RLIMIT_NOFILE)[0]
-    except (ImportError, OSError, ValueError):
-        pass
-    return info
-
-
-# --------------------------------------------------------------------------
-# Analysis and reporting
-# --------------------------------------------------------------------------
-
-Results = dict[str, dict[str, list[RunResult]]]
-
-# key, title, formatter, higher is better
-METRICS = (
-    ("rate", "Throughput (sockets/s, from RustScan's scan timer)", lambda v: f"{v:,.0f}", True),
-    ("wall_s", "Wall time of the whole process", lambda v: f"{v * 1000:,.0f} ms", False),
-    ("first_open_s", "Time to the first open port", lambda v: f"{v * 1000:,.1f} ms", False),
-    ("rss_kb", "Peak RSS", lambda v: f"{v / 1024:,.1f} MiB", False),
-    ("cpu_s", "CPU time (user + system)", lambda v: f"{v * 1000:,.0f} ms", False),
-    ("peak_fds", "Peak open descriptors (sampled; Windows: handles)", lambda v: f"{v:,.0f}", False),
-    ("peak_threads", "Peak threads (sampled)", lambda v: f"{v:,.0f}", False),
-)
-FLAG_WORSE = 10.0  # percent
-
-
-def metric_values(runs: list[RunResult], key: str) -> list[float]:
-    return [float(getattr(r, key)) for r in runs if r.ok and getattr(r, key) is not None]
-
-
-def median(values: list[float]) -> float | None:
-    return statistics.median(values) if values else None
-
-
-def worse_by(value: float, base: float, higher_is_better: bool) -> float:
-    """How much worse `value` is than `base`, in percent (negative: better)."""
-    pct = (value / base - 1.0) * 100.0
-    return -pct if higher_is_better else pct
-
-
-def check_open_ports(
-    results: Results, scenarios: list[Scenario], listeners: dict, baseline: str
-) -> tuple[list[str], list[str], list[str]]:
-    """Returns (failed runs, open-port mismatches, notes).
-
-    No build may miss more listeners than the baseline: none at all if the
-    baseline found every listener in every run, or barely more if it did not
-    either (a runner too slow for the scenario's timeout). Ports that are
-    open beyond the listeners (services already running on the runner) must
-    match what the baseline found: a build may not miss a port the baseline
-    found in every run, nor report one the baseline never found.
-    """
-    failures, mismatches, notes = [], [], []
-    for sc in scenarios:
-        expected = sc.expected(listeners)
-        per_build = results[sc.name]
-
-        def missed(runs: list[RunResult]) -> int:
-            return sum(len(expected - set(r.open)) for r in runs if r.ok)
-
-        base_missed = missed(per_build[baseline])
-        allowed = base_missed + max(3, base_missed // 4) if base_missed else 0
-        if base_missed:
-            notes.append(
-                f"`{sc.name}`: `{baseline}` itself missed {base_missed} of "
-                f"{len(expected) * len(per_build[baseline])} listener answers"
-            )
-
-        base_extra = [set(r.open) - expected for r in per_build[baseline] if r.ok]
-        always = set.intersection(*base_extra) if base_extra else set()
-        ever = set.union(*base_extra) if base_extra else set()
-        for name, runs in per_build.items():
-            for index, res in enumerate(runs, start=1):
-                if not res.ok:
-                    failures.append(f"`{sc.name}` `{name}` run {index}: {res.error.splitlines()[0]}")
-            if name == baseline:
-                continue
-            if (count := missed(runs)) > allowed:
-                mismatches.append(
-                    f"`{sc.name}` `{name}`: missed {count} listener answers "
-                    f"(`{baseline}`: {base_missed})"
-                )
-            for index, res in enumerate(runs, start=1):
-                if not res.ok:
-                    continue
-                found = set(res.open)
-                where = f"`{sc.name}` `{name}` run {index}"
-                if lost := always - found:
-                    mismatches.append(f"{where}: missed ports `{baseline}` always found {sorted(lost)[:5]}")
-                if bogus := (found - expected) - ever:
-                    mismatches.append(f"{where}: reported ports `{baseline}` never found {sorted(bogus)[:5]}")
-    return failures, mismatches, notes
-
-
-def gate(
-    results: Results,
-    scenarios: list[Scenario],
-    baseline: str,
-    candidate: str,
-    max_slowdown: float,
-    max_memory: float,
-) -> list[str]:
-    """Scenarios where `candidate` is clearly worse than `baseline`."""
-    problems = []
-    for sc in scenarios:
-        checks = [("wall_s", False, max_slowdown, 0.002), ("rss_kb", False, max_memory, 1024)]
-        if sc.sockets >= 1000:  # throughput is meaningless for a handful of sockets
-            checks.append(("rate", True, max_slowdown, 0))
-        for key, higher, limit, floor in checks:
-            base = median(metric_values(results[sc.name][baseline], key))
-            value = median(metric_values(results[sc.name][candidate], key))
-            if not base or value is None or abs(value - base) <= floor:
-                continue
-            if (worse := worse_by(value, base, higher)) > limit:
-                problems.append(f"{sc.name}: {key} {worse:+.1f}% worse than {baseline} (limit {limit}%)")
-    return problems
-
-
-def render_markdown(report: dict, results: Results, scenarios: list[Scenario]) -> str:
-    builds = report["builds"]
-    baseline = report["baseline"]
-    env = report["environment"]
-    out = [f"### Scan benchmark: {env['system']} {env['machine']}", ""]
-    out.append(
-        "; ".join(f"`{b['name']}`: {b['label']}" for b in builds)
-        + f". {report['runs']} interleaved runs per build and scenario (`tcp-1-port`: x3), medians."
-        f" Runner: {env.get('cpu') or '?'}, {env['cpus']} CPUs, {env['system']} {env['release']},"
-        f" open-file limit {env.get('open_file_limit', 'n/a')}."
-    )
-    out.append("")
-
-    problems = report["failures"] + report["mismatches"]
-    tcp = sum(len(ports) for ports in report["listeners"]["tcp"].values())
-    udp = sum(len(ports) for ports in report["listeners"]["udp"].values())
-    if not problems:
-        out.append(
-            f"✅ Every run of every build exited cleanly; no build missed more of the {tcp} TCP "
-            f"and {udp} UDP listeners than `{baseline}`, or disagreed with it about other ports."
-        )
-    else:
-        out.append(f"❌ {len(report['failures'])} failed runs, {len(report['mismatches'])} open-port mismatches:")
-        out += [f"- {p}" for p in problems[:25]]
-    if report["notes"]:
-        out.append("")
-        out += [f"- ⚠️ {note}" for note in report["notes"]]
-    out.append("")
-    out.append("| Scenario | What is scanned |")
-    out.append("|---|---|")
-    out += [f"| `{sc.name}` | {sc.describe()} ({sc.sockets:,} sockets) |" for sc in scenarios]
-    out.append("")
-    out.append(f"Changes are relative to `{baseline}`; ❗ marks a median more than {FLAG_WORSE:.0f}% worse.")
-    out.append("")
-
-    for key, title, fmt, higher in METRICS:
-        rows = []
-        for sc in scenarios:
-            base_value = median(metric_values(results[sc.name][baseline], key))
-            cells = []
-            for build in builds:
-                value = median(metric_values(results[sc.name][build["name"]], key))
-                if value is None:
-                    cells.append("–")
-                    continue
-                cell = fmt(value)
-                if build["name"] != baseline and base_value:
-                    pct = (value / base_value - 1.0) * 100.0
-                    flag = " ❗" if worse_by(value, base_value, higher) > FLAG_WORSE else ""
-                    cell += f" ({pct:+.1f}%{flag})"
-                cells.append(cell)
-            if any(c != "–" for c in cells):
-                rows.append(f"| `{sc.name}` | " + " | ".join(cells) + " |")
-        out.append(f"**{title}**")
-        out.append("")
-        if rows:
-            out.append("| Scenario | " + " | ".join(f"`{b['name']}`" for b in builds) + " |")
-            out.append("|---|" + "---:|" * len(builds))
-            out += rows
-        else:
-            out.append("_Not measured on this platform._")
-        out.append("")
-    return "\n".join(out)
-
-
-# --------------------------------------------------------------------------
-# Main
-# --------------------------------------------------------------------------
-
-
-def parse_pairs(values: list[str], option: str) -> dict[str, str]:
-    pairs = {}
-    for value in values:
-        name, sep, rest = value.partition("=")
-        if not sep or not name or not rest:
-            raise SystemExit(f"--{option} expects NAME=VALUE, got {value!r}")
-        pairs[name] = rest
-    return pairs
-
-
-def run(args: argparse.Namespace) -> int:
-    builds = parse_pairs(args.build, "build")
-    labels = parse_pairs(args.label, "label")
-    baseline = args.baseline or next(iter(builds))
-    if baseline not in builds:
-        raise SystemExit(f"unknown baseline build {baseline!r}")
-    for name, path in builds.items():
-        if not os.path.isfile(path):
-            raise SystemExit(f"build {name!r}: {path} does not exist")
-    if args.gate and args.gate not in builds:
-        raise SystemExit(f"unknown --gate build {args.gate!r}")
-    if args.delayed_host and not args.delayed_listeners:
-        raise SystemExit("--delayed-host needs --delayed-listeners")
-
-    filtered = None
-    if args.filtered:
-        low, _, high = args.filtered.partition("-")
-        filtered = (int(low), int(high))
-
-    scenarios = build_scenarios(platform.system(), filtered, args.delayed_host, args.delay_ms)
-    if args.only:
-        wanted = set(args.only.split(","))
-        scenarios = [sc for sc in scenarios if sc.name in wanted]
-
-    if ephemeral := ephemeral_range():
-        low, high = ephemeral
-        for sc in scenarios:
-            for start, end in sc.ranges:
-                if start <= high and end >= low:
-                    raise SystemExit(
-                        f"scenario {sc.name} scans {start}-{end}, which overlaps the ephemeral "
-                        f"port range {low}-{high}: a scan could connect a socket to itself"
-                    )
-
-    server, listeners = start_server([LOOPBACK])
-    if args.delayed_host:
-        # The workflow runs a second `serve` on the delayed host (inside a
-        # network namespace) and stores its first line in this file.
-        with open(args.delayed_listeners, encoding="utf-8") as f:
-            far = json.loads(f.readline())
-        for kind in ("tcp", "udp"):
-            listeners[kind][args.delayed_host] = far[kind].get(args.delayed_host, [])
-    for kind in ("tcp", "udp"):
-        counts = {address: len(ports) for address, ports in listeners[kind].items()}
-        print(f"{kind} listeners: {counts}", flush=True)
-    if listeners["busy"]:
-        print(f"already in use: {listeners['busy']}", flush=True)
-
-    names = list(builds)
-    results: Results = {sc.name: {name: [] for name in names} for sc in scenarios}
-    diagnostics = []
-    try:
-        warmup = next((sc for sc in scenarios if sc.name == "tcp-sweep"), scenarios[0])
-        for name in names:
-            run_once(name, builds[name], warmup)
-
-        rounds = args.runs * max(sc.repeat for sc in scenarios)
-        for index in range(rounds):
-            shift = index % len(names)
-            order = names[shift:] + names[:shift]
-            for sc in scenarios:
-                if index >= args.runs * sc.repeat:
-                    continue
-                for name in order:
-                    res = run_once(name, builds[name], sc)
-                    results[sc.name][name].append(res)
-                    print(
-                        f"[{index + 1}/{rounds}] {sc.name:<17} {name:<14} ok={res.ok} "
-                        f"scan={res.scan_s} wall={res.wall_s} first={res.first_open_s} "
-                        f"rss={res.rss_kb} cpu={res.cpu_s} fds={res.peak_fds} "
-                        f"threads={res.peak_threads} open={len(res.open)}",
-                        flush=True,
-                    )
-                    if not res.ok:
-                        print(f"    {res.error}", flush=True)
-                if platform.system() == "Darwin" and not diagnostics and any(
-                    sc.expected(listeners) - set(results[sc.name][name][-1].open) for name in names
-                ):
-                    # A failed pair cannot support a performance claim. Capture
-                    # its OS errors immediately, outside both measured scans.
-                    for name in names:
-                        record = {"build": name, "scenario": sc.name,
-                                  **diagnose_tcp(builds[name], sc, listeners)}
-                        diagnostics.append(record)
-                        print("TCP diagnosis: " + json.dumps(record), flush=True)
-    finally:
-        server.kill()
-        server.wait()
-
-    failures, mismatches, notes = check_open_ports(results, scenarios, listeners, baseline)
-    report = {
-        "environment": environment(),
-        "baseline": baseline,
-        "builds": [{"name": n, "label": labels.get(n, n), "path": builds[n]} for n in names],
-        "runs": args.runs,
-        "listeners": listeners,
-        "scenarios": [asdict(sc) | {"sockets": sc.sockets} for sc in scenarios],
-        "results": {
-            sc: {n: [asdict(r) | {"rate": r.rate} for r in runs] for n, runs in per.items()}
-            for sc, per in results.items()
-        },
-        "failures": failures,
-        "mismatches": mismatches,
-        "notes": notes,
-        "diagnostics": diagnostics,
-    }
-    markdown = render_markdown(report, results, scenarios)
-    if args.json:
-        with open(args.json, "w", encoding="utf-8") as f:
-            json.dump(report, f, indent=1)
-    if args.markdown:
-        with open(args.markdown, "w", encoding="utf-8") as f:
-            f.write(markdown + "\n")
-    print(markdown)
-
-    status = 1 if failures or mismatches else 0
-    if args.gate:
-        for problem in gate(results, scenarios, baseline, args.gate, args.max_slowdown, args.max_memory):
-            print(f"::error::performance regression: {problem}", flush=True)
-            status = 1
-    return status
-
-
-def main() -> int:
-    parser = argparse.ArgumentParser(
-        description=__doc__, formatter_class=argparse.RawDescriptionHelpFormatter
-    )
-    sub = parser.add_subparsers(dest="command", required=True)
-    listen = sub.add_parser("serve", help="run the loopback listeners (started by `run`)")
-    listen.add_argument("--address", action="append", default=[], help="address to listen on")
-    bench = sub.add_parser("run", help="benchmark the given builds")
-    bench.add_argument("--build", action="append", required=True, metavar="NAME=PATH")
-    bench.add_argument("--label", action="append", default=[], metavar="NAME=TEXT")
-    bench.add_argument("--baseline", help="build the others are compared with (default: the first)")
-    bench.add_argument("--runs", type=int, default=9, help="interleaved runs per build and scenario")
-    bench.add_argument(
-        "--filtered", metavar="START-END", help="ports the runner drops; enables filtered scenarios"
-    )
-    bench.add_argument(
-        "--delayed-host",
-        metavar="ADDRESS",
-        help="address behind an artificial delay, with its own `serve`; enables delayed scenarios",
-    )
-    bench.add_argument(
-        "--delayed-listeners", metavar="FILE", help="first output line of that host's `serve`"
-    )
-    bench.add_argument("--delay-ms", type=int, default=5, help="that delay, for the summary")
-    bench.add_argument("--only", metavar="NAMES", help="comma-separated scenarios to run")
-    bench.add_argument("--json", help="write the raw results to this file")
-    bench.add_argument("--markdown", help="write the Markdown summary to this file")
-    bench.add_argument("--gate", metavar="NAME", help="fail if this build regresses against the baseline")
-    bench.add_argument("--max-slowdown", type=float, default=15.0, help="percent, throughput and wall time")
-    bench.add_argument("--max-memory", type=float, default=25.0, help="percent, peak RSS")
-    args = parser.parse_args()
-    # The summary contains emoji; Windows runners default to a legacy code page.
-    sys.stdout.reconfigure(encoding="utf-8", errors="replace")
-    return serve(args.address or [LOOPBACK]) if args.command == "serve" else run(args)
-
-
-if __name__ == "__main__":
-    sys.exit(main())

.github/workflows/scan-benchmark.yml

@@ -1,256 +0,0 @@
-name: Scan benchmark
-
-# Compares the scanning speed and resource use of a change with its base.
-#
-# Each run builds both versions in release mode and runs
-# .github/scan-benchmark/scan_bench.py, which scans listeners it opens on
-# 127.0.0.1 inside the runner (plus closed and, on Linux, filtered ports) and
-# reports medians over interleaved runs. Nothing outside the runner is
-# scanned. The unit tests deliberately never open sockets; this workflow is
-# where scanning behaviour and performance are measured.
-#
-# It only runs for pull requests that touch the scanner, its runtime or its
-# dependencies, or when started by hand, so it does not slow down normal CI.
-
-on:
-  pull_request:
-    branches: [master]
-    paths:
-      - "src/scanner/**"
-      - "src/main.rs"
-      - "src/lib.rs"
-      - "Cargo.toml"
-      - "Cargo.lock"
-      - ".github/scan-benchmark/**"
-      - ".github/workflows/scan-benchmark.yml"
-  workflow_dispatch:
-    inputs:
-      base:
-        description: "Branch, tag or commit to compare against"
-        default: "master"
-      runs:
-        description: "Interleaved runs per build and scenario"
-        default: "9"
-      extra:
-        description: "More builds to compare, as space-separated NAME=REF pairs"
-        default: ""
-
-permissions:
-  contents: read
-
-concurrency:
-  group: scan-benchmark-${{ github.ref }}
-  cancel-in-progress: true
-
-jobs:
-  benchmark:
-    name: Benchmark (${{ matrix.os }})
-    runs-on: ${{ matrix.os }}
-    timeout-minutes: 90
-    strategy:
-      fail-fast: false
-      matrix:
-        os: [ubuntu-latest, ubuntu-24.04-arm, macos-latest, windows-latest]
-    defaults:
-      run:
-        shell: bash
-    env:
-      # Ports the Linux runners drop, to measure filtered ports (timeouts).
-      # They lie above Linux's ephemeral port range (32768-60999).
-      FILTERED_PORTS: "61000-62999"
-      # The Linux runners' host behind a 5 ms delay (see "Set up a host 5 ms away").
-      DELAYED_HOST: "10.200.0.2"
-      RUNS: ${{ inputs.runs || '9' }}
-      EXTRA: ${{ inputs.extra }}
-    steps:
-      - name: Checkout sources
-        uses: actions/checkout@v7
-
-      - name: Setup Python
-        uses: actions/setup-python@v7
-        with:
-          python-version: "3.12"
-
-      - name: Check the benchmark script
-        run: python -m py_compile .github/scan-benchmark/scan_bench.py
-
-      - name: Setup Rust
-        uses: actions-rust-lang/setup-rust-toolchain@ecabd13d1c56bd1345c230e542e9144811ad706f # v2.0.0
-        with:
-          toolchain: stable
-          # build.rs formats the generated payload table with rustfmt.
-          components: rustfmt
-          # Caching is handled by Swatinem/rust-cache below.
-          cache: false
-          rustflags: ""
-
-      - uses: Swatinem/rust-cache@6323deb102c322ba6fcbdcafc7e3dddab59af2b6 # v2.9.2
-
-      - name: Build the change
-        env:
-          PR_NUMBER: ${{ github.event.pull_request.number }}
-          PR_HEAD: ${{ github.event.pull_request.head.sha }}
-        run: |
-          exe=""
-          if [ "$RUNNER_OS" = Windows ]; then exe=".exe"; fi
-          echo "EXE=$exe" >> "$GITHUB_ENV"
-          mkdir -p "$RUNNER_TEMP/bin"
-          cargo build --release --locked
-          cp "target/release/rustscan$exe" "$RUNNER_TEMP/bin/head$exe"
-          if [ -n "$PR_NUMBER" ]; then
-            label="#$PR_NUMBER at ${PR_HEAD::7}, merged into its base"
-          else
-            label="$(git log -1 --format='%h %s' HEAD)"
-          fi
-          echo "HEAD_LABEL=$label" >> "$GITHUB_ENV"
-
-      - name: Build the base and any extra builds
-        env:
-          EVENT: ${{ github.event_name }}
-          PR_BASE: ${{ github.event.pull_request.base.sha }}
-          INPUT_BASE: ${{ inputs.base }}
-        run: |
-          build_ref() {
-            local name="$1" ref="$2"
-            if [[ ! "$name" =~ ^[A-Za-z0-9._+-]+$ ]]; then
-              echo "::error::invalid build name '$name'"
-              exit 1
-            fi
-            git fetch --no-tags --depth=1 origin "$ref"
-            git worktree add --detach "$RUNNER_TEMP/src-$name" FETCH_HEAD
-            cargo build --release --locked \
-              --manifest-path "$RUNNER_TEMP/src-$name/Cargo.toml" --target-dir target
-            cp "target/release/rustscan$EXE" "$RUNNER_TEMP/bin/$name$EXE"
-            echo "$name=$(git log -1 --format='%h %s' FETCH_HEAD)" >> "$RUNNER_TEMP/labels"
-          }
-
-          if [ "$EVENT" = pull_request ]; then base="$PR_BASE"; else base="${INPUT_BASE:-master}"; fi
-          build_ref base "$base"
-          for spec in $EXTRA; do
-            build_ref "${spec%%=*}" "${spec#*=}"
-          done
-          cat "$RUNNER_TEMP/labels"
-
-      - name: Drop traffic to the filtered ports
-        if: runner.os == 'Linux'
-        run: |
-          if ! command -v iptables >/dev/null; then
-            echo "::warning::iptables is not available; skipping the filtered-port scenarios"
-            exit 0
-          fi
-          for proto in tcp udp; do
-            sudo iptables -I INPUT -i lo -p "$proto" --dport "${FILTERED_PORTS/-/:}" -j DROP
-          done
-          sudo iptables -S INPUT
-          echo "FILTERED_OK=1" >> "$GITHUB_ENV"
-
-      - name: Set up a host 5 ms away
-        if: runner.os == 'Linux'
-        run: |
-          # A network namespace on the runner, reached over a veth pair whose
-          # outbound side delays every packet. Its answers (SYN-ACKs, resets,
-          # ICMP errors, UDP replies) arrive a round trip after the probe, as
-          # from a remote host, instead of while the probe is being sent.
-          skip() {
-            echo "::warning::$1; skipping the delayed-host scenarios"
-            exit 0
-          }
-          sudo ip netns add far || skip "cannot create a network namespace"
-          sudo ip link add rs-near type veth peer name rs-far || skip "cannot create a veth pair"
-          sudo ip link set rs-far netns far || skip "cannot move the veth end"
-          sudo ip addr add 10.200.0.1/30 dev rs-near || skip "cannot address the veth pair"
-          sudo ip link set rs-near up || skip "cannot bring up the veth pair"
-          sudo ip netns exec far ip addr add "$DELAYED_HOST/30" dev rs-far || skip "cannot address the namespace"
-          sudo ip netns exec far ip link set rs-far up || skip "cannot bring up the namespace"
-          sudo ip netns exec far ip link set lo up || true
-          # Answer every probe to a closed UDP port, like the loopback
-          # interface does (ICMP errors are rate limited everywhere else).
-          for setting in icmp_ratelimit=0 icmp_msgs_per_sec=100000000 icmp_msgs_burst=100000000; do
-            sudo ip netns exec far sysctl -qw "net.ipv4.$setting" || true
-          done
-          sudo tc qdisc add dev rs-near root netem delay 5ms limit 1000000 || skip "netem is not available"
-          tc qdisc show dev rs-near
-
-          sudo ip netns exec far "$(command -v python)" .github/scan-benchmark/scan_bench.py \
-            serve --address "$DELAYED_HOST" > "$RUNNER_TEMP/far.json" 2> "$RUNNER_TEMP/far.log" &
-          for _ in $(seq 100); do
-            if [ -s "$RUNNER_TEMP/far.json" ]; then break; fi
-            sleep 0.1
-          done
-          if [ ! -s "$RUNNER_TEMP/far.json" ]; then
-            cat "$RUNNER_TEMP/far.log" || true
-            skip "the delayed host's listeners did not start"
-          fi
-          head -c 200 "$RUNNER_TEMP/far.json"; echo
-          echo "DELAYED_OK=1" >> "$GITHUB_ENV"
-
-      - name: Provision macOS TCP memory for repeated socket batches
-        if: runner.os == 'macOS'
-        run: |
-          # XNU's TCP accounting hard limit otherwise makes both builds miss
-          # listeners with ENOBUFS during repeated sweeps. Apply one budget to
-          # both builds before timing; keep all input sizes and gates intact.
-          sudo "$(command -v python)" .github/scan-benchmark/macos_tcp_memory.py
-
-      - name: Describe the runner
-        continue-on-error: true
-        run: |
-          uname -a
-          if [ "$RUNNER_OS" = Linux ]; then
-            nproc
-            grep -m1 'model name' /proc/cpuinfo || true
-            sysctl net.ipv4.ip_local_port_range net.ipv4.tcp_tw_reuse net.core.somaxconn || true
-            ss -Hltnu | sort -k5 | head -n 50 || true
-          elif [ "$RUNNER_OS" = macOS ]; then
-            for oid in hw.ncpu machdep.cpu.brand_string kern.maxfilesperproc \
-              net.inet.ip.portrange.first net.inet.ip.portrange.last \
-              kern.ipc.maxsockbuf kern.ipc.nmbclusters; do
-              sysctl "$oid" || true
-            done
-          fi
-
-      - name: Run the benchmark
-        run: |
-          # Allow the largest batch size without RustScan shrinking it to fit
-          # the descriptor limit (macOS defaults to 256, and caps processes at
-          # kern.maxfilesperproc, 10240 on its runners).
-          if [ "$RUNNER_OS" != Windows ]; then
-            for limit in 65536 24576 10240; do
-              ulimit -n "$limit" 2>/dev/null && break
-            done
-            echo "open-file limit: $(ulimit -n)"
-          fi
-
-          # Fail on wrong results, or if the change is clearly slower or
-          # bigger than its base (see --max-slowdown/--max-memory).
-          args=(--runs "$RUNS" --gate head --json bench-results.json --markdown bench-results.md)
-          if [ -n "${FILTERED_OK:-}" ]; then args+=(--filtered "$FILTERED_PORTS"); fi
-          if [ -n "${DELAYED_OK:-}" ]; then
-            args+=(--delayed-host "$DELAYED_HOST" --delayed-listeners "$RUNNER_TEMP/far.json")
-          fi
-          args+=(--build "base=$RUNNER_TEMP/bin/base$EXE")
-          args+=(--build "head=$RUNNER_TEMP/bin/head$EXE" --label "head=$HEAD_LABEL")
-          while IFS= read -r line; do
-            name="${line%%=*}"
-            args+=(--label "$line")
-            if [ "$name" != base ]; then
-              args+=(--build "$name=$RUNNER_TEMP/bin/$name$EXE")
-            fi
-          done < "$RUNNER_TEMP/labels"
-
-          python .github/scan-benchmark/scan_bench.py run "${args[@]}"
-
-      - name: Publish the summary
-        if: always()
-        run: |
-          if [ -f bench-results.md ]; then cat bench-results.md >> "$GITHUB_STEP_SUMMARY"; fi
-
-      - name: Upload the results
-        if: always()
-        uses: actions/upload-artifact@v7
-        with:
-          name: scan-benchmark-${{ matrix.os }}
-          path: |
-            bench-results.json
-            bench-results.md
-          if-no-files-found: ignore

.github/workflows/test.yml

@@ -26,7 +26,7 @@ jobs:
         uses: actions/checkout@v7
 
       - name: Setup Rust
-        uses: actions-rust-lang/setup-rust-toolchain@ecabd13d1c56bd1345c230e542e9144811ad706f # v2.0.0
+        uses: actions-rust-lang/setup-rust-toolchain@166cdcfd11aee3cb47222f9ddb555ce30ddb9659 # v1.17.0
         with:
           toolchain: ${{ matrix.rust }}
           components: clippy, rustfmt

Cargo.lock

@@ -2,6 +2,21 @@
 # It is not intended for manual editing.
 version = 4
 
+[[package]]
+name = "addr2line"
+version = "0.24.2"
+source = "registry+https://github.com/rust-lang/crates.io-index"
+checksum = "dfbe277e56a376000877090da837660b4427aad530e3028d44e0bffe4f89a1c1"
+dependencies = [
+ "gimli",
+]
+
+[[package]]
+name = "adler2"
+version = "2.0.0"
+source = "registry+https://github.com/rust-lang/crates.io-index"
+checksum = "512761e0bb2578dd7380c6baaa0f4ce03e84f95e960231d1dec8bf4d7d6e2627"
+
 [[package]]
 name = "ahash"
 version = "0.8.11"
@@ -103,6 +118,119 @@ version = "1.0.96"
 source = "registry+https://github.com/rust-lang/crates.io-index"
 checksum = "6b964d184e89d9b6b67dd2715bc8e74cf3107fb2b529990c90cf517326150bf4"
 
+[[package]]
+name = "async-channel"
+version = "1.9.0"
+source = "registry+https://github.com/rust-lang/crates.io-index"
+checksum = "81953c529336010edd6d8e358f886d9581267795c61b19475b71314bffa46d35"
+dependencies = [
+ "concurrent-queue",
+ "event-listener 2.5.3",
+ "futures-core",
+]
+
+[[package]]
+name = "async-channel"
+version = "2.3.1"
+source = "registry+https://github.com/rust-lang/crates.io-index"
+checksum = "89b47800b0be77592da0afd425cc03468052844aff33b84e33cc696f64e77b6a"
+dependencies = [
+ "concurrent-queue",
+ "event-listener-strategy",
+ "futures-core",
+ "pin-project-lite",
+]
+
+[[package]]
+name = "async-executor"
+version = "1.13.1"
+source = "registry+https://github.com/rust-lang/crates.io-index"
+checksum = "30ca9a001c1e8ba5149f91a74362376cc6bc5b919d92d988668657bd570bdcec"
+dependencies = [
+ "async-task",
+ "concurrent-queue",
+ "fastrand",
+ "futures-lite",
+ "slab",
+]
+
+[[package]]
+name = "async-global-executor"
+version = "2.4.1"
+source = "registry+https://github.com/rust-lang/crates.io-index"
+checksum = "05b1b633a2115cd122d73b955eadd9916c18c8f510ec9cd1686404c60ad1c29c"
+dependencies = [
+ "async-channel 2.3.1",
+ "async-executor",
+ "async-io",
+ "async-lock",
+ "blocking",
+ "futures-lite",
+ "once_cell",
+]
+
+[[package]]
+name = "async-io"
+version = "2.4.0"
+source = "registry+https://github.com/rust-lang/crates.io-index"
+checksum = "43a2b323ccce0a1d90b449fd71f2a06ca7faa7c54c2751f06c9bd851fc061059"
+dependencies = [
+ "async-lock",
+ "cfg-if",
+ "concurrent-queue",
+ "futures-io",
+ "futures-lite",
+ "parking",
+ "polling",
+ "rustix 0.38.44",
+ "slab",
+ "tracing",
+ "windows-sys 0.59.0",
+]
+
+[[package]]
+name = "async-lock"
+version = "3.4.0"
+source = "registry+https://github.com/rust-lang/crates.io-index"
+checksum = "ff6e472cdea888a4bd64f342f09b3f50e1886d32afe8df3d663c01140b811b18"
+dependencies = [
+ "event-listener 5.4.0",
+ "event-listener-strategy",
+ "pin-project-lite",
+]
+
+[[package]]
+name = "async-std"
+version = "1.13.2"
+source = "registry+https://github.com/rust-lang/crates.io-index"
+checksum = "2c8e079a4ab67ae52b7403632e4618815d6db36d2a010cfe41b02c1b1578f93b"
+dependencies = [
+ "async-channel 1.9.0",
+ "async-global-executor",
+ "async-io",
+ "async-lock",
+ "crossbeam-utils",
+ "futures-channel",
+ "futures-core",
+ "futures-io",
+ "futures-lite",
+ "gloo-timers",
+ "kv-log-macro",
+ "log",
+ "memchr",
+ "once_cell",
+ "pin-project-lite",
+ "pin-utils",
+ "slab",
+ "wasm-bindgen-futures",
+]
+
+[[package]]
+name = "async-task"
+version = "4.7.1"
+source = "registry+https://github.com/rust-lang/crates.io-index"
+checksum = "8b75356056920673b02621b35afd0f7dda9306d03c79a30f5c56c44cf256e3de"
+
 [[package]]
 name = "async-trait"
 version = "0.1.86"
@@ -114,12 +242,33 @@ dependencies = [
  "syn 2.0.117",
 ]
 
+[[package]]
+name = "atomic-waker"
+version = "1.1.2"
+source = "registry+https://github.com/rust-lang/crates.io-index"
+checksum = "1505bd5d3d116872e7271a6d4e16d81d0c8570876c8de68093a09ac269d8aac0"
+
 [[package]]
 name = "autocfg"
 version = "1.4.0"
 source = "registry+https://github.com/rust-lang/crates.io-index"
 checksum = "ace50bade8e6234aa140d9a2f552bbee1db4d353f69b8217bc503490fc1a9f26"
 
+[[package]]
+name = "backtrace"
+version = "0.3.74"
+source = "registry+https://github.com/rust-lang/crates.io-index"
+checksum = "8d82cb332cdfaed17ae235a638438ac4d4839913cc2af585c3c6746e8f8bee1a"
+dependencies = [
+ "addr2line",
+ "cfg-if",
+ "libc",
+ "miniz_oxide",
+ "object",
+ "rustc-demangle",
+ "windows-targets 0.52.6",
+]
+
 [[package]]
 name = "base64"
 version = "0.21.7"
@@ -132,6 +281,19 @@ version = "2.8.0"
 source = "registry+https://github.com/rust-lang/crates.io-index"
 checksum = "8f68f53c83ab957f72c32642f3868eec03eb974d1fb82e453128456482613d36"
 
+[[package]]
+name = "blocking"
+version = "1.6.1"
+source = "registry+https://github.com/rust-lang/crates.io-index"
+checksum = "703f41c54fc768e63e091340b424302bb1c29ef4aa0c7f10fe849dfb114d29ea"
+dependencies = [
+ "async-channel 2.3.1",
+ "async-task",
+ "futures-io",
+ "futures-lite",
+ "piper",
+]
+
 [[package]]
 name = "bumpalo"
 version = "3.17.0"
@@ -279,7 +441,7 @@ version = "3.1.1"
 source = "registry+https://github.com/rust-lang/crates.io-index"
 checksum = "faf9468729b8cbcea668e36183cb69d317348c2e08e994829fb56ebfdfbaac34"
 dependencies = [
- "windows-sys 0.61.2",
+ "windows-sys 0.60.2",
 ]
 
 [[package]]
@@ -288,6 +450,15 @@ version = "0.3.2"
 source = "registry+https://github.com/rust-lang/crates.io-index"
 checksum = "ffb474a9c3219a8254ead020421ecf1b90427f29b55f6aae9a2471fa62c126ef"
 
+[[package]]
+name = "concurrent-queue"
+version = "2.5.0"
+source = "registry+https://github.com/rust-lang/crates.io-index"
+checksum = "4ca0197aee26d1ae37445ee532fefce43251d24cc7c166799f4d46817f1d3973"
+dependencies = [
+ "crossbeam-utils",
+]
+
 [[package]]
 name = "cpufeatures"
 version = "0.3.0"
@@ -387,7 +558,7 @@ dependencies = [
  "libc",
  "option-ext",
  "redox_users",
- "windows-sys 0.61.2",
+ "windows-sys 0.60.2",
 ]
 
 [[package]]
@@ -455,9 +626,42 @@ source = "registry+https://github.com/rust-lang/crates.io-index"
 checksum = "33d852cb9b869c2a9b3df2f71a3074817f01e1844f839a144f5fcef059a4eb5d"
 dependencies = [
  "libc",
- "windows-sys 0.52.0",
+ "windows-sys 0.59.0",
 ]
 
+[[package]]
+name = "event-listener"
+version = "2.5.3"
+source = "registry+https://github.com/rust-lang/crates.io-index"
+checksum = "0206175f82b8d6bf6652ff7d71a1e27fd2e4efde587fd368662814d6ec1d9ce0"
+
+[[package]]
+name = "event-listener"
+version = "5.4.0"
+source = "registry+https://github.com/rust-lang/crates.io-index"
+checksum = "3492acde4c3fc54c845eaab3eed8bd00c7a7d881f78bfc801e43a93dec1331ae"
+dependencies = [
+ "concurrent-queue",
+ "parking",
+ "pin-project-lite",
+]
+
+[[package]]
+name = "event-listener-strategy"
+version = "0.5.3"
+source = "registry+https://github.com/rust-lang/crates.io-index"
+checksum = "3c3e4e0dd3673c1139bf041f3008816d9cf2946bbfac2945c09e523b8d7b05b2"
+dependencies = [
+ "event-listener 5.4.0",
+ "pin-project-lite",
+]
+
+[[package]]
+name = "fastrand"
+version = "2.3.0"
+source = "registry+https://github.com/rust-lang/crates.io-index"
+checksum = "37909eebbb50d72f9059c3b6d82c0463f2ff062c9e95845c43a6c9c0355411be"
+
 [[package]]
 name = "foldhash"
 version = "0.1.5"
@@ -481,6 +685,7 @@ checksum = "65bc07b1a8bc7c85c5f2e110c476c7389b4554ba72af57d8445ea63a576b0876"
 dependencies = [
  "futures-channel",
  "futures-core",
+ "futures-executor",
  "futures-io",
  "futures-sink",
  "futures-task",
@@ -503,12 +708,47 @@ version = "0.3.31"
 source = "registry+https://github.com/rust-lang/crates.io-index"
 checksum = "05f29059c0c2090612e8d742178b0580d2dc940c837851ad723096f87af6663e"
 
+[[package]]
+name = "futures-executor"
+version = "0.3.31"
+source = "registry+https://github.com/rust-lang/crates.io-index"
+checksum = "1e28d1d997f585e54aebc3f97d39e72338912123a67330d723fdbb564d646c9f"
+dependencies = [
+ "futures-core",
+ "futures-task",
+ "futures-util",
+]
+
 [[package]]
 name = "futures-io"
 version = "0.3.31"
 source = "registry+https://github.com/rust-lang/crates.io-index"
 checksum = "9e5c1b78ca4aae1ac06c48a526a655760685149f0d465d21f37abfe57ce075c6"
 
+[[package]]
+name = "futures-lite"
+version = "2.6.0"
+source = "registry+https://github.com/rust-lang/crates.io-index"
+checksum = "f5edaec856126859abb19ed65f39e90fea3a9574b9707f13539acf4abf7eb532"
+dependencies = [
+ "fastrand",
+ "futures-core",
+ "futures-io",
+ "parking",
+ "pin-project-lite",
+]
+
+[[package]]
+name = "futures-macro"
+version = "0.3.31"
+source = "registry+https://github.com/rust-lang/crates.io-index"
+checksum = "162ee34ebcb7c64a8abebc059ce0fee27c2262618d7b60ed8faf72fef13c3650"
+dependencies = [
+ "proc-macro2",
+ "quote",
+ "syn 2.0.117",
+]
+
 [[package]]
 name = "futures-sink"
 version = "0.3.31"
@@ -530,6 +770,7 @@ dependencies = [
  "futures-channel",
  "futures-core",
  "futures-io",
+ "futures-macro",
  "futures-sink",
  "futures-task",
  "memchr",
@@ -569,6 +810,24 @@ dependencies = [
  "wasip3",
 ]
 
+[[package]]
+name = "gimli"
+version = "0.31.1"
+source = "registry+https://github.com/rust-lang/crates.io-index"
+checksum = "07e28edb80900c19c28f1072f2e8aeca7fa06b23cd4169cefe1af5aa3260783f"
+
+[[package]]
+name = "gloo-timers"
+version = "0.3.0"
+source = "registry+https://github.com/rust-lang/crates.io-index"
+checksum = "bbb143cf96099802033e0d4f4963b19fd2e0b728bcf076cd9cf7f6634f092994"
+dependencies = [
+ "futures-channel",
+ "futures-core",
+ "js-sys",
+ "wasm-bindgen",
+]
+
 [[package]]
 name = "half"
 version = "2.4.1"
@@ -615,6 +874,12 @@ version = "0.5.0"
 source = "registry+https://github.com/rust-lang/crates.io-index"
 checksum = "2304e00983f87ffb38b55b444b5e3b60a884b5d30c0fca7d82fe33449bbe55ea"
 
+[[package]]
+name = "hermit-abi"
+version = "0.4.0"
+source = "registry+https://github.com/rust-lang/crates.io-index"
+checksum = "fbf6a919d6cf397374f7dfeeea91d974c7c0a7221d0d0f4f20d859d329e53fcc"
+
 [[package]]
 name = "hickory-proto"
 version = "0.24.4"
@@ -849,7 +1114,7 @@ version = "0.3.2"
 source = "registry+https://github.com/rust-lang/crates.io-index"
 checksum = "b58db92f96b720de98181bbbe63c831e87005ab460c1bf306eb2622b4707997f"
 dependencies = [
- "socket2 0.5.8",
+ "socket2",
  "widestring",
  "windows-sys 0.48.0",
  "winreg",
@@ -925,6 +1190,15 @@ dependencies = [
  "wasm-bindgen",
 ]
 
+[[package]]
+name = "kv-log-macro"
+version = "1.0.7"
+source = "registry+https://github.com/rust-lang/crates.io-index"
+checksum = "0de8b303297635ad57c9f5059fd9cee7a47f8e8daa09df0fcd07dd39fb22977f"
+dependencies = [
+ "log",
+]
+
 [[package]]
 name = "leb128fmt"
 version = "0.1.0"
@@ -933,9 +1207,9 @@ checksum = "09edd9e8b54e49e587e4f6295a7d29c3ea94d469cb40ab8ca70b288248a81db2"
 
 [[package]]
 name = "libc"
-version = "0.2.191"
+version = "0.2.181"
 source = "registry+https://github.com/rust-lang/crates.io-index"
-checksum = "65cd87c9a6d60536580b316ad985726104730a5151c99845f2490428ef91240a"
+checksum = "459427e2af2b9c839b132acb702a1c654d95e10f8c326bfc2ad11310e458b1c5"
 
 [[package]]
 name = "libredox"
@@ -953,6 +1227,12 @@ version = "0.5.6"
 source = "registry+https://github.com/rust-lang/crates.io-index"
 checksum = "0717cef1bc8b636c6e1c1bbdefc09e6322da8a9321966e8928ef80d20f7f770f"
 
+[[package]]
+name = "linux-raw-sys"
+version = "0.4.15"
+source = "registry+https://github.com/rust-lang/crates.io-index"
+checksum = "d26c52dbd32dccf2d10cac7725f8eae5296885fb5703b261f7d0a0739ec807ab"
+
 [[package]]
 name = "linux-raw-sys"
 version = "0.11.0"
@@ -977,9 +1257,12 @@ dependencies = [
 
 [[package]]
 name = "log"
-version = "0.4.35"
+version = "0.4.34"
 source = "registry+https://github.com/rust-lang/crates.io-index"
-checksum = "5d15f86d12da44cd8d4ec3c73984537eb7f4cadcb9776a575a0a8a05ca11005c"
+checksum = "f9f8bd3e56ce4dfc153cf470fffbfa98c7620958b312ca5c3a4b8d5181fd13c6"
+dependencies = [
+ "value-bag",
+]
 
 [[package]]
 name = "lru-cache"
@@ -1002,6 +1285,15 @@ version = "2.7.4"
 source = "registry+https://github.com/rust-lang/crates.io-index"
 checksum = "78ca9ab1a0babb1e7d5695e3530886289c18cf2f87ec19a575a0abdce112e3a3"
 
+[[package]]
+name = "miniz_oxide"
+version = "0.8.5"
+source = "registry+https://github.com/rust-lang/crates.io-index"
+checksum = "8e3e04debbb59698c15bacbb6d93584a8c0ca9cc3213cb423d31f760d8843ce5"
+dependencies = [
+ "adler2",
+]
+
 [[package]]
 name = "mio"
 version = "1.0.3"
@@ -1041,6 +1333,15 @@ dependencies = [
  "autocfg",
 ]
 
+[[package]]
+name = "object"
+version = "0.36.7"
+source = "registry+https://github.com/rust-lang/crates.io-index"
+checksum = "62948e14d923ea95ea2c7c86c71013138b66525b86bdc08d2dcc262bdb497b87"
+dependencies = [
+ "memchr",
+]
+
 [[package]]
 name = "once_cell"
 version = "1.21.4"
@@ -1090,6 +1391,12 @@ dependencies = [
  "syn 2.0.117",
 ]
 
+[[package]]
+name = "parking"
+version = "2.2.1"
+source = "registry+https://github.com/rust-lang/crates.io-index"
+checksum = "f38d5652c16fde515bb1ecef450ab0f6a219d619a7274976324d5e377f7dceba"
+
 [[package]]
 name = "parking_lot"
 version = "0.12.3"
@@ -1131,6 +1438,17 @@ version = "0.1.0"
 source = "registry+https://github.com/rust-lang/crates.io-index"
 checksum = "8b870d8c151b6f2fb93e84a13146138f05d02ed11c7e7c54f8826aaaf7c9f184"
 
+[[package]]
+name = "piper"
+version = "0.2.4"
+source = "registry+https://github.com/rust-lang/crates.io-index"
+checksum = "96c8c490f422ef9a4efd2cb5b42b76c8613d7e7dfc1caf667b8a3350a5acc066"
+dependencies = [
+ "atomic-waker",
+ "fastrand",
+ "futures-io",
+]
+
 [[package]]
 name = "plotters"
 version = "0.3.7"
@@ -1159,6 +1477,21 @@ dependencies = [
  "plotters-backend",
 ]
 
+[[package]]
+name = "polling"
+version = "3.7.4"
+source = "registry+https://github.com/rust-lang/crates.io-index"
+checksum = "a604568c3202727d1507653cb121dbd627a58684eb09a820fd746bee38b4442f"
+dependencies = [
+ "cfg-if",
+ "concurrent-queue",
+ "hermit-abi",
+ "pin-project-lite",
+ "rustix 0.38.44",
+ "tracing",
+ "windows-sys 0.59.0",
+]
+
 [[package]]
 name = "portable-atomic"
 version = "1.11.0"
@@ -1372,6 +1705,25 @@ dependencies = [
  "libc",
 ]
 
+[[package]]
+name = "rustc-demangle"
+version = "0.1.24"
+source = "registry+https://github.com/rust-lang/crates.io-index"
+checksum = "719b953e2095829ee67db738b3bfa9fa368c94900df327b3f07fe6e794d2fe1f"
+
+[[package]]
+name = "rustix"
+version = "0.38.44"
+source = "registry+https://github.com/rust-lang/crates.io-index"
+checksum = "fdb5bc1ae2baa591800df16c9ca78619bf65c0488b41b96ccec5d11220d8c154"
+dependencies = [
+ "bitflags",
+ "errno",
+ "libc",
+ "linux-raw-sys 0.4.15",
+ "windows-sys 0.59.0",
+]
+
 [[package]]
 name = "rustix"
 version = "1.1.3"
@@ -1381,8 +1733,8 @@ dependencies = [
  "bitflags",
  "errno",
  "libc",
- "linux-raw-sys",
- "windows-sys 0.61.2",
+ "linux-raw-sys 0.11.0",
+ "windows-sys 0.60.2",
 ]
 
 [[package]]
@@ -1423,6 +1775,7 @@ dependencies = [
  "ansi_term",
  "anstream",
  "anyhow",
+ "async-std",
  "cidr-utils",
  "clap",
  "colored",
@@ -1434,7 +1787,6 @@ dependencies = [
  "gcd",
  "hickory-resolver",
  "itertools 0.15.0",
- "libc",
  "log",
  "once_cell",
  "parameterized",
@@ -1443,9 +1795,7 @@ dependencies = [
  "serde",
  "serde_derive",
  "text_placeholder",
- "tokio",
  "toml",
- "windows-sys 0.61.2",
 ]
 
 [[package]]
@@ -1535,9 +1885,9 @@ dependencies = [
 
 [[package]]
 name = "serde_spanned"
-version = "1.1.2"
+version = "1.1.1"
 source = "registry+https://github.com/rust-lang/crates.io-index"
-checksum = "4e7523beb55eece201a2356bee0bbca0d1ab466c14c07703b2e0ee6d42cb0c2c"
+checksum = "6662b5879511e06e8999a8a235d848113e942c9124f211511b16466ee2995f26"
 dependencies = [
  "serde_core",
 ]
@@ -1573,16 +1923,6 @@ dependencies = [
  "windows-sys 0.52.0",
 ]
 
-[[package]]
-name = "socket2"
-version = "0.6.5"
-source = "registry+https://github.com/rust-lang/crates.io-index"
-checksum = "c3d1e2c7f27f8d4cb10542a02c49005dbd6e93095799d6f3be745fae9f8fedd4"
-dependencies = [
- "libc",
- "windows-sys 0.61.2",
-]
-
 [[package]]
 name = "stable_deref_trait"
 version = "1.2.0"
@@ -1634,8 +1974,8 @@ version = "0.4.4"
 source = "registry+https://github.com/rust-lang/crates.io-index"
 checksum = "230a1b821ccbd75b185820a1f1ff7b14d21da1e442e22c0863ea5f08771a8874"
 dependencies = [
- "rustix",
- "windows-sys 0.61.2",
+ "rustix 1.1.3",
+ "windows-sys 0.60.2",
 ]
 
 [[package]]
@@ -1726,16 +2066,17 @@ checksum = "1f3ccbac311fea05f86f61904b462b55fb3df8837a366dfc601a0161d0532f20"
 
 [[package]]
 name = "tokio"
-version = "1.50.0"
+version = "1.44.2"
 source = "registry+https://github.com/rust-lang/crates.io-index"
-checksum = "27ad5e34374e03cfffefc301becb44e9dc3c17584f414349ebe29ed26661822d"
+checksum = "e6b88822cbe49de4185e3a4cbf8321dd487cf5fe0c5c65695fef6346371e9c48"
 dependencies = [
+ "backtrace",
  "bytes",
  "libc",
  "mio",
  "pin-project-lite",
- "socket2 0.6.5",
- "windows-sys 0.61.2",
+ "socket2",
+ "windows-sys 0.52.0",
 ]
 
 [[package]]
@@ -1750,9 +2091,9 @@ dependencies = [
 
 [[package]]
 name = "toml"
-version = "1.1.8+spec-1.1.0"
+version = "1.1.6+spec-1.1.0"
 source = "registry+https://github.com/rust-lang/crates.io-index"
-checksum = "20489e00e4d8741d6be680764cc12e270655e375a20d1011e844a9c3379e678d"
+checksum = "920602543f0911ab71da12c50d59701da54c196d1a2bf5cb4b75667f137a406a"
 dependencies = [
  "indexmap 2.13.0",
  "serde_core",
@@ -1765,27 +2106,27 @@ dependencies = [
 
 [[package]]
 name = "toml_datetime"
-version = "1.1.2+spec-1.1.0"
+version = "1.1.1+spec-1.1.0"
 source = "registry+https://github.com/rust-lang/crates.io-index"
-checksum = "2b86d767906c6c42421dcba507eb9d203e779497710a47782a224bb871653053"
+checksum = "3165f65f62e28e0115a00b2ebdd37eb6f3b641855f9d636d3cd4103767159ad7"
 dependencies = [
  "serde_core",
 ]
 
 [[package]]
 name = "toml_parser"
-version = "1.1.5+spec-1.1.0"
+version = "1.1.3+spec-1.1.0"
 source = "registry+https://github.com/rust-lang/crates.io-index"
-checksum = "baa693a8032d7e1cada7d0041e96126df243179ff061456783ac7f12bda4744c"
+checksum = "1d38ac1cf9b95face32296c0a3ede1fdc270627c9d9c02a7274dd6d960dc4d56"
 dependencies = [
  "winnow",
 ]
 
 [[package]]
 name = "toml_writer"
-version = "1.1.3+spec-1.1.0"
+version = "1.1.2+spec-1.1.0"
 source = "registry+https://github.com/rust-lang/crates.io-index"
-checksum = "06bdbd8cfc056b8d2e2e85f29b56a3bdbecb527cef81eb39e3e7b98af4652770"
+checksum = "7d56353a2a665ad0f41a421187180aab746c8c325620617ad883a99a1cbe66d2"
 
 [[package]]
 name = "tracing"
@@ -1865,6 +2206,12 @@ version = "0.2.2"
 source = "registry+https://github.com/rust-lang/crates.io-index"
 checksum = "06abde3611657adf66d383f00b093d7faecc7fa57071cce2578660c9f1010821"
 
+[[package]]
+name = "value-bag"
+version = "1.12.0"
+source = "registry+https://github.com/rust-lang/crates.io-index"
+checksum = "7ba6f5989077681266825251a52748b8c1d8a4ad098cc37e440103d0ea717fc0"
+
 [[package]]
 name = "version_check"
 version = "0.9.5"
@@ -1931,6 +2278,19 @@ dependencies = [
  "wasm-bindgen-shared",
 ]
 
+[[package]]
+name = "wasm-bindgen-futures"
+version = "0.4.50"
+source = "registry+https://github.com/rust-lang/crates.io-index"
+checksum = "555d470ec0bc3bb57890405e5d4322cc9ea83cebb085523ced7be4144dac1e61"
+dependencies = [
+ "cfg-if",
+ "js-sys",
+ "once_cell",
+ "wasm-bindgen",
+ "web-sys",
+]
+
 [[package]]
 name = "wasm-bindgen-macro"
 version = "0.2.100"
@@ -2035,7 +2395,7 @@ version = "0.1.9"
 source = "registry+https://github.com/rust-lang/crates.io-index"
 checksum = "cf221c93e13a30d793f7645a0e7762c55d169dbb0a49671918a2319d289b10bb"
 dependencies = [
- "windows-sys 0.48.0",
+ "windows-sys 0.59.0",
 ]
 
 [[package]]
@@ -2079,11 +2439,11 @@ dependencies = [
 
 [[package]]
 name = "windows-sys"
-version = "0.61.2"
+version = "0.60.2"
 source = "registry+https://github.com/rust-lang/crates.io-index"
-checksum = "ae137229bcbd6cdf0f7b80a31df61766145077ddf49416a728b02cb3921ff3fc"
+checksum = "f2f500e4d28234f72040990ec9d39e3a6b950f9f22d3dba18416c35882612bcb"
 dependencies = [
- "windows-link",
+ "windows-targets 0.53.5",
 ]
 
 [[package]]
@@ -2110,13 +2470,30 @@ dependencies = [
  "windows_aarch64_gnullvm 0.52.6",
  "windows_aarch64_msvc 0.52.6",
  "windows_i686_gnu 0.52.6",
- "windows_i686_gnullvm",
+ "windows_i686_gnullvm 0.52.6",
  "windows_i686_msvc 0.52.6",
  "windows_x86_64_gnu 0.52.6",
  "windows_x86_64_gnullvm 0.52.6",
  "windows_x86_64_msvc 0.52.6",
 ]
 
+[[package]]
+name = "windows-targets"
+version = "0.53.5"
+source = "registry+https://github.com/rust-lang/crates.io-index"
+checksum = "4945f9f551b88e0d65f3db0bc25c33b8acea4d9e41163edf90dcd0b19f9069f3"
+dependencies = [
+ "windows-link",
+ "windows_aarch64_gnullvm 0.53.1",
+ "windows_aarch64_msvc 0.53.1",
+ "windows_i686_gnu 0.53.1",
+ "windows_i686_gnullvm 0.53.1",
+ "windows_i686_msvc 0.53.1",
+ "windows_x86_64_gnu 0.53.1",
+ "windows_x86_64_gnullvm 0.53.1",
+ "windows_x86_64_msvc 0.53.1",
+]
+
 [[package]]
 name = "windows_aarch64_gnullvm"
 version = "0.48.5"
@@ -2129,6 +2506,12 @@ version = "0.52.6"
 source = "registry+https://github.com/rust-lang/crates.io-index"
 checksum = "32a4622180e7a0ec044bb555404c800bc9fd9ec262ec147edd5989ccd0c02cd3"
 
+[[package]]
+name = "windows_aarch64_gnullvm"
+version = "0.53.1"
+source = "registry+https://github.com/rust-lang/crates.io-index"
+checksum = "a9d8416fa8b42f5c947f8482c43e7d89e73a173cead56d044f6a56104a6d1b53"
+
 [[package]]
 name = "windows_aarch64_msvc"
 version = "0.48.5"
@@ -2141,6 +2524,12 @@ version = "0.52.6"
 source = "registry+https://github.com/rust-lang/crates.io-index"
 checksum = "09ec2a7bb152e2252b53fa7803150007879548bc709c039df7627cabbd05d469"
 
+[[package]]
+name = "windows_aarch64_msvc"
+version = "0.53.1"
+source = "registry+https://github.com/rust-lang/crates.io-index"
+checksum = "b9d782e804c2f632e395708e99a94275910eb9100b2114651e04744e9b125006"
+
 [[package]]
 name = "windows_i686_gnu"
 version = "0.48.5"
@@ -2153,12 +2542,24 @@ version = "0.52.6"
 source = "registry+https://github.com/rust-lang/crates.io-index"
 checksum = "8e9b5ad5ab802e97eb8e295ac6720e509ee4c243f69d781394014ebfe8bbfa0b"
 
+[[package]]
+name = "windows_i686_gnu"
+version = "0.53.1"
+source = "registry+https://github.com/rust-lang/crates.io-index"
+checksum = "960e6da069d81e09becb0ca57a65220ddff016ff2d6af6a223cf372a506593a3"
+
 [[package]]
 name = "windows_i686_gnullvm"
 version = "0.52.6"
 source = "registry+https://github.com/rust-lang/crates.io-index"
 checksum = "0eee52d38c090b3caa76c563b86c3a4bd71ef1a819287c19d586d7334ae8ed66"
 
+[[package]]
+name = "windows_i686_gnullvm"
+version = "0.53.1"
+source = "registry+https://github.com/rust-lang/crates.io-index"
+checksum = "fa7359d10048f68ab8b09fa71c3daccfb0e9b559aed648a8f95469c27057180c"
+
 [[package]]
 name = "windows_i686_msvc"
 version = "0.48.5"
@@ -2171,6 +2572,12 @@ version = "0.52.6"
 source = "registry+https://github.com/rust-lang/crates.io-index"
 checksum = "240948bc05c5e7c6dabba28bf89d89ffce3e303022809e73deaefe4f6ec56c66"
 
+[[package]]
+name = "windows_i686_msvc"
+version = "0.53.1"
+source = "registry+https://github.com/rust-lang/crates.io-index"
+checksum = "1e7ac75179f18232fe9c285163565a57ef8d3c89254a30685b57d83a38d326c2"
+
 [[package]]
 name = "windows_x86_64_gnu"
 version = "0.48.5"
@@ -2183,6 +2590,12 @@ version = "0.52.6"
 source = "registry+https://github.com/rust-lang/crates.io-index"
 checksum = "147a5c80aabfbf0c7d901cb5895d1de30ef2907eb21fbbab29ca94c5b08b1a78"
 
+[[package]]
+name = "windows_x86_64_gnu"
+version = "0.53.1"
+source = "registry+https://github.com/rust-lang/crates.io-index"
+checksum = "9c3842cdd74a865a8066ab39c8a7a473c0778a3f29370b5fd6b4b9aa7df4a499"
+
 [[package]]
 name = "windows_x86_64_gnullvm"
 version = "0.48.5"
@@ -2195,6 +2608,12 @@ version = "0.52.6"
 source = "registry+https://github.com/rust-lang/crates.io-index"
 checksum = "24d5b23dc417412679681396f2b49f3de8c1473deb516bd34410872eff51ed0d"
 
+[[package]]
+name = "windows_x86_64_gnullvm"
+version = "0.53.1"
+source = "registry+https://github.com/rust-lang/crates.io-index"
+checksum = "0ffa179e2d07eee8ad8f57493436566c7cc30ac536a3379fdf008f47f6bb7ae1"
+
 [[package]]
 name = "windows_x86_64_msvc"
 version = "0.48.5"
@@ -2207,6 +2626,12 @@ version = "0.52.6"
 source = "registry+https://github.com/rust-lang/crates.io-index"
 checksum = "589f6da84c646204747d1270a2a5661ea66ed1cced2631d546fdfb155959f9ec"
 
+[[package]]
+name = "windows_x86_64_msvc"
+version = "0.53.1"
+source = "registry+https://github.com/rust-lang/crates.io-index"
+checksum = "d6bbff5f0aada427a1e5a6da5f1f98158182f26556f345ac9e04d36d0ebed650"
+
 [[package]]
 name = "winnow"
 version = "1.0.0"

Cargo.toml

@@ -19,11 +19,8 @@ exclude = [
 [dependencies]
 clap = { version = "4.6.1", features = ["derive", "wrap_help"] }
 colored = "3.1.1"
-# Async runtime for the scanner. hickory-resolver already depends on tokio, so
-# this adds no crates; only the features the scanner itself uses are listed.
-tokio = { version = "1.44.2", features = ["rt", "net", "time"] }
-# Only `FuturesUnordered`/`StreamExt` are used; the executor is not needed.
-futures = { version = "0.3", default-features = false, features = ["std"] }
+async-std = "1.13.2"
+futures = "0.3"
 rlimit = "0.11.0"
 log = "0.4.32"
 env_logger = "0.11.10"
@@ -43,12 +40,6 @@ anyhow = "1.0.40"
 text_placeholder = { version = "0.5", features = ["struct_context"] }
 once_cell = "1.21.4"
 
-[target.'cfg(unix)'.dependencies]
-libc = "0.2"
-
-[target.'cfg(windows)'.dependencies]
-windows-sys = { version = "0.61", features = ["Win32_Networking_WinSock"] }
-
 [dev-dependencies]
 parameterized = "2.0.0"
 criterion = { version = "0.8", features = ["html_reports"] }

benches/benchmark_helpers.rs

@@ -2,7 +2,7 @@ use criterion::{criterion_group, criterion_main, Criterion};
 use rustscan::generated::get_parsed_data;
 use rustscan::input::{Opts, PortRanges, ScanOrder};
 use rustscan::port_strategy::PortStrategy;
-use rustscan::scanner::{build_udp_payload_lookup, Scanner};
+use rustscan::scanner::build_udp_payload_lookup;
 use std::collections::BTreeMap;
 use std::hint::black_box;
 use std::net::IpAddr;
@@ -60,50 +60,6 @@ fn criterion_benchmark(c: &mut Criterion) {
     });
     address_group.finish();
 
-    // Exercise production port preparation without opening a socket. The
-    // scanner has no target addresses, and runtime construction is not timed.
-    let runtime = tokio::runtime::Builder::new_current_thread()
-        .enable_all()
-        .build()
-        .unwrap();
-    let mut preparation = c.benchmark_group("port preparation");
-    preparation.sample_size(20);
-    preparation.warm_up_time(Duration::from_millis(500));
-    preparation.measurement_time(Duration::from_secs(2));
-    for (port_count, excluded_count) in [
-        (1, 1),
-        (16, 4),
-        (64, 1),
-        (64, 4096),
-        (4096, 0),
-        (4096, 64),
-        (4096, 4096),
-        (65535, 0),
-        (65535, 4),
-        (65535, 64),
-        (65535, 1024),
-        (65535, 4096),
-    ] {
-        let scanner = Scanner::new(
-            &[],
-            500,
-            Duration::from_millis(100),
-            1,
-            true,
-            PortStrategy::Manual((1..=port_count).collect()),
-            true,
-            (0..excluded_count)
-                .map(|i| (i * 13 % 65536) as u16)
-                .collect(),
-            false,
-        );
-        preparation.bench_function(
-            format!("{port_count} ports, {excluded_count} exclusions"),
-            |b| b.iter(|| black_box(runtime.block_on(black_box(&scanner).run_with_status()))),
-        );
-    }
-    preparation.finish();
-
     // UDP payload lookup micro-benchmark: compares the old linear scan of the
     // payload map with the precomputed port -> payload lookup. No sockets.
     let udp_map = get_parsed_data();

contributing.md

@@ -72,12 +72,3 @@ Automated tests and benchmarks must run without network traffic or external scri
 execution. Keep coverage focused on parsing, configuration, port ordering, socket
 address iteration, and timing helpers. Do not add tests that run TCP/UDP scans
 (including localhost), resolve hostnames, or launch Nmap or other scripts.
-
-Scanning behaviour and speed are measured separately, by the opt-in "Scan benchmark"
-GitHub Actions workflow (`.github/workflows/scan-benchmark.yml`). It runs for pull
-requests that touch the scanner, its runtime or its dependencies, and can be started
-by hand from the Actions tab. It builds the change and its base in release mode,
-scans listeners that it opens on 127.0.0.1 inside the runner, checks that both
-builds find the same open ports, and adds a comparison table to the job summary. Its
-script (`.github/scan-benchmark/scan_bench.py`) port-scans the loopback interface,
-so it is meant for disposable CI runners only.

docs/lazy-dns-benchmark-results.json

@@ -1,151 +0,0 @@
-{
-  "metadata": {
-    "base": "67c08a33100362c54b56fde784b0fd234f70fa11",
-    "candidate": "a05592a8a88b77ad85d961137943dfadfe98b612",
-    "environment": "Linux-7.2.6-1-cachyos-x86_64-with-glibc2.44",
-    "rust": "rustc 1.98.1 (48a229cea 2026-09-01) (Arch Linux rust 1:1.98.1-1.1)",
-    "hosts_bytes": 2760051,
-    "profile": "release with LTO",
-    "network_namespace": "unshare --user --map-root-user --net",
-    "common_args": [
-      "--no-config",
-      "--config-path",
-      "/dev/null",
-      "--no-banner",
-      "--scripts",
-      "none",
-      "--greppable",
-      "--ports",
-      "80",
-      "--exclude-ports",
-      "80"
-    ],
-    "workloads": {
-      "literal-ipv4": [
-        "-a",
-        "127.0.0.1"
-      ],
-      "mixed-cidrs": [
-        "-a",
-        "127.0.0.1,192.0.2.0/30,2001:db8::/126",
-        "--exclude-addresses",
-        "192.0.2.1,2001:db8::1"
-      ]
-    },
-    "warmups_each": 2,
-    "samples_each": 20,
-    "binary_cpus": "0-3",
-    "harness_cpus": "8-9",
-    "binary_sha256": {
-      "base": "4d6b090218ff36249ed95f14ec1a82626e8ea33854f3823f19a6de247b1524b8",
-      "dns": "a9d6bf949f56c53d18cc85bc9b53b21bd85a63bcdd04ad78992d392c675c77ac"
-    }
-  },
-  "summary": {
-    "literal-ipv4": {
-      "base_ms": 168.23068849999999,
-      "dns_ms": 2.248762,
-      "samples_each": 20
-    },
-    "mixed-cidrs": {
-      "base_ms": 178.0855925,
-      "dns_ms": 2.4539600000000004,
-      "samples_each": 20
-    }
-  },
-  "samples": {
-    "literal-ipv4": {
-      "base": [
-        143.391226,
-        146.562804,
-        149.615525,
-        151.188597,
-        152.269193,
-        159.499601,
-        164.926958,
-        164.347529,
-        167.208931,
-        165.433184,
-        172.246939,
-        169.252446,
-        172.777899,
-        170.729249,
-        170.092681,
-        171.863255,
-        169.612842,
-        170.857396,
-        173.117403,
-        175.206345
-      ],
-      "dns": [
-        2.241793,
-        1.817341,
-        2.39956,
-        2.048986,
-        3.123401,
-        1.820017,
-        3.615094,
-        2.165312,
-        2.675535,
-        2.103261,
-        2.531392,
-        2.215141,
-        2.572939,
-        2.063803,
-        2.671431,
-        2.059846,
-        2.598858,
-        2.230916,
-        2.678259,
-        2.255731
-      ]
-    },
-    "mixed-cidrs": {
-      "base": [
-        175.504767,
-        178.797842,
-        186.183728,
-        176.885958,
-        177.989816,
-        180.028568,
-        177.66299,
-        174.757221,
-        175.468485,
-        175.328023,
-        177.639639,
-        175.637593,
-        177.922435,
-        185.394396,
-        182.587204,
-        180.58311,
-        178.961137,
-        178.293186,
-        180.482419,
-        178.181369
-      ],
-      "dns": [
-        3.275193,
-        2.114074,
-        2.700771,
-        2.183317,
-        2.606538,
-        2.368046,
-        2.599702,
-        2.238105,
-        2.683892,
-        2.292249,
-        2.528366,
-        2.114478,
-        2.668921,
-        2.103271,
-        3.275514,
-        2.27974,
-        3.103311,
-        2.265298,
-        2.580743,
-        2.379554
-      ]
-    }
-  },
-  "invocations": 88
-}

docs/lazy-dns-benchmark.md

@@ -1,48 +0,0 @@
-# Lazy DNS initialization benchmark
-
-Compared current Tokio master `67c08a33100362c54b56fde784b0fd234f70fa11` with the lazy-resolver implementation
-at `a05592a8a88b77ad85d961137943dfadfe98b612` on 2026-10-02.
-
-## Method
-
-Both binaries are release builds with LTO, using Rust 1.98.1 on an Intel Core
-Ultra 7 165U running Linux 7.2.6-1-cachyos. Each workload used twenty measured
-runs and two warmups per binary, alternating A/B and B/A order. Binaries used
-CPUs 0-3; the harness used CPUs 8-9. No builds ran during measurement.
-
-The harness ran in a private network namespace. Every invocation excluded
-its only requested port, so no scan traffic was sent. Configuration and
-scripts were disabled. Timing spans process launch through exit, including
-the same `taskset` affinity launcher for both builds.
-
-Common arguments:
-
-```sh
---no-config --config-path /dev/null --no-banner --scripts none --greppable \
-  --ports 80 --exclude-ports 80
-```
-
-The literal case adds `-a 127.0.0.1`. The mixed case adds
-`-a 127.0.0.1,192.0.2.0/30,2001:db8::/126` and
-`--exclude-addresses 192.0.2.1,2001:db8::1`. Both use `RUST_LOG=rustscan=info`.
-
-## Results
-
-| Workload | Master median | Lazy DNS median | Time saved |
-| --- | ---: | ---: | ---: |
-| Literal IPv4 | 168.23 ms | 2.25 ms | 165.98 ms (98.66%) |
-| IPv4/IPv6 CIDRs and exclusions | 178.09 ms | 2.45 ms | 175.63 ms (98.62%) |
-
-All 88 invocations exited successfully with the expected empty scan output.
-A separate syscall trace of the optimized literal-IP invocation recorded no
-reads of `/etc/hosts` or `/etc/resolv.conf` and no network connects or sends.
-
-This host has a 2,760,051-byte hosts file. Eager Hickory initialization parses
-that file even for numeric targets. Systems with smaller hosts files will
-save less; this measures CLI startup, not socket throughput. Hostnames that
-need fallback DNS still pay resolver initialization once, and parsing order,
-file handling, deduplication and exclusions keep their existing behavior.
-
-[Raw samples, setup and binary hashes](lazy-dns-benchmark-results.json) are
-included. Network-free regression tests verify that literal IPv4/IPv6 targets,
-CIDRs, exclusions, duplicates and empty inputs do not construct the resolver.

docs/port-exclusion-benchmark.md

@@ -1,114 +0,0 @@
-# Port exclusion preparation benchmark
-
-Compared Tokio master `b7119f7f3eb1601e81126ef3572699ed7b3b7c0b` with
-implementation `270561d553688acbd99f61c460cdfa7c76c1423e` on 2026-10-02.
-
-## Method
-
-Release builds with LTO, Rust 1.98.1, Intel Core Ultra 7 165U, Linux
-7.2.6-1-cachyos. Both versions use the same lockfile and identical
-`benches/benchmark_helpers.rs`; copy the final benchmark harness into the
-baseline worktree before building it. The benchmark runs the production
-`Scanner::run_with_status` with **zero target addresses**, so it opens no
-sockets and does no DNS or scripting. It includes generating/cloning the
-port list, filtering and empty scan setup. Scanner construction and the
-current-thread Tokio runtime construction are outside the timer.
-
-Manual ports are `1..=port_count`; exclusions are
-`(i * 13 % 65536) as u16` for `i` in `0..excluded_count`. Each case has two
-independent twenty-sample Criterion runs per build, a 500 ms warmup and a
-2 s target measurement. Cases run in A/B then B/A order on CPU 0. No builds
-or other local checks ran during timing. CPU frequency was not fixed on
-this shared laptop, so the table retains both repetitions rather than
-selecting one result. Values are Criterion's mean point estimates.
-
-Build command for each worktree:
-
-```sh
-cargo bench --locked --bench benchmark_helpers --no-run
-```
-
-The saved benchmark executables are then run serially, for each case:
-
-```sh
-CARGO_TARGET_DIR=/tmp/preparation-results taskset -c 0 /path/to/benchmark_helpers \
-  --bench 'port preparation/65535 ports, 4096 exclusions' --save-baseline base-a
-```
-
-Repeat with the candidate binary as `head-a`, then the candidate as
-`head-b` and the baseline as `base-b`. To run the group normally without
-comparing builds:
-
-```sh
-cargo bench --locked --bench benchmark_helpers -- 'port preparation'
-```
-
-## Results
-
-Times are microseconds; A and B are separate matched repetitions.
-
-| Ports | Exclusions | Master µs A / B | Optimized µs A / B | Time saved A / B |
-| ---: | ---: | ---: | ---: | ---: |
-| 1 | 1 | 0.289 / 0.286 | 0.261 / 0.271 | 9.7% / 5.2% |
-| 16 | 4 | 0.260 / 0.268 | 0.232 / 0.237 | 10.6% / 11.6% |
-| 64 | 1 | 0.433 / 0.437 | 0.237 / 0.239 | 45.2% / 45.4% |
-| 64 | 4,096 | 6.231 / 10.093 | 2.484 / 3.038 | 60.1% / 69.9% |
-| 4,096 | 0 | 10.889 / 10.037 | 0.335 / 0.397 | 96.9% / 96.0% |
-| 4,096 | 64 | 20.680 / 20.363 | 2.930 / 3.064 | 85.8% / 85.0% |
-| 4,096 | 4,096 | 524.223 / 519.670 | 6.805 / 6.710 | 98.7% / 98.7% |
-| 65,535 | 0 | 166.113 / 115.686 | 3.655 / 2.333 | 97.8% / 98.0% |
-| 65,535 | 4 | 175.717 / 181.343 | 29.769 / 30.230 | 83.1% / 83.3% |
-| 65,535 | 64 | 207.617 / 206.910 | 31.595 / 31.462 | 84.8% / 84.8% |
-| 65,535 | 1,024 | 2018.463 / 3271.564 | 33.403 / 33.520 | 98.3% / 99.0% |
-| 65,535 | 4,096 | 8811.234 / 8469.706 | 59.853 / 63.211 | 99.3% / 99.3% |
-
-All twelve cases improved in both repetitions. This measures **preparation,
-not end-to-end network scan speed**: many scans are dominated by socket I/O,
-timeouts and reporting. In particular, the default no-exclusion saving is
-only a fraction of a millisecond.
-
-## Implementation and correctness
-
-With no exclusions, keep the already generated vector. For fewer than 64
-ports, filter in place with linear membership checks; this avoids bitmap
-setup and a second allocation. Larger lists use an 8 KiB bitmap covering
-all 65,536 values representable by `u16`, then retain ports in place. This
-changes the large-list filtering cost from ports × exclusions to ports +
-exclusions, while preserving order, duplicate ports and boundary values.
-The helper is kept out of line: release assembly showed an inlined bitmap
-reserved 8,424 bytes in every async poll, whereas the final poll frame uses
-344 bytes and the helper reserves its bitmap only during preparation.
-
-Exploration found that in-place linear membership slows large cases. A
-first production candidate also slowed 64 ports with 4,096 exclusions;
-using the bitmap for that case corrected the regression before the final
-matched runs. The cutoff is a performance choice, not a limit on input.
-
-One focused network-free regression test compares the implementation with
-an independent membership oracle for empty inputs, endpoints 0/65535,
-duplicate ports/exclusions, all ports, and a frozen random order.
-`cargo test --locked` passes 74 tests with one pre-existing ignored doctest.
-All-target Clippy, formatting, documentation and diff checks pass. The
-CI-only scan harness adds an excluded-port sweep using the existing
-listeners, checking expected results and using the actual attempted socket
-count for throughput.
-
-[Raw Criterion samples, confidence intervals, revisions and binary hashes](port-exclusion-benchmark-results.json)
-are included. Hosted correctness/performance results are linked in the PR.
-
-## macOS runner setup
-
-Repeated hosted sweeps on Darwin 25.6 returned `ENOBUFS` (error 55) on
-both master and the candidate, missing fixture listeners. Such runs are
-invalid for performance claims. The workflow provisions the TCP memory
-budget once before either build is measured, raising it from the default
-1/32 to 1/8 of physical memory when necessary, and prints the actual limit
-and allocation counters. It retains every scenario, repeat, listener
-comparison and performance threshold. An immediate untimed diagnostic
-scan records errors and kernel counters if a measured pair misses a
-listener; that failed dataset is not used to claim a speedup.
-
-This uses XNU's [TCP budget initialization](https://github.com/apple-oss-distributions/xnu/blob/main/bsd/netinet/tcp_subr.c)
-and [memory accounting interface](https://github.com/apple-oss-distributions/xnu/blob/main/bsd/sys/mem_acct_private.h).
-It changes the disposable benchmark environment, not RustScan's behavior
-or the operating system configuration on users' machines.

src/address.rs

@@ -1,5 +1,4 @@
 //! Provides functions to parse input IP addresses, CIDRs or files.
-use std::cell::LazyCell;
 use std::collections::BTreeSet;
 use std::fs::{self, File};
 use std::io::{prelude::*, BufReader};
@@ -31,32 +30,13 @@ use crate::warning;
 /// ```
 ///
 /// Finally, any duplicates are removed to avoid excessive scans.
-///
-/// # Blocking
-///
-/// This function blocks: it reads files and resolves host names, and
-/// hickory's synchronous [`Resolver`] runs a Tokio runtime of its own, which
-/// Tokio refuses to drop inside an asynchronous context. Call it before you
-/// start your runtime (as the `rustscan` binary does) or from
-/// `tokio::task::spawn_blocking`, not directly from async code.
 pub fn parse_addresses(input: &Opts) -> Vec<IpAddr> {
-    parse_addresses_with_resolver(input, || get_resolver(&input.resolver))
-}
-
-fn parse_addresses_with_resolver(
-    input: &Opts,
-    create_resolver: impl FnOnce() -> Resolver,
-) -> Vec<IpAddr> {
     let mut ips: Vec<IpAddr> = Vec::new();
     let mut unresolved_addresses: Vec<&str> = Vec::new();
-    // Most scans use literal IPs or CIDRs. Defer the resolver's runtime and
-    // hosts-file loading until a DNS fallback actually needs them, then reuse
-    // that resolver for the rest of the targets and exclusions.
-    let backup_resolver = LazyCell::new(create_resolver);
-    let resolver = || &*backup_resolver;
+    let backup_resolver = get_resolver(&input.resolver);
 
     for address in &input.addresses {
-        let parsed_ips = parse_address_with_resolver(address, &resolver);
+        let parsed_ips = parse_address(address, &backup_resolver);
         if !parsed_ips.is_empty() {
             ips.extend(parsed_ips);
         } else {
@@ -78,7 +58,7 @@ fn parse_addresses_with_resolver(
             continue;
         }
 
-        if let Ok(x) = read_ips_from_file(file_path, &resolver) {
+        if let Ok(x) = read_ips_from_file(file_path, &backup_resolver) {
             ips.extend(x);
         } else {
             warning!(
@@ -89,7 +69,7 @@ fn parse_addresses_with_resolver(
         }
     }
 
-    let excluded_cidrs = parse_excluded_networks_with_resolver(&input.exclude_addresses, &resolver);
+    let excluded_cidrs = parse_excluded_networks(&input.exclude_addresses, &backup_resolver);
 
     // Remove duplicated/excluded IPs.
     let mut seen = BTreeSet::new();
@@ -112,13 +92,6 @@ fn parse_addresses_with_resolver(
 /// let ips = parse_address("127.0.0.1", &Resolver::default().unwrap());
 /// ```
 pub fn parse_address(address: &str, resolver: &Resolver) -> Vec<IpAddr> {
-    parse_address_with_resolver(address, &|| resolver)
-}
-
-fn parse_address_with_resolver<'a>(
-    address: &str,
-    resolver: &impl Fn() -> &'a Resolver,
-) -> Vec<IpAddr> {
     if let Ok(addr) = IpAddr::from_str(address) {
         // `address` is an IP string
         vec![addr]
@@ -137,17 +110,14 @@ fn parse_address_with_resolver<'a>(
 }
 
 /// Uses DNS to get the IPS associated with host
-fn resolve_ips_from_host<'a>(
-    source: &str,
-    backup_resolver: &impl Fn() -> &'a Resolver,
-) -> Vec<IpAddr> {
+fn resolve_ips_from_host(source: &str, backup_resolver: &Resolver) -> Vec<IpAddr> {
     let mut ips: Vec<IpAddr> = Vec::new();
 
     if let Ok(addrs) = source.to_socket_addrs() {
         for ip in addrs {
             ips.push(ip.ip());
         }
-    } else if let Ok(addrs) = backup_resolver().lookup_ip(source) {
+    } else if let Ok(addrs) = backup_resolver.lookup_ip(source) {
         ips.extend(addrs.iter());
     }
 
@@ -170,13 +140,6 @@ fn resolve_ips_from_host<'a>(
 pub fn parse_excluded_networks(
     exclude_addresses: &Option<Vec<String>>,
     resolver: &Resolver,
-) -> Vec<IpCidr> {
-    parse_excluded_networks_with_resolver(exclude_addresses, &|| resolver)
-}
-
-fn parse_excluded_networks_with_resolver<'a>(
-    exclude_addresses: &Option<Vec<String>>,
-    resolver: &impl Fn() -> &'a Resolver,
 ) -> Vec<IpCidr> {
     exclude_addresses
         .iter()
@@ -186,10 +149,7 @@ fn parse_excluded_networks_with_resolver<'a>(
 }
 
 /// Parses a single address into an IpCidr, handling CIDR notation, IP addresses, and hostnames.
-fn parse_single_excluded_address<'a>(
-    addr: &str,
-    resolver: &impl Fn() -> &'a Resolver,
-) -> Vec<IpCidr> {
+fn parse_single_excluded_address(addr: &str, resolver: &Resolver) -> Vec<IpCidr> {
     if let Ok(cidr) = IpCidr::from_str(addr) {
         return vec![cidr];
     }
@@ -293,9 +253,9 @@ fn read_resolver_from_file(path: &str) -> Result<Vec<IpAddr>, std::io::Error> {
 
 #[cfg(not(tarpaulin_include))]
 /// Parses an input file of IPs and uses those
-fn read_ips_from_file<'a>(
+fn read_ips_from_file(
     ips: &std::path::Path,
-    backup_resolver: &impl Fn() -> &'a Resolver,
+    backup_resolver: &Resolver,
 ) -> Result<Vec<IpAddr>, std::io::Error> {
     let file = File::open(ips)?;
     let reader = BufReader::new(file);
@@ -304,7 +264,7 @@ fn read_ips_from_file<'a>(
 
     for address_line in reader.lines() {
         if let Ok(address) = address_line {
-            ips.extend(parse_address_with_resolver(&address, backup_resolver));
+            ips.extend(parse_address(&address, backup_resolver));
         } else {
             debug!("Line in file is not valid");
         }
@@ -315,48 +275,9 @@ fn read_ips_from_file<'a>(
 
 #[cfg(test)]
 mod tests {
-    use super::{parse_addresses, parse_addresses_with_resolver, Opts};
+    use super::{parse_addresses, Opts};
     use std::net::{IpAddr, Ipv4Addr};
 
-    #[test]
-    fn literal_targets_and_exclusions_do_not_initialize_a_resolver() {
-        let opts = Opts {
-            addresses: vec![
-                "192.0.2.0/30".to_owned(),
-                "192.0.2.2".to_owned(),
-                "2001:db8::/126".to_owned(),
-                "2001:db8::3".to_owned(),
-            ],
-            exclude_addresses: Some(vec![
-                "192.0.2.0/31".to_owned(),
-                "192.0.2.3".to_owned(),
-                "2001:db8::/127".to_owned(),
-                "2001:db8::3".to_owned(),
-            ]),
-            ..Default::default()
-        };
-
-        let ips = parse_addresses_with_resolver(&opts, || {
-            panic!("literal targets and exclusions must not initialize DNS")
-        });
-
-        assert_eq!(
-            ips,
-            [
-                "192.0.2.2".parse::<IpAddr>().unwrap(),
-                "2001:db8::2".parse::<IpAddr>().unwrap(),
-            ]
-        );
-    }
-
-    #[test]
-    fn empty_targets_do_not_initialize_a_resolver() {
-        assert!(parse_addresses_with_resolver(&Opts::default(), || {
-            panic!("empty targets must not initialize DNS")
-        })
-        .is_empty());
-    }
-
     #[test]
     fn parse_correct_addresses() {
         let opts = Opts {

src/input.rs

@@ -25,11 +25,8 @@ pub enum ScanOrder {
 ///   - custom will read the ScriptConfig file and the available scripts in the predefined folders
 #[derive(Deserialize, Debug, ValueEnum, Clone, PartialEq, Eq, Copy)]
 pub enum ScriptsRequired {
-    #[serde(alias = "none")]
     None,
-    #[serde(alias = "default")]
     Default,
-    #[serde(alias = "custom")]
     Custom,
 }
 
@@ -529,55 +526,6 @@ mod tests {
         }
     }
 
-    fn assert_config_script_mode(value: &str, expected: ScriptsRequired) {
-        let config: Config = toml::from_str(&format!("scripts = \"{value}\""))
-            .expect("documented script mode must deserialize");
-        assert_eq!(config.scripts, Some(expected));
-    }
-
-    #[test]
-    fn config_scripts_lowercase_none() {
-        assert_config_script_mode("none", ScriptsRequired::None);
-    }
-
-    #[test]
-    fn config_scripts_lowercase_default() {
-        assert_config_script_mode("default", ScriptsRequired::Default);
-    }
-
-    #[test]
-    fn config_scripts_lowercase_custom() {
-        assert_config_script_mode("custom", ScriptsRequired::Custom);
-    }
-
-    #[test]
-    fn config_scripts_legacy_and_unknown_controls() {
-        assert_config_script_mode("None", ScriptsRequired::None);
-        assert_config_script_mode("Default", ScriptsRequired::Default);
-        assert_config_script_mode("Custom", ScriptsRequired::Custom);
-        for invalid in ["disabled", "unknown", ""] {
-            assert!(toml::from_str::<Config>(&format!("scripts = \"{invalid}\"")).is_err());
-        }
-    }
-
-    #[test]
-    fn config_read_accepts_documented_lowercase_script_modes() {
-        for (value, expected) in [
-            ("none", ScriptsRequired::None),
-            ("default", ScriptsRequired::Default),
-            ("custom", ScriptsRequired::Custom),
-        ] {
-            let fixture = std::env::temp_dir().join(format!(
-                "rustscan-config-scripts-{}-{value}.toml",
-                std::process::id()
-            ));
-            std::fs::write(&fixture, format!("scripts = \"{value}\"\n")).unwrap();
-            let config = Config::read(Some(fixture.clone()));
-            std::fs::remove_file(fixture).unwrap();
-            assert_eq!(config.scripts, Some(expected));
-        }
-    }
-
     #[test]
     fn verify_cli() {
         Opts::command().debug_assert();

src/lib.rs

@@ -5,17 +5,12 @@
 //!
 //! The core scanning behaviour is managed by
 //! [`Scanner`](crate::scanner::Scanner) which in turn requires a
-//! [`PortStrategy`](crate::port_strategy::PortStrategy).
-//!
-//! The scanner is asynchronous and runs on [Tokio](https://docs.rs/tokio):
-//! await [`Scanner::run`](crate::scanner::Scanner::run) from a Tokio runtime
-//! with the I/O and time drivers enabled. It never spawns tasks, so a
-//! current-thread runtime is enough (see [`Scanner`](crate::scanner::Scanner)
-//! for details).
+//! [`PortStrategy`](crate::port_strategy::PortStrategy):
 //!
 //! This example is compiled but never executed by the test suite.
 //!
 //! ```no_run
+//! use async_std::task::block_on;
 //! use std::{net::IpAddr, time::Duration};
 //!
 //! use rustscan::input::{PortRanges, ScanOrder};
@@ -38,13 +33,7 @@
 //!         false, // is this a UDP scan?
 //!     );
 //!
-//!     // Inside an async context (e.g. `#[tokio::main]`) this is simply
-//!     // `scanner.run().await`.
-//!     let runtime = tokio::runtime::Builder::new_current_thread()
-//!         .enable_all()
-//!         .build()
-//!         .unwrap();
-//!     let scan_result = runtime.block_on(scanner.run());
+//!     let scan_result = block_on(scanner.run());
 //!
 //!     println!("{:?}", scan_result);
 //! }

src/main.rs

@@ -11,6 +11,7 @@ use rustscan::tui::println_safe;
 use rustscan::{detail, funny_opening, output, warning};
 
 use colorful::{Color, Colorful};
+use futures::executor::block_on;
 use std::collections::HashMap;
 use std::net::IpAddr;
 use std::string::ToString;
@@ -107,13 +108,7 @@ fn main() {
     debug!("Scanner finished building: {scanner:?}");
 
     let mut portscan_bench = NamedTimer::start("Portscan");
-    let scan_result = match run_scan(&scanner) {
-        Ok(scan_result) => scan_result,
-        Err(e) => {
-            eprintln!("error: could not start the scan: {e}");
-            std::process::exit(1);
-        }
-    };
+    let scan_result = block_on(scanner.run_with_status());
     portscan_bench.end();
     benchmarks.push(portscan_bench);
 
@@ -128,7 +123,20 @@ fn main() {
         map.entry(socket.ip()).or_default().push(socket.port());
     }
 
-    if let Some(x) = no_open_ports_warning(&ips, &ports_per_ip, &closed_ports_per_ip, batch_size) {
+    for ip in ips {
+        if ports_per_ip.contains_key(&ip) {
+            continue;
+        }
+
+        // If we got here it means the IP was not found within the HashMap, this
+        // means the scan couldn't find any open ports for it.
+
+        let x = format!("Looks like I didn't find any open ports for {:?}. This is usually caused by a high batch size.
+        \n*I used {} batch size, consider lowering it with {} or a comfortable number for your system.
+        \n Alternatively, increase the timeout if your ping is high. Rustscan -t 2000 for 2000 milliseconds (2s) timeout.\n",
+        ip,
+        batch_size,
+        "'rustscan -b <batch_size> -a <ip address>'");
         warning!(x, opts.greppable, opts.accessible);
     }
 
@@ -211,35 +219,6 @@ fn main() {
     info!("{}", benchmarks.summary());
 }
 
-/// Builds the hint printed when the scan found no open ports.
-///
-/// A host without open ports is normal when another host answered (a subnet
-/// sweep for one port): replies got through at this batch size and timeout,
-/// so batch size and timeout are unlikely to be the cause. The hint is printed
-/// once, and only when no host answered with an open or, under `--closed`, a
-/// closed port.
-fn no_open_ports_warning(
-    ips: &[IpAddr],
-    ports_per_ip: &HashMap<IpAddr, Vec<u16>>,
-    closed_ports_per_ip: &HashMap<IpAddr, Vec<u16>>,
-    batch_size: usize,
-) -> Option<String> {
-    if !ports_per_ip.is_empty() || !closed_ports_per_ip.is_empty() {
-        return None;
-    }
-
-    let target = match ips {
-        [ip] => format!("for {ip:?}"),
-        _ => format!("on any of the {} addresses", ips.len()),
-    };
-    Some(format!("Looks like I didn't find any open ports {}. This is usually caused by a high batch size.
-        \n*I used {} batch size, consider lowering it with {} or a comfortable number for your system.
-        \n Alternatively, increase the timeout if your ping is high. Rustscan -t 2000 for 2000 milliseconds (2s) timeout.\n",
-        target,
-        batch_size,
-        "'rustscan -b <batch_size> -a <ip address>'"))
-}
-
 /// Determines the actual batch size used by the scanner.
 ///
 /// Unix systems may reduce the requested batch size according to the process
@@ -257,25 +236,6 @@ fn effective_batch_size(opts: &Opts) -> usize {
     }
 }
 
-/// Runs the scan to completion on a current-thread Tokio runtime.
-///
-/// The scanner drives every socket from a single future and never spawns
-/// tasks, so one thread is all it can use: a current-thread runtime runs the
-/// scan, the I/O driver and the timers on the calling thread, without the
-/// worker threads and cross-thread wake-ups of a multi-threaded runtime.
-///
-/// The runtime only exists for the duration of the scan. Address resolution
-/// (blocking DNS lookups and file reads) happens before it is created, so it
-/// can never stall the scan, and scripts run after it is gone.
-fn run_scan(scanner: &Scanner) -> std::io::Result<Vec<PortStatus>> {
-    let runtime = tokio::runtime::Builder::new_current_thread()
-        .enable_io()
-        .enable_time()
-        .build()?;
-
-    Ok(runtime.block_on(scanner.run_with_status()))
-}
-
 /// Prints the opening title of RustScan
 #[allow(clippy::items_after_statements, clippy::needless_raw_string_hashes)]
 fn print_opening(opts: &Opts) {
@@ -389,48 +349,7 @@ mod tests {
     use super::effective_batch_size;
     #[cfg(unix)]
     use super::{adjust_ulimit_size, infer_batch_size};
-    use super::{no_open_ports_warning, print_opening, Opts};
-    use std::collections::HashMap;
-    use std::net::IpAddr;
-
-    fn hosts(n: u8) -> Vec<IpAddr> {
-        (1..=n).map(|i| IpAddr::from([192, 0, 2, i])).collect()
-    }
-
-    #[test]
-    fn no_open_ports_silent_when_another_host_answered() {
-        // Sweeping a subnet for one port: one host has it open, the rest do not.
-        let ips = hosts(254);
-        let ports_per_ip = HashMap::from([(ips[0], vec![22])]);
-
-        assert!(no_open_ports_warning(&ips, &ports_per_ip, &HashMap::new(), 4500).is_none());
-    }
-
-    #[test]
-    fn no_open_ports_silent_when_a_host_answered_closed() {
-        // With --closed, a refused connection also proves replies get through.
-        let ips = hosts(254);
-        let closed_ports_per_ip = HashMap::from([(ips[0], vec![443])]);
-
-        assert!(no_open_ports_warning(&ips, &HashMap::new(), &closed_ports_per_ip, 4500).is_none());
-    }
-
-    #[test]
-    fn no_open_ports_warns_once_when_nothing_found() {
-        let warning = no_open_ports_warning(&hosts(254), &HashMap::new(), &HashMap::new(), 4500);
-
-        let warning = warning.expect("one hint when nothing was found");
-        assert!(warning.contains("on any of the 254 addresses."));
-        assert!(warning.contains("4500 batch size"));
-    }
-
-    #[test]
-    fn no_open_ports_names_the_single_host() {
-        let warning = no_open_ports_warning(&hosts(1), &HashMap::new(), &HashMap::new(), 4500);
-
-        let warning = warning.expect("one hint for the single host");
-        assert!(warning.contains("for 192.0.2.1."));
-    }
+    use super::{print_opening, Opts};
 
     #[test]
     #[cfg(unix)]

src/scanner/errors.rs

@@ -1,163 +0,0 @@
-use std::{collections::HashSet, fmt, io, net::IpAddr};
-
-/// Keep the OS error and its target intact until diagnostics need formatting.
-#[derive(Debug)]
-struct ScanError {
-    ip: IpAddr,
-    error: io::Error,
-}
-
-impl fmt::Display for ScanError {
-    fn fmt(&self, f: &mut fmt::Formatter<'_>) -> fmt::Result {
-        write!(f, "{} {}", self.error, self.ip)
-    }
-}
-
-impl std::error::Error for ScanError {}
-
-/// Preserve the normal I/O result layout; only diagnostics need target context.
-pub(super) fn diagnostic_error(error: io::Error, ip: IpAddr, enabled: bool) -> io::Error {
-    if enabled {
-        io::Error::other(ScanError { ip, error })
-    } else {
-        error
-    }
-}
-
-pub(super) struct ScanErrors {
-    enabled: bool,
-    limit: usize,
-    messages: HashSet<String>,
-}
-
-impl ScanErrors {
-    pub(super) fn new(enabled: bool, limit: usize) -> Self {
-        Self {
-            enabled,
-            limit,
-            messages: HashSet::new(),
-        }
-    }
-
-    pub(super) fn record(&mut self, failure: io::Error) {
-        if self.enabled && self.messages.len() < self.limit {
-            self.messages.insert(failure.to_string());
-        }
-    }
-
-    pub(super) fn messages(&self) -> &HashSet<String> {
-        &self.messages
-    }
-}
-
-/// Detect descriptor exhaustion without allocating or interpreting OS messages.
-#[cfg(unix)]
-pub(super) fn is_descriptor_exhaustion(error: &io::Error) -> bool {
-    matches!(error.raw_os_error(), Some(libc::EMFILE | libc::ENFILE))
-}
-
-#[cfg(windows)]
-pub(super) fn is_descriptor_exhaustion(error: &io::Error) -> bool {
-    error.raw_os_error() == Some(windows_sys::Win32::Networking::WinSock::WSAEMFILE)
-}
-
-// Retain the previous fallback for targets without Unix or Winsock error codes.
-#[cfg(not(any(unix, windows)))]
-pub(super) fn is_descriptor_exhaustion(error: &io::Error) -> bool {
-    error
-        .to_string()
-        .to_lowercase()
-        .contains("too many open files")
-}
-
-#[cfg(test)]
-mod tests {
-    use super::*;
-    use std::{fmt, net::SocketAddr};
-
-    #[derive(Debug)]
-    struct MustNotFormat;
-
-    impl fmt::Display for MustNotFormat {
-        fn fmt(&self, _: &mut fmt::Formatter<'_>) -> fmt::Result {
-            panic!("discarded diagnostics must not format errors")
-        }
-    }
-
-    impl std::error::Error for MustNotFormat {}
-
-    fn failure(error: io::Error) -> io::Error {
-        diagnostic_error(error, "127.0.0.1".parse().unwrap(), true)
-    }
-
-    #[test]
-    fn disabled_diagnostics_do_not_format_errors() {
-        let mut errors = ScanErrors::new(false, 1000);
-        errors.record(diagnostic_error(
-            io::Error::other(MustNotFormat),
-            "127.0.0.1".parse().unwrap(),
-            false,
-        ));
-        assert!(errors.messages().is_empty());
-    }
-
-    #[test]
-    fn disabled_diagnostics_preserve_the_original_os_error() {
-        let original = io::Error::from_raw_os_error(123);
-        let error = diagnostic_error(original, "127.0.0.1".parse().unwrap(), false);
-        assert_eq!(error.raw_os_error(), Some(123));
-    }
-
-    #[test]
-    fn full_diagnostics_do_not_format_more_errors() {
-        let mut errors = ScanErrors::new(true, 1);
-        errors.record(failure(io::ErrorKind::ConnectionRefused.into()));
-        errors.record(failure(io::Error::other(MustNotFormat)));
-        assert_eq!(errors.messages().len(), 1);
-    }
-
-    #[test]
-    fn enabled_diagnostics_keep_error_details_and_deduplicate_per_ip() {
-        let mut errors = ScanErrors::new(true, 1000);
-        for port in [80, 443] {
-            errors.record(diagnostic_error(
-                io::Error::other("test socket error"),
-                SocketAddr::from(([127, 0, 0, 1], port)).ip(),
-                true,
-            ));
-        }
-        assert_eq!(errors.messages().len(), 1);
-        assert!(errors.messages().contains("test socket error 127.0.0.1"));
-    }
-
-    #[cfg(unix)]
-    #[test]
-    fn detects_unix_descriptor_limits_by_code() {
-        for code in [libc::EMFILE, libc::ENFILE] {
-            assert!(is_descriptor_exhaustion(&io::Error::from_raw_os_error(
-                code
-            )));
-        }
-        assert!(!is_descriptor_exhaustion(&io::Error::from_raw_os_error(
-            libc::ECONNREFUSED
-        )));
-    }
-
-    #[cfg(windows)]
-    #[test]
-    fn detects_winsock_descriptor_limits_by_code() {
-        use windows_sys::Win32::Networking::WinSock::{WSAECONNREFUSED, WSAEMFILE};
-        assert!(is_descriptor_exhaustion(&io::Error::from_raw_os_error(
-            WSAEMFILE
-        )));
-        assert!(!is_descriptor_exhaustion(&io::Error::from_raw_os_error(
-            WSAECONNREFUSED
-        )));
-    }
-
-    #[cfg(any(unix, windows))]
-    #[test]
-    fn descriptor_check_never_formats_an_error() {
-        assert!(!is_descriptor_exhaustion(&io::Error::other(MustNotFormat)));
-    }
-}

src/scanner/mod.rs

@@ -7,56 +7,20 @@ use log::debug;
 mod socket_iterator;
 use socket_iterator::SocketIterator;
 
-mod errors;
-use errors::{diagnostic_error, is_descriptor_exhaustion, ScanErrors};
-
+use async_std::net::TcpStream;
+use async_std::prelude::*;
+use async_std::task::sleep;
+use async_std::{io, net::UdpSocket};
 use colored::Colorize;
-use futures::stream::{FuturesUnordered, StreamExt};
+use futures::stream::FuturesUnordered;
 use std::collections::BTreeMap;
-use std::future::poll_fn;
-use std::task::Poll;
 use std::{
-    collections::HashMap,
-    io,
-    net::{IpAddr, Ipv4Addr, Ipv6Addr, Shutdown, SocketAddr},
+    collections::{HashMap, HashSet},
+    net::{IpAddr, Shutdown, SocketAddr},
     num::NonZeroU8,
     sync::Arc,
     time::Duration,
 };
-use tokio::io::Interest;
-use tokio::net::{TcpStream, UdpSocket};
-use tokio::time::{sleep, timeout};
-
-/// How many sockets the scan starts or finishes (counted together) between
-/// two polls of the runtime's I/O driver; see `Scanner::scan_sockets`.
-///
-/// Every round ends with a non-blocking poll of the OS selector, so this
-/// keeps the overhead far below 1% while bounding the time between two
-/// polls to about a millisecond on Linux.
-const WORK_PER_TURN: usize = 128;
-
-/// Keeps the generated order, including duplicate ports, while removing exclusions.
-// Keep the bitmap out of the async polling frame, which runs on every wake.
-#[inline(never)]
-fn filter_excluded_ports(mut ports: Vec<u16>, excluded: &[u16]) -> Vec<u16> {
-    if excluded.is_empty() {
-        return ports;
-    }
-
-    // Bitmap setup costs more than membership checks on short port lists.
-    if ports.len() < 64 {
-        ports.retain(|port| !excluded.contains(port));
-        return ports;
-    }
-
-    // The complete u16 port space fits in an 8 KiB bitmap.
-    let mut excluded_bits = [0_u64; 1024];
-    for &port in excluded {
-        excluded_bits[usize::from(port) / 64] |= 1 << (port % 64);
-    }
-    ports.retain(|&port| excluded_bits[usize::from(port) / 64] & (1 << (port % 64)) == 0);
-    ports
-}
 
 /// UDP payload lookup: port -> payload bytes
 ///
@@ -86,19 +50,6 @@ pub fn build_udp_payload_lookup(udp_map: &'static BTreeMap<Vec<u16>, Vec<u8>>) -
 /// batch_size is how many ports at a time should be scanned
 /// Timeout is the time RustScan should wait before declaring a port closed. As datatype Duration.
 /// greppable is whether or not RustScan should print things, or wait until the end to print only the ip and open ports.
-///
-/// # Runtime
-///
-/// The futures returned by [`Scanner::run`] and [`Scanner::run_with_status`]
-/// use Tokio sockets and timers, so they must be polled from within a
-/// [Tokio](https://docs.rs/tokio) runtime that has both the I/O and the time
-/// drivers enabled (for example `#[tokio::main]`, or a runtime built with
-/// `enable_all()`). Polling them from another executor panics.
-///
-/// The scan never spawns tasks: every socket is driven from the one future
-/// you await, so a current-thread runtime is enough (it is what the
-/// `rustscan` binary uses), and the future is `Send` if you prefer to spawn
-/// it on a multi-threaded runtime.
 #[cfg(not(tarpaulin_include))]
 #[derive(Debug)]
 pub struct Scanner {
@@ -206,20 +157,15 @@ impl Scanner {
     /// definitive answer: open sockets, plus closed sockets when
     /// [`Self::with_closed_ports`] is enabled.
     pub async fn run_with_status(&self) -> Vec<PortStatus> {
-        // Every in-flight socket is polled from this single future through a
-        // `FuturesUnordered`. Under Tokio's cooperative budget the future
-        // would be forced to yield after ~128 sockets made progress, and the
-        // `FuturesUnordered` would then re-poll every other ready socket just
-        // to have it return `Pending` again. Opt out; `scan_sockets` yields to
-        // the runtime by itself, in rounds of `WORK_PER_TURN` sockets.
-        tokio::task::unconstrained(self.scan()).await
-    }
-
-    async fn scan(&self) -> Vec<PortStatus> {
-        let ports = filter_excluded_ports(self.port_strategy.order(), &self.exclude_ports);
+        let ports: Vec<u16> = self
+            .port_strategy
+            .order()
+            .iter()
+            .filter(|&port| !self.exclude_ports.contains(port))
+            .copied()
+            .collect();
         let mut found_sockets: Vec<PortStatus> = Vec::new();
-        let mut errors =
-            ScanErrors::new(log::log_enabled!(log::Level::Debug), self.ips.len() * 1000);
+        let mut errors: HashSet<String> = HashSet::new();
 
         // Build UDP payload lookup once (only if we are scanning UDP).
         // This avoids cloning a big map into every spawned future and turns
@@ -254,88 +200,52 @@ impl Scanner {
             }
         }
 
-        debug!("Typical socket connection errors {:?}", errors.messages());
+        debug!("Typical socket connection errors {errors:?}");
         debug!("Sockets found: {:?}", found_sockets);
         found_sockets
     }
 
     /// Scans every socket yielded by `sockets`, keeping at most `batch_size`
     /// connection attempts in flight.
-    ///
-    /// Starts sockets and handles finished ones in rounds of at most
-    /// [`WORK_PER_TURN`] of either, and lets the runtime poll for I/O and fire
-    /// timers between rounds. One poll of the scan future otherwise lasts
-    /// until no socket is ready, which can take long: starting a whole large
-    /// batch at once, or a run of UDP probes that all finish straight away.
-    /// Sockets that finish in the meantime are only noticed once the poll
-    /// ends, so their results come late, and if the poll outlasts the timeout
-    /// their timers fire before their answers are seen.
     async fn scan_sockets(
         &self,
         mut sockets: SocketIterator<'_>,
         udp_payloads: &Option<Arc<UdpPayloadLookup>>,
         found_sockets: &mut Vec<PortStatus>,
-        errors: &mut ScanErrors,
+        errors: &mut HashSet<String>,
     ) {
         let mut ftrs = FuturesUnordered::new();
 
-        loop {
-            let mut work = 0;
-            while work < WORK_PER_TURN {
-                let mut started = false;
-                if ftrs.len() < self.batch_size {
-                    if let Some(socket) = sockets.next() {
-                        ftrs.push(self.scan_socket(socket, udp_payloads.clone()));
-                        started = true;
-                        work += 1;
-                    }
-                }
-
-                // Poll once without waiting; this starts the new socket.
-                match poll_fn(|cx| Poll::Ready(ftrs.poll_next_unpin(cx))).await {
-                    Poll::Ready(Some(result)) => {
-                        self.record(result, found_sockets, errors);
-                        work += 1;
-                    }
-                    // Nothing in flight and nothing left to start.
-                    Poll::Ready(None) => return,
-                    // Nothing has finished; keep starting sockets while there is room.
-                    Poll::Pending if started => {}
-                    Poll::Pending => break,
-                }
-            }
-
-            if work >= WORK_PER_TURN {
-                tokio::task::yield_now().await;
+        for _ in 0..self.batch_size {
+            if let Some(socket) = sockets.next() {
+                ftrs.push(self.scan_socket(socket, udp_payloads.clone()));
             } else {
-                // The batch is full (or complete) and every socket in it is
-                // waiting on the network.
-                match ftrs.next().await {
-                    Some(result) => self.record(result, found_sockets, errors),
-                    None => return,
-                }
+                break;
             }
         }
-    }
 
-    /// Records the outcome of one socket.
-    fn record(
-        &self,
-        result: io::Result<PortStatus>,
-        found_sockets: &mut Vec<PortStatus>,
-        errors: &mut ScanErrors,
-    ) {
-        match result {
-            Ok(status) => found_sockets.push(status),
-            Err(error) => errors.record(error),
+        while let Some(result) = ftrs.next().await {
+            if let Some(socket) = sockets.next() {
+                ftrs.push(self.scan_socket(socket, udp_payloads.clone()));
+            }
+
+            match result {
+                Ok(status) => found_sockets.push(status),
+                Err(e) => {
+                    let error_string = e.to_string();
+                    if errors.len() < self.ips.len() * 1000 {
+                        errors.insert(error_string);
+                    }
+                }
+            }
         }
     }
 
     /// Given a socket, scan it self.tries times.
     /// Turns the address into a SocketAddr
     /// Deals with the `<result>` type
-    /// Panics if the OS reports descriptor exhaustion.
-    /// Other failures retain their I/O error, with target context for debug diagnostics.
+    /// If it experiences error ErrorKind::Other then too many files are open and it Panics!
+    /// Else any other error, it returns the error in Result as a string
     /// If no errors occur, it returns the port number in Result to signify the port is open.
     /// This function mainly deals with the logic of Results handling.
     /// # Example
@@ -359,7 +269,7 @@ impl Scanner {
             match self.connect(socket).await {
                 Ok(tcp_stream) => {
                     debug!("Connection was successful, shutting down stream {}", socket);
-                    if let Err(e) = shutdown_both(tcp_stream) {
+                    if let Err(e) = tcp_stream.shutdown(Shutdown::Both) {
                         debug!("Shutdown stream error {}", e);
                     }
                     self.fmt_ports(socket);
@@ -375,14 +285,14 @@ impl Scanner {
                         return Ok(PortStatus::Closed(socket));
                     }
 
-                    assert!(!is_descriptor_exhaustion(&e), "Too many open files. Please reduce batch size. The default is 5000. Try -b 2500.");
+                    let mut error_string = e.to_string();
+
+                    assert!(!error_string.to_lowercase().contains("too many open files"), "Too many open files. Please reduce batch size. The default is 5000. Try -b 2500.");
 
                     if nr_try == tries {
-                        return Err(diagnostic_error(
-                            e,
-                            socket.ip(),
-                            log::log_enabled!(log::Level::Debug),
-                        ));
+                        error_string.push(' ');
+                        error_string.push_str(&socket.ip().to_string());
+                        return Err(io::Error::other(error_string));
                     }
                 }
             };
@@ -429,22 +339,35 @@ impl Scanner {
     /// ```
     ///
     async fn connect(&self, socket: SocketAddr) -> io::Result<TcpStream> {
-        timeout(self.timeout, TcpStream::connect(socket))
-            .await
-            .unwrap_or_else(|_elapsed| Err(timed_out()))
+        let stream = io::timeout(
+            self.timeout,
+            async move { TcpStream::connect(socket).await },
+        )
+        .await?;
+        Ok(stream)
     }
 
-    /// Binds a non-blocking UDP socket on the unspecified address of the
-    /// target's address family, so we can send and receive packets.
-    fn udp_bind(socket: SocketAddr) -> io::Result<std::net::UdpSocket> {
+    /// Binds to a UDP socket so we can send and receive packets
+    /// # Example
+    ///
+    /// ```compile_fail
+    /// # use std::net::{IpAddr, Ipv6Addr, SocketAddr};
+    /// let port: u16 = 80;
+    /// // ip is an IpAddr type
+    /// let ip = IpAddr::V6(Ipv6Addr::new(0, 0, 0, 0, 0, 0, 0, 1));
+    /// let socket = SocketAddr::new(ip, port);
+    /// scanner.udp_bind(socket);
+    /// // returns Result which is either Ok(stream) for port is open, or Err for port is closed.
+    /// // Timeout occurs after self.timeout seconds
+    /// ```
+    ///
+    async fn udp_bind(&self, socket: SocketAddr) -> io::Result<UdpSocket> {
         let local_addr = match socket {
-            SocketAddr::V4(_) => SocketAddr::from((Ipv4Addr::UNSPECIFIED, 0)),
-            SocketAddr::V6(_) => SocketAddr::from((Ipv6Addr::UNSPECIFIED, 0)),
+            SocketAddr::V4(_) => "0.0.0.0:0".parse::<SocketAddr>().unwrap(),
+            SocketAddr::V6(_) => "[::]:0".parse::<SocketAddr>().unwrap(),
         };
 
-        let udp_socket = std::net::UdpSocket::bind(local_addr)?;
-        udp_socket.set_nonblocking(true)?;
-        Ok(udp_socket)
+        UdpSocket::bind(local_addr).await
     }
 
     /// Performs a UDP scan on the specified socket with a payload and wait duration
@@ -468,59 +391,32 @@ impl Scanner {
         payload: &[u8],
         wait: Duration,
     ) -> io::Result<bool> {
-        let udp_socket = match Self::udp_bind(socket) {
-            Ok(udp_socket) => udp_socket,
+        match self.udp_bind(socket).await {
+            Ok(udp_socket) => {
+                let mut buf = [0u8; 1024];
+
+                udp_socket.connect(socket).await?;
+                udp_socket.send(payload).await?;
+
+                match io::timeout(wait, udp_socket.recv(&mut buf)).await {
+                    Ok(size) => {
+                        debug!("Received {size} bytes");
+                        self.fmt_ports(socket);
+                        Ok(true)
+                    }
+                    Err(e) => {
+                        if e.kind() == io::ErrorKind::TimedOut {
+                            Ok(false)
+                        } else {
+                            Err(e)
+                        }
+                    }
+                }
+            }
             Err(e) => {
                 debug!("Error binding UDP socket: {e:?}");
-                return Err(e);
+                Err(e)
             }
-        };
-        let mut buf = [0u8; 1024];
-
-        udp_socket.connect(socket)?;
-
-        // Send the probe and try the first receive straight away, as
-        // async-std did. Tokio's readiness-based I/O would first wait for the
-        // reactor to report the socket ready, costing every probe extra trips
-        // through the event loop, while the probe can almost always be sent
-        // immediately and, on the local host, the answer (often an ICMP "port
-        // unreachable", seen as a refused connection) is usually already
-        // there when the send returns. The socket is only registered with
-        // Tokio when we really have to wait.
-        let sent = match udp_socket.send(payload) {
-            Ok(_) => true,
-            Err(e) if e.kind() == io::ErrorKind::WouldBlock => false,
-            Err(e) => return Err(e),
-        };
-        let early = if sent {
-            udp_socket.recv(&mut buf)
-        } else {
-            Err(io::ErrorKind::WouldBlock.into())
-        };
-
-        let received = match early {
-            Err(e) if e.kind() == io::ErrorKind::WouldBlock => {
-                let udp_socket = UdpSocket::from_std(udp_socket)?;
-                if !sent {
-                    udp_socket.send(payload).await?;
-                }
-                match timeout(wait, recv_or_error(&udp_socket, &mut buf)).await {
-                    Ok(received) => received,
-                    // Nothing came back in time.
-                    Err(_elapsed) => return Ok(false),
-                }
-            }
-            early => early,
-        };
-
-        match received {
-            Ok(size) => {
-                debug!("Received {size} bytes");
-                self.fmt_ports(socket);
-                Ok(true)
-            }
-            Err(e) if e.kind() == io::ErrorKind::TimedOut => Ok(false),
-            Err(e) => Err(e),
         }
     }
 
@@ -547,69 +443,14 @@ impl Scanner {
     }
 }
 
-/// Shuts down both halves of a connected stream before it is closed, as the
-/// async-std implementation did (`TcpStream::shutdown(Shutdown::Both)`).
-///
-/// Tokio only offers an asynchronous write-side shutdown, so take the socket
-/// back from the reactor and shut it down synchronously instead.
-fn shutdown_both(stream: TcpStream) -> io::Result<()> {
-    stream.into_std()?.shutdown(Shutdown::Both)
-}
-
-/// Waits for a datagram on a connected UDP socket, or for the error the
-/// system queued for it: an ICMP "port unreachable" is reported as a refused
-/// connection, which is how closed UDP ports are told apart from filtered
-/// ones.
-///
-/// `UdpSocket::recv` alone does not do this on Linux: a queued ICMP error
-/// only raises `EPOLLERR`, which Tokio does not count as readable, so `recv`
-/// would sleep until the timeout. async-io treated `EPOLLERR` as readable.
-async fn recv_or_error(socket: &UdpSocket, buf: &mut [u8]) -> io::Result<usize> {
-    loop {
-        let ready = socket.ready(Interest::READABLE | Interest::ERROR).await?;
-
-        if ready.is_readable() {
-            match socket.try_recv(buf) {
-                // A spurious wake-up; wait again (unless the socket is closed
-                // for reading, which would wake us up forever).
-                Err(e) if e.kind() == io::ErrorKind::WouldBlock && !ready.is_read_closed() => {}
-                received => return received,
-            }
-        }
-
-        if ready.is_error() {
-            // Take (and clear) the queued error. `WouldBlock` when there is
-            // none makes Tokio clear the error readiness, so this cannot spin.
-            let queued = socket.try_io(Interest::ERROR, || {
-                socket
-                    .take_error()?
-                    .map_or_else(|| Err(io::ErrorKind::WouldBlock.into()), Ok)
-            });
-            match queued {
-                Ok(error) => return Err(error),
-                Err(e) if e.kind() == io::ErrorKind::WouldBlock => {}
-                Err(e) => return Err(e),
-            }
-        }
-    }
-}
-
-/// The error reported for a connection attempt that hit the timeout; the same
-/// kind and message async-std's `io::timeout` used.
-fn timed_out() -> io::Error {
-    io::Error::new(io::ErrorKind::TimedOut, "future timed out")
-}
-
 #[cfg(test)]
 mod tests {
     use super::*;
 
     use crate::input::{PortRanges, ScanOrder};
-    use std::collections::HashSet;
 
-    // These tests never open sockets: they only build a `Scanner` (and its
-    // futures), run a scan that has no socket to scan, or inspect the payload
-    // table generated by build.rs.
+    // These tests never open sockets: they only build a `Scanner` or inspect
+    // the payload table generated by build.rs.
 
     fn test_scanner() -> Scanner {
         let addrs = vec!["127.0.0.1".parse::<IpAddr>().unwrap()];
@@ -627,35 +468,6 @@ mod tests {
         )
     }
 
-    #[test]
-    fn port_exclusions_preserve_order_duplicates_and_boundaries() {
-        let inputs = [
-            Vec::new(),
-            vec![65535, 0, 80, 443, 80, 65535, 1],
-            (0..=65535).collect(),
-            PortStrategy::pick(&Some(PortRanges(vec![(0, 1023)])), None, ScanOrder::Random).order(),
-        ];
-        let exclusions = [
-            Vec::new(),
-            vec![0],
-            vec![0, 65535, 0, 443],
-            (0..1024).collect(),
-            (0..=65535).collect(),
-        ];
-
-        for ports in inputs {
-            for excluded in &exclusions {
-                let excluded_set: HashSet<_> = excluded.iter().copied().collect();
-                let expected: Vec<u16> = ports
-                    .iter()
-                    .filter(|&port| !excluded_set.contains(port))
-                    .copied()
-                    .collect();
-                assert_eq!(filter_excluded_ports(ports.clone(), excluded), expected);
-            }
-        }
-    }
-
     #[test]
     fn library_scanner_is_quiet_by_default() {
         let scanner = test_scanner();
@@ -692,55 +504,6 @@ mod tests {
         assert_eq!(scanner.interval, Duration::from_millis(250));
     }
 
-    /// Embedders may spawn the scan on a multi-threaded runtime, which needs
-    /// `Send` futures. The futures are only created here, never polled, so
-    /// no socket is opened.
-    #[test]
-    fn scan_futures_are_send() {
-        fn assert_send<T: Send>(_: &T) {}
-
-        let scanner = test_scanner();
-        assert_send(&scanner.run());
-        assert_send(&scanner.run_with_status());
-    }
-
-    /// Drives a scan on the same kind of runtime the CLI builds. Every port
-    /// is excluded, so the scan has no socket to open and returns at once.
-    #[test]
-    fn scan_without_sockets_completes_on_a_current_thread_runtime() {
-        let addrs = vec!["127.0.0.1".parse::<IpAddr>().unwrap()];
-        let runtime = tokio::runtime::Builder::new_current_thread()
-            .enable_io()
-            .enable_time()
-            .build()
-            .unwrap();
-
-        for interval in [Duration::ZERO, Duration::from_millis(10)] {
-            let scanner = Scanner::new(
-                &addrs,
-                10,
-                Duration::from_millis(100),
-                1,
-                true,
-                PortStrategy::Manual(vec![1, 2]),
-                true,
-                vec![1, 2],
-                false,
-            )
-            .with_interval(interval);
-
-            assert!(runtime.block_on(scanner.run_with_status()).is_empty());
-        }
-    }
-
-    #[test]
-    fn timeouts_are_reported_as_timed_out_errors() {
-        let error = timed_out();
-
-        assert_eq!(error.kind(), io::ErrorKind::TimedOut);
-        assert_eq!(error.to_string(), "future timed out");
-    }
-
     /// Regression test for https://github.com/bee-san/RustScan/issues/933:
     /// the SNMP public-walk probe must be the exact 33-byte BER packet, with
     /// the literal `public` community string intact. The old hexdigits-only

src/scripts/mod.rs

@@ -143,78 +143,9 @@ pub fn init_scripts(scripts: &ScriptsRequired) -> Result<Vec<ScriptFile>> {
         }
     }
 
-    verify_script_programs(&scripts_to_run)?;
-
     Ok(scripts_to_run)
 }
 
-/// Whether `path/program` is something the shell could run.
-#[cfg(not(windows))]
-fn program_exists_in(dir: &std::path::Path, program: &str) -> bool {
-    dir.join(program).is_file()
-}
-
-/// Whether `path/program` is something `cmd.exe` could run, which appends
-/// entries from `PATHEXT` to a bare name.
-#[cfg(windows)]
-fn program_exists_in(dir: &std::path::Path, program: &str) -> bool {
-    if dir.join(program).is_file() {
-        return true;
-    }
-    let extensions = std::env::var("PATHEXT").unwrap_or_default();
-    extensions
-        .split(';')
-        .filter(|extension| !extension.is_empty())
-        .any(|extension| dir.join(format!("{program}{extension}")).is_file())
-}
-
-fn program_is_available(program: &str) -> bool {
-    match std::env::var_os("PATH") {
-        Some(path) => std::env::split_paths(&path).any(|dir| program_exists_in(&dir, program)),
-        None => false,
-    }
-}
-
-/// Check that the program each selected script starts with is available.
-///
-/// Scripts run through `sh -c`, so a missing binary surfaces only as a bare
-/// "Exit code = 127" once the scan has already finished. Report it up front
-/// instead, before a long scan has been wasted.
-fn verify_script_programs(scripts_to_run: &[ScriptFile]) -> Result<()> {
-    for script in scripts_to_run {
-        let call_format = match &script.call_format {
-            Some(call_format) => call_format,
-            None => continue,
-        };
-        let program = match call_format.split_whitespace().next() {
-            Some(program) => program,
-            None => continue,
-        };
-        // A leading assignment (`FOO=bar nmap ...`) or an absolute path is not
-        // a bare program name, and `{{...}}` placeholders are only filled in
-        // when the script actually runs, so leave those formats alone.
-        if program.contains('=')
-            || program.contains('/')
-            || program.contains('$')
-            || program.contains("{{")
-        {
-            continue;
-        }
-        if !program_is_available(program) {
-            return Err(anyhow!(
-                "The script `{}` runs `{}`, which is not installed or not in PATH. \
-                 Install it, or skip scripts with `--scripts none`.",
-                script
-                    .path
-                    .as_ref()
-                    .map_or_else(|| "default".to_string(), |path| path.display().to_string()),
-                program
-            ));
-        }
-    }
-    Ok(())
-}
-
 pub fn parse_scripts(scripts: Vec<PathBuf>) -> Vec<ScriptFile> {
     let mut parsed_scripts: Vec<ScriptFile> = Vec::with_capacity(scripts.len());
     for script in scripts {
@@ -417,7 +348,8 @@ impl ScriptFile {
         if let Ok(file) = File::open(script) {
             for mut line in io::BufReader::new(file).lines().skip(1).flatten() {
                 if line.starts_with('#') {
-                    line = line[1..].trim().to_string();
+                    line.retain(|c| c != '#');
+                    line = line.trim().to_string();
                     line.push('\n');
                     lines_buf.push_str(&line);
                 } else {
@@ -471,21 +403,6 @@ impl ScriptConfig {
 mod tests {
     use super::*;
 
-    #[test]
-    fn script_headers_preserve_literal_hash_characters() {
-        let fixture =
-            std::env::temp_dir().join(format!("rustscan-literal-hash-{}.txt", std::process::id()));
-        fs::write(&fixture, "#!/usr/bin/env python3\n#tags = [\"C#\"]\n#call_format = \"fixture {{ip}} {{port}} https://example.invalid/#section\"\n\n").unwrap();
-        let parsed = ScriptFile::new(fixture.clone());
-        fs::remove_file(fixture).unwrap();
-        let parsed = parsed.expect("valid TOML metadata");
-        assert_eq!(parsed.tags, Some(vec!["C#".to_owned()]));
-        assert_eq!(
-            parsed.call_format.as_deref(),
-            Some("fixture {{ip}} {{port}} https://example.invalid/#section")
-        );
-    }
-
     #[test]
     fn find_and_parse_scripts() {
         let scripts = find_scripts("fixtures/.rustscan_scripts".into()).unwrap();
@@ -578,40 +495,4 @@ mod tests {
 
         assert_eq!(script_dir_base, dirs::home_dir().unwrap());
     }
-
-    fn script_with_call_format(call_format: &str) -> ScriptFile {
-        ScriptFile {
-            path: Some(PathBuf::from("/tmp/script.txt")),
-            tags: None,
-            developer: None,
-            port: None,
-            ports_separator: None,
-            call_format: Some(call_format.to_string()),
-        }
-    }
-
-    #[test]
-    fn missing_script_program_is_reported() {
-        let scripts = vec![script_with_call_format("definitely-not-a-real-binary {{ip}}")];
-        let err = verify_script_programs(&scripts).unwrap_err();
-        assert!(err.to_string().contains("definitely-not-a-real-binary"));
-    }
-
-    #[test]
-    fn available_script_program_is_accepted() {
-        // `sh` is what the scripts are actually run through, so it is present
-        // on every platform that can execute them.
-        let scripts = vec![script_with_call_format("sh -c true {{ip}}")];
-        assert!(verify_script_programs(&scripts).is_ok());
-    }
-
-    #[test]
-    fn placeholder_and_assignment_prefixes_are_skipped() {
-        let scripts = vec![
-            script_with_call_format("{{script}} {{ip}} {{port}}"),
-            script_with_call_format("FOO=bar definitely-not-a-real-binary"),
-            script_with_call_format("/opt/custom/nmap {{ip}}"),
-        ];
-        assert!(verify_script_programs(&scripts).is_ok());
-    }
 }