Skip to content

MavenV4 node24 upgrade#21628

Merged
Deekshitha2812 merged 6 commits intomasterfrom
users/deekshitha2812/MavenV4-Node24
Jan 6, 2026
Merged

MavenV4 node24 upgrade#21628
Deekshitha2812 merged 6 commits intomasterfrom
users/deekshitha2812/MavenV4-Node24

Conversation

@Deekshitha2812
Copy link
Contributor

@Deekshitha2812 Deekshitha2812 commented Dec 24, 2025

Context

Node24 migration
📌 AB#2340631


Task Name

MavenV4.


Description

Updates the task execution handlers (Node24 handler is added), types@node version is updated to 24.10.0 ,task-lib and type-script version are also updated to 5.2.4 and 5.7.2.
The task is build by using --BypassNpmudit.
The node make.js test --task MavenV4 is failed with error.

AssertionError [ERR_ASSERTION]: should not have written to stderr=(node:78088) [DEP0169] DeprecationWarning: `url.parse()` behavior is not standardized and prone to errors that have security implications. Use the WHATWG URL API instead. CVEs are not issued for `url.parse()` vulnerabilities.
(Use `node --trace-deprecation ...` to show where the warning was created)

      + expected - actual

      -false
      +true

      at Context.<anonymous> (C:\azure-pipelines-tasks\_build\Tasks\MavenV4\Tests\L0.js:149:13)
      at Generator.next (<anonymous>)
      at fulfilled (C:\azure-pipelines-tasks\_build\Tasks\MavenV4\Tests\L0.js:5:58)
      how to solve this error

this because of url.parse() API deprecation in node24 - Deprecated APIs | Node.js v24.12.0 Documentation.
updated the url.parse() with URL()


Risk Assessment (Low / Medium / High)

Assess the level of risk and provide reasoning (e.g., scope, impact, backward compatibility).


Change Behind Feature Flag ( No)

Can this change be behine feature flag, if not why?


Tech Design / Approach

  • Design has been written and reviewed.
  • Any architectural decisions, trade-offs, and alternatives are captured.

Documentation Changes Required (Yes/No)

Indicate whether related documentation needs to be updated.

  • User guides, API specs, system diagrams, or runbooks are updated.

Unit Tests Added or Updated (Yes / No)

Indicate whether unit tests were added or modified to reflect these changes.


Additional Testing Performed

List all other tests performed (manual or automated, including integration, regression, scenario tests, etc.).


Logging Added/Updated (Yes/No)

  • Appropriate log statements are added with meaningful messages.
  • Logging does not expose sensitive data.
  • Log levels are used correctly (e.g., info, warn, error).

Telemetry Added/Updated (Yes/No)

  • Custom telemetry (e.g., counters, timers, error tracking) is added as needed.
  • Events are tagged with proper metadata for filtering and analysis.
  • Telemetry is validated in staging or test environments.

Rollback Scenario and Process (Yes/No)

  • Rollback plan is documented.

Dependency Impact Assessed and Regression Tested (Yes/No)

  • All impacted internal modules, APIs, services, and third-party libraries are analyzed.
  • Results are reviewed and confirmed to not break existing functionality.

Checklist

  • Related issue linked (if applicable)
  • Task version was bumped — see versioning guide
  • Verified the task behaves as expected

@Deekshitha2812
Copy link
Contributor Author

/azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 3 pipeline(s).

@Deekshitha2812
Copy link
Contributor Author

/azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 3 pipeline(s).

@Deekshitha2812
Copy link
Contributor Author

/azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 3 pipeline(s).

@Deekshitha2812 Deekshitha2812 marked this pull request as ready for review December 30, 2025 10:52
Copy link
Contributor

@sanjuyadav24 sanjuyadav24 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please check for build failures

Image

@Deekshitha2812
Copy link
Contributor Author

Please check for build failures

Image

I think it occurred due to network issues, on retry the task was able to build

@Deekshitha2812
Copy link
Contributor Author

/azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 3 pipeline(s).

@Deekshitha2812 Deekshitha2812 merged commit 91c6206 into master Jan 6, 2026
14 checks passed
@Deekshitha2812 Deekshitha2812 deleted the users/deekshitha2812/MavenV4-Node24 branch January 6, 2026 03:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants