Skip to content

Migrate to request to axios and moment with Luxon #1

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
wants to merge 4 commits into
base: master
Choose a base branch
from

Conversation

ppamorim
Copy link

@ppamorim ppamorim commented Nov 21, 2024

Note: A significant portion of the +2000 line changes are due to updates in package-lock.json.

This PR addresses the migration of outdated and deprecated libraries to improve security and keep the project updated with the current recommended libraries.

Migration from request

The request library, now deprecated and no longer receiving updates, has been replaced.
The library relied on tough-cookie:2.4.3, which contains a critical vulnerability: [CVE-2023-26136] (Severity: 9.8).
Replacing request resolves dependency on this vulnerable library.

Migration from moment to Luxon

moment is also deprecated and has been replaced with Luxon to take advantage of its modern, immutable date-time API and improved performance.
Luxon offers better support for time zones and localization while aligning with modern JavaScript standards.

@ppamorim ppamorim force-pushed the feature/axios branch 2 times, most recently from 0b95be0 to a55469d Compare November 22, 2024 00:57
@ppamorim ppamorim changed the title Migrate to request to axios Migrate to request to axios and moment with Luxon Nov 22, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant