From 045f995203a4d2532281617d11d8ccd63a5a44ad Mon Sep 17 00:00:00 2001
From: Andrew Dunham <andrew@du.nham.ca>
Date: Sun, 5 Mar 2023 00:45:53 -0500
Subject: [PATCH] ipn/localapi: close portmapper after debug

This ensures that any mappings that are created are correctly cleaned
up, instead of waiting for them to expire in the router.

Updates #7377

Signed-off-by: Andrew Dunham <andrew@du.nham.ca>
Change-Id: I436248ee7740eded6d8adae5df525e785a8f7ccb
---
 ipn/localapi/localapi.go | 2 ++
 1 file changed, 2 insertions(+)

diff --git a/ipn/localapi/localapi.go b/ipn/localapi/localapi.go
index 574fa12a0..9997f4999 100644
--- a/ipn/localapi/localapi.go
+++ b/ipn/localapi/localapi.go
@@ -689,6 +689,8 @@ func (h *Handler) serveDebugPortmap(w http.ResponseWriter, r *http.Request) {
 		}
 		logf("cb: no mapping")
 	})
+	defer c.Close()
+
 	linkMon, err := monitor.New(logger.WithPrefix(logf, "monitor: "))
 	if err != nil {
 		logf("error creating monitor: %v", err)