basename and extname added to URL.pathname

[SetTrend]

Added two new getter properties to URL::pathname property:

1. basename
2. extname

Fixes: #47756

[nodejs-github-bot]

Review requested:

• @nodejs/url

[mscdex]

These changes don't seem to be part of the standard?

[anonrig]

This change is against the WHATWG URL standard. Pathname getter is defined in the spec: https://url.spec.whatwg.org/#dom-url-pathname

[SetTrend]

Yes, you are right.

My implementation is merely the result from a discussion in issue #47756. It's a work in progress; a draft; a suggestion; a quick shot; to be discussed.

From the NodeJS source code I expected that the project is using its own URL implementation. So, I extended the original implementation, not altering its main interface.

Which kind of changes would you like me to implement?

[tniessen]

Anything we add to the node:url module that is not covered by the WhatWG URL Standard should be well-justified. I don't see that being the case here. The proposed additions cover niche cases that are better handled using existing APIs, as described in various comments in #47756.