the-control-group / authx

An authentication & authorization platform for service-oriented architectures.
MIT License
47 stars 6 forks source link

Update deps and switch to ES modules #625

Closed mike-marcacci closed 6 months ago

mike-marcacci commented 9 months ago

This has ended up becoming more substantial than originally intended. Several dependencies were abandoned or went completely different directions with major breaking changes.

This PR makes the following changes, many of them broken out into isolated commits when possible:

I explicitly avoided making substantial changes to the API or business logic – the purpose of this PR is strictly to bring this codebase up to date.

mike-marcacci commented 6 months ago

Note that complaints of "duplication" are primarily because tests and other high-duplication files were moved, causing them to be seen as "new code" by SonarCloud. We should ignore that gate for this PR.

sonarcloud[bot] commented 6 months ago

Quality Gate Failed Quality Gate failed

Failed conditions
8.2% Duplication on New Code (required ≤ 3%)

See analysis details on SonarCloud

nickcarenza commented 6 months ago

These changes look good to me. It's strange that sonar is categorizing moved code as duplication.

mike-marcacci commented 6 months ago

Thanks @nickcarenza! Just to clarify Sonar's behavior: it already considers this code as duplicate in main, but normally only reports on changes during PR scans, and because I moved them it considers them changed.