Add support for Microsoft Entra ID

[nickclyde]

PULL REQUEST

Summary

This pull request introduces support for Microsoft Entra ID alongside the existing Keycloak integration.

Authentication Enhancements

• Added support for Microsoft Entra ID as an identity provider, including a new provider configuration in src/auth.ts and dynamic provider selection based on the NEXT_PUBLIC_AUTH_PROVIDER environment variable. [1] [2] [3]
• Updated the handleClick function in src/app/(pages)/landingPage/landingPage.tsx to dynamically use the selected identity provider for sign-in.

Configuration Updates

• Expanded the .env.sample file to include new environment variables for Microsoft Entra ID (AUTH_MICROSOFT_ENTRA_ID_ID, AUTH_MICROSOFT_ENTRA_ID_SECRET, AUTH_MICROSOFT_ENTRA_ID_ISSUER) and reorganized authentication-related settings for clarity.

Documentation Improvements

• Updated docs/deployment.md to include instructions for configuring identity providers, detailing how to set the NEXT_PUBLIC_AUTH_PROVIDER environment variable and the required settings for Keycloak and Microsoft Entra ID.

Related Issue

Fixes #595 (https://linear.app/skylight-cdc/issue/QUE-280/add-support-for-microsoft-entra-id)

Additional Information

Something to keep an eye out for, I don't think this is a breaking change but good to keep in mind if we upgrade AuthJS: nextauthjs/next-auth#12616

If you want to test this locally, follow this guide with your Skylight Azure account, let me know if you need one set up

[codecov-commenter]

Codecov Report

Attention: Patch coverage is 57.14286% with 6 lines in your changes missing coverage. Please review.

Project coverage is 66.08%. Comparing base (d7974c6) to head (dbc8b3a).
Report is 2 commits behind head on main.

Files with missing lines	Patch %	Lines
src/auth.ts	57.14%	6 Missing ⚠️

Additional details and impacted files

@@            Coverage Diff             @@
##             main     #596      +/-   ##
==========================================
- Coverage   66.10%   66.08%   -0.03%     
==========================================
  Files         104      104              
  Lines        4706     4720      +14     
  Branches     1114     1094      -20     
==========================================
+ Hits         3111     3119       +8     
- Misses       1583     1589       +6     
  Partials       12       12              

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:

• ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
• 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

[robertandremitchell]

lgtm! @nickclyde , I don't have Microsoft creds; do you think they're worth getting? Happy to request, but wondering if it in favor of not holding up the ticket you could maybe do a run through of the startup for Microsoft at a parking lot?

[fzhao99]

works like a charm! just a flag to drop a slack message for folks to update their local setup after this goes in since we're renaming the env vars. otherwise would bork the local sign in flow