Skip to content

fix(auth): load dotenv in auth.ts for Netlify SSR OAuth env vars#429

Merged
ViktorSvertoka merged 14 commits intodevelopfrom
sl/feat/db-optimization
Mar 28, 2026
Merged

fix(auth): load dotenv in auth.ts for Netlify SSR OAuth env vars#429
ViktorSvertoka merged 14 commits intodevelopfrom
sl/feat/db-optimization

Conversation

@LesiaUKR
Copy link
Copy Markdown
Collaborator

@LesiaUKR LesiaUKR commented Mar 28, 2026

fix(auth): load dotenv in auth.ts for Netlify SSR OAuth env vars

Summary by CodeRabbit

  • Chores
    • Improved environment variable loading to support .env file configuration during application startup.

@vercel
Copy link
Copy Markdown
Contributor

vercel bot commented Mar 28, 2026

The latest updates on your projects. Learn more about Vercel for GitHub.

1 Skipped Deployment
Project Deployment Actions Updated (UTC)
devlovers-net Ignored Ignored Preview Mar 28, 2026 6:01pm

Request Review

@coderabbitai
Copy link
Copy Markdown
Contributor

coderabbitai bot commented Mar 28, 2026

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro

Run ID: 41d6e44e-c23b-4daa-bff2-763bac9e42d8

📥 Commits

Reviewing files that changed from the base of the PR and between 9240c21 and b1bfbdc.

📒 Files selected for processing (1)
  • frontend/lib/env/auth.ts

📝 Walkthrough

Walkthrough

The frontend/lib/env/auth.ts module now imports and initializes dotenv at load time, ensuring environment variables from a .env file are loaded into process.env before the existing validation logic executes. No changes to exported entities or control flow logic.

Changes

Cohort / File(s) Summary
Environment Variable Loading
frontend/lib/env/auth.ts
Added dotenv import and dotenv.config() call to load environment variables from .env file at module initialization, prior to existing environment validation.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Possibly related PRs

Suggested reviewers

  • AM1007
  • ViktorSvertoka
  • liudmylasovetovs

Poem

🐰 A dotenv hop, so quick and neat,
Environment vars now skip a beat,
Before validation starts its run,
The config loads—our work is done! ✨

🚥 Pre-merge checks | ✅ 3
✅ Passed checks (3 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title clearly and specifically describes the main change: loading dotenv in auth.ts to address Netlify SSR OAuth environment variables. It directly corresponds to the changeset which adds dotenv import and initialization.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
📝 Generate docstrings
  • Create stacked PR
  • Commit on current branch
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch sl/feat/db-optimization

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@ViktorSvertoka ViktorSvertoka merged commit d44be7d into develop Mar 28, 2026
8 checks passed
@ViktorSvertoka ViktorSvertoka deleted the sl/feat/db-optimization branch March 28, 2026 17:20
@LesiaUKR LesiaUKR restored the sl/feat/db-optimization branch March 28, 2026 17:59
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.

2 participants