fix: add option ignoreTrailingSlash to MockAgent and .intercept()

[Uzlopak]

fixes #3649

Normalize the path in the mock interceptor

[Uzlopak]

@metcoder95

Should we wait for a second opinion/approval?

[KhafraDev]

I don't think this is a good idea.

[metcoder95]

@KhafraDev can you elaborate on that?

[Uzlopak]

@metcoder95

I chattet with @KhafraDev and he sent me following tweet of @mcollina

https://x.com/matteocollina/status/1828802494832705896

Seems like to every issue there is a tweet by matteo :D.

Anyhow:
I reworked on this and added ignoreTrailingSlashes as an option of MockAgent. Where do we document stuff like this?

[KhafraDev]

Why have it as an option on the client rather than passed in the .intercept options?

[metcoder95]

lgtm, here we can update the docs: https://github.com/nodejs/undici/blob/main/docs/docs/api/MockPool.md#mockpoolinterceptoptions

[Uzlopak]

@KhafraDev
@metcoder95

Now you can set it globally on agent level and on intercept level.

[mcollina]

lgtm

[metcoder95]

Shall we backport it?

[github-actions]

The backport to v6.x failed:

The process '/usr/bin/git' failed with exit code 1 

To backport manually, run these commands in your terminal:

# Fetch latest updates from GitHub git fetch # Create a new working tree git worktree add .worktrees/backport-v6.x v6.x # Navigate to the new working tree cd .worktrees/backport-v6.x # Create a new branch git switch --create backport-3655-to-v6.x # Cherry-pick the merged commit of this pull request and resolve the conflicts git cherry-pick -x --mainline 1 c0be359aa5b3f2b8076fbad3413a43e8406f6741 # Push it to GitHub git push --set-upstream origin backport-3655-to-v6.x # Go back to the original working tree cd ../.. # Delete the working tree git worktree remove .worktrees/backport-v6.x

Then, create a pull request where the base branch is v6.x and the compare/head branch is backport-3655-to-v6.x.