Uh oh!
There was an error while loading. Please reload this page.
url: doc deprecate url.resolve - #61780
Conversation
richardlau
commented
Feb 12, 2026
We need to update Lines 2657 to 2686 in 27ec431 |
legendecas
commented
Feb 13, 2026
Updated |
| - version: REPLACEME | ||
| pr-url: https://github.com/nodejs/node/pull/61780 | ||
| description: Documentation-only deprecation. |
There was a problem hiding this comment.
Without #61776, this is actually an application deprecation introduced in v24.0.0. The versioning semantics are a bit tricky, but we shouldn't call this a doc deprecation if it's emitting warnings!
This has got a little confusing because of the mixture of issues and decisions about whether the software needs to conform to the documentation or the other way around. I moved the documentation issue to #61816 and my suggestion is to retroactively document in this PR that #55017 (implicitly) introduced an Application deprecation (non- |
MikeMcC399
commented
Feb 20, 2026
Have you been able to look at the suggestion to change this PR so that it reflects the deprecation change in v24.0.0 instead of tying it to a The PR is currently marked with
author ready |
legendecas
commented
Feb 20, 2026
As comment #61780 (comment) mentioned, I think this PR no longer serves my original intention. So I'm going to close this. |
MikeMcC399
commented
Feb 25, 2026
Were you intending to resubmit a PR for this subject? If not, I can offer to do this, based on your work in this PR and the comments received. |
legendecas
commented
Feb 25, 2026
@MikeMcC399 please feel free to go ahead! |
MikeMcC399
commented
Feb 26, 2026
Thanks for your feedback! I didn't want to duplicate anything that you already had planned. I've submitted #62002 now for this documentation change. |
Refs: #61776 (comment)
url.resolvewas deprecated but the deprecation was revoked in #37784. Given thaturl.resolvedepends on a deprecated insecure API, it should also be deprecated.