[pcbeingused333]

Fixes #475.

processAttempt builds the forwarded URL as cfg.URL.Path + webhookEvent.Body.Path, so a forwarding URL whose path ends in / (hookdeck listen http://127.0.0.1:47913/base/ <source>) sends every request to /base//webhook instead of /base/webhook.

The two paths are now joined by joinForwardPath, which drops one trailing / from the forwarding URL's path when the event path starts with /. The event path is otherwise kept as is, since a receiver may route on its own trailing slash (which is why path.Join is not used), and an empty event path still forwards to the forwarding URL's path unchanged.

forward_path_test.go runs one attempt through processAttempt against a local server for six cases: forwarding path with and without a trailing slash, the root, no path, an event path with its own trailing slash, and an empty event path. On master the three cases whose forwarding path ends in / (/base/, / and an event path with its own trailing slash) fail with //; with this change all six pass. go vet ./... passes. go test ./... passes apart from two tests in packages this change does not touch, which fail the same way on master in the container used: TestCheckServerHealth_DefaultPorts/HTTPS_default_port (needs an outbound connection to port 443) and TestLogin_rejectedKeyHeadlessFailsFast in pkg/login. The acceptance tests were not run.

The issue also notes that a query string on the forwarding URL is dropped; that is not changed here, since forwarding it or rejecting it at startup is a behaviour choice.

This change was prepared with Claude Code.

[pcbeingused333]

Hi, friendly ping on this one when you have a moment. Happy to adjust anything or rebase if needed. Thanks for maintaining the project!

[leggetter]

Thanks for this, and sorry for the wait. The fix to the forwarded request looks right, and the new test fails on main in exactly the // cases, so it pins the change.

While checking it I found the same // in the URL the CLI prints, which is built separately from the request URL. I've pushed a commit to this branch that covers it, so the printed URL matches the one the request went to:

• pkg/listen/forwardurl now holds the join (moved out of proxy.go), and proxy.go calls it.
• The compact and interactive output lines use it.
• The TUI event details and the TUI "Forwards to →" header use it too. Both used to leave out the forwarding URL's path.
• There is one test per place.

I merged main into the branch rather than rebasing, so your commit is unchanged. CI is approved and running.