Skip to content

Conversation

@BartoszBlizniak
Copy link
Member

This pull request replaces the use of axios and node-fetch with the native fetch API throughout the codebase to address Node.js 24 deprecation warnings and reduce bundle size. It also updates related code to support the new HTTP request approach, including adding timeout handling and updating error logging. The ESLint configuration is updated to recognize new global variables, and unused dependencies are removed.

Migration to native fetch and HTTP handling improvements:

  • Replaced all usage of axios and node-fetch with the native fetch API, including implementing a custom fetchWithTimeout function to provide timeout support and updating all HTTP request logic in src/oidc-auth.js and src/download-cli.js. This fixes Node.js 24 url.parse() deprecation warnings and reduces bundle size. [1] [2] [3] [4] [5] [6] [7]
  • Improved HTTP error logging to handle the native fetch API's response and request objects, including better formatting of headers and request details.

Dependency and configuration updates:

  • Removed axios and node-fetch from package.json dependencies since they are no longer needed.
  • Updated ESLint configuration to add clearTimeout and AbortController as read-only globals, supporting the new fetch-based code.

Copilot AI review requested due to automatic review settings December 23, 2025 10:04
@BartoszBlizniak BartoszBlizniak merged commit 74f3999 into master Dec 23, 2025
22 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

2 participants