Debugging Google OAuthentication and Adding connection to supabase by MayankRai89 · Pull Request #69 · arpit2006/CampussCompass

MayankRai89 · 2026-08-29T04:50:03Z

Description

This pull request improves user authentication reliability, integrates the official Google Identity Services (GIS) SDK on login/registration pages, resolves a token verification mismatch in the authentication controller, and establishes immediate database connection logging on application startup.

Key Changes:

Google Identity Services Integration:
- Integrated the official Google Sign-In SDK (https://accounts.google.com/gsi/client) into views/login.ejs and views/register.ejs.
- Added automatic credential callback handling (handleGoogleCredentialResponse) to submit genuine ID tokens to /auth/google.
- Retained developer prompt fallback for environments where client IDs or origins are not yet configured.
OAuth Token Verification Fix:
- Updated controllers/authController.js with smart token inspection (token.split('.').length === 3) to verify genuine Google JWTs with OAuth2Client.verifyIdToken() while safely decoding mock/development payloads without throwing Wrong number of segments in token errors.
- Enforced session persistence via req.session.save() before performing redirect responses.
Startup Database Connection:
- Updated app.js to initialize and synchronize the Sequelize connection (Supabase / SQLite) immediately on server boot outside serverless environments, providing immediate feedback in console logs.

Related Issue

Fixes Google authentication failure when clicking "Continue with Google"
Resolves lazy database connection confusion on server startup

Type of Change

Bug fix (non-breaking change which fixes an issue)
New feature (non-breaking change which adds functionality)
Design/CSS styling changes
Documentation update (README, CONTRIBUTING, etc.)

How Has This Been Tested?

Automated Unit & Integration Tests:
- Ran npm test across the full test suite (test/**/*.test.js) — 43 / 43 tests passing (100%).
Supabase PostgreSQL Verification:
- Verified live connection, schema synchronization, and user lookup in production mode against remote Supabase database.
Manual Flow Testing:
- Tested /auth/google with both Google JWT and developer mock payloads, verifying successful user creation/login and redirect to /dashboard / /profile/setup.

Checklist:

My code follows the coding guidelines of this project.
I have performed a self-review of my own code.
I have commented my code, particularly in hard-to-understand areas.
My changes generate no new warnings or console errors.

…n and skill definition files.

…ussCompass

…trollers, and integration tests

…e, routes, and UI components

…rvice, and dashboard view

…acking, streak management, and calendar export functionality

…ing, and iCalendar export functionality

…and iCal generation support

… navigation, modals, and tab switching functionality

…heduling, completion tracking, and calendar integration

…edule management

…ss monitoring

…ient-side JavaScript functionality

…are, friend management controllers, and dashboard views with testing support.

…ntegration

…on, OAuth support, and database initialization on startup

vercel · 2026-08-29T04:50:08Z

@MayankRai89 is attempting to deploy a commit to the arpit2006's projects Team on Vercel.

A member of the Team first needs to authorize it.

github-actions · 2026-08-29T04:50:11Z

🎉 Thank you @MayankRai89 for submitting a Pull Request!

We're excited to review your contribution.

Before Review

✅ Ensure all CI checks pass
✅ Complete the PR template
✅ Link the related issue

📖 Before your PR is reviewed, check the PR Checklist in our Contributing Guide.

Happy Contributing! 🚀

github-actions · 2026-08-29T04:50:11Z

📏 Pull Request Size Report

🔴 Classification: Huge

Metric	Value
Files Changed	72
Lines Added	+5816
Lines Deleted	-63
Total Changes	5879

Size Meter

██████████

Recommendation

🚨 This PR is very large. Splitting it into multiple PRs is strongly recommended.

Generated automatically by PR Size Analyzer.

github-actions · 2026-08-29T04:50:12Z

👋 Welcome Back!

Hi @MayankRai89,

Thanks for contributing to CampusCompass again!

We appreciate your continued support ❤️

🗄 Database Changes

Please Verify

Migration tested
Existing data remains compatible
Rollback possible
Queries updated

🌟 General Tips

Keep commits focused
Respond to review comments
Ask questions if you're unsure
Make sure CI passes before requesting review

Happy Coding! 🚀

Generated automatically by Contributor Learning Assistant.

github-actions · 2026-08-29T04:50:12Z

📋 Pull Request Preview

📊 Summary

Metric	Value
📄 Files Changed	72
➕ Additions	+5816
➖ Deletions	-63
🔄 Total Changes	5879
📦 PR Size	🔴 Very Large

📂 File Breakdown

Category	Files
🎨 Frontend	0
⚙️ Backend	0
🧪 Tests	4
📚 Documentation	40
🤖 GitHub Actions	0
📦 Other	28

⏱️ Estimated Review Time

147 minute(s)

✅ Reviewer Checklist

⚠️ Large Pull Request

This PR contains 5879 changed lines.

Consider splitting it into smaller pull requests for easier review.

🤖 Generated automatically by the PR Preview Bot

github-actions · 2026-08-29T04:50:12Z

🛡️ Critical File Change Detector

This Pull Request modifies important project files.

Changed Files

• .agents/skills/supabase-postgres-best-practices/references/security-privileges.md (Security)
• .agents/skills/supabase-postgres-best-practices/references/security-rls-basics.md (Security)
• .agents/skills/supabase-postgres-best-practices/references/security-rls-performance.md (Security)
• .env.example (Environment)
• controllers/authController.js (Security)
• package-lock.json (Dependencies)
• package.json (Dependencies)
• routes/auth.js (Security)
• services/securityMiddleware.js (Security)
• test/google-auth.test.js (Security)

Labels Added

🏷️ security-review
🏷️ environment
🏷️ dependencies

Reviewer Checklist

Generated automatically by Critical File Change Detector.

github-actions · 2026-08-29T04:50:16Z

⚠️ Suspicious Generated Code Detected

This PR contains patterns commonly associated with low-quality AI-generated submissions.

🔍 Detected Signals

Excessive comments in .agents/skills/supabase-postgres-best-practices/CHANGELOG.md
Generated-style separators in .agents/skills/supabase-postgres-best-practices/SKILL.md
Large insertion in .agents/skills/supabase-postgres-best-practices/references/_contributing.md (+170)
Excessive comments in .agents/skills/supabase-postgres-best-practices/references/_contributing.md
Generated-style separators in .agents/skills/supabase-postgres-best-practices/references/_contributing.md
Excessive comments in .agents/skills/supabase-postgres-best-practices/references/_sections.md
Excessive comments in .agents/skills/supabase-postgres-best-practices/references/conn-pooling.md
Excessive comments in .agents/skills/supabase-postgres-best-practices/references/data-n-plus-one.md
Excessive comments in .agents/skills/supabase-postgres-best-practices/references/data-pagination.md
Many helper functions added in .agents/skills/supabase-postgres-best-practices/references/security-rls-performance.md
Excessive comments in .agents/skills/supabase/CHANGELOG.md
Large insertion in .agents/skills/supabase/SKILL.md (+149)
Excessive comments in .agents/skills/supabase/SKILL.md
Many helper functions added in .agents/skills/supabase/SKILL.md
Excessive comments in .agents/skills/supabase/assets/feedback-issue-template.md
Excessive comments in .agents/skills/supabase/references/skill-feedback.md
Excessive comments in app.js
Excessive comments in controllers/authController.js
Large insertion in controllers/socialController.js (+317)
Excessive comments in controllers/socialController.js
Large insertion in controllers/studySprintController.js (+223)
Large insertion in package-lock.json (+521)
Excessive comments in package-lock.json
Large insertion in public/js/main.js (+213)
Excessive comments in public/js/main.js
Many helper functions added in public/js/main.js
Excessive comments in services/securityMiddleware.js
Large insertion in services/studySprintService.js (+367)
Excessive comments in services/studySprintService.js
Many helper functions added in services/studySprintService.js
Large insertion in test/google-auth.test.js (+148)
Large insertion in test/studySprint.test.js (+144)
Large insertion in views/social.ejs (+267)
Many helper functions added in views/social.ejs
Large insertion in views/studysprint.ejs (+627)

This PR has been flagged for manual review.

This is not an automatic rejection. A maintainer will review the changes before making a decision.

🤖 Generated automatically by PR Quality Guard

…vices, scripts, and tests

…ynchronous read (arpit2006#63)

MayankRai89 added 19 commits August 18, 2026 23:11

feat: add comprehensive Supabase Postgres best practices documentatio…

2bb908a

…n and skill definition files.

Merge branches 'main' and 'main' of https://github.com/arpit2006/Camp…

48f1c94

…ussCompass

feat: implement Google OAuth authentication flow with UI, backend con…

d84e451

…trollers, and integration tests

feat: implement StudySprint feature with dedicated controller, servic…

9411f30

…e, routes, and UI components

feat: implement study sprint tracking system including controller, se…

d9c0e8c

…rvice, and dashboard view

feat: implement StudySprint module with automated scheduling, task tr…

4d7e84f

…acking, streak management, and calendar export functionality

test: add unit tests for study sprint schedule generation, task track…

20729b6

…ing, and iCalendar export functionality

feat: implement StudySprint module with scheduling, streak tracking, …

3463f94

…and iCal generation support

feat: implement main.js with UI interactivity including GitHub stats,…

0a4a745

… navigation, modals, and tab switching functionality

reduce alert notification time

548cb22

feat: implement study sprint service and controller to manage task sc…

ee597c1

…heduling, completion tracking, and calendar integration

feat: create StudySprint dashboard view for progress tracking and sch…

6b1c66c

…edule management

feat: add study sprint dashboard view with streak tracking and progre…

ff90a0f

…ss monitoring

feat: add connet you linked in account in you social and profile

d50d617

feat: implement security middleware for rate limiting and add main cl…

2f2cd6d

…ient-side JavaScript functionality

feat: implement social networking features including security middlew…

866bed1

…are, friend management controllers, and dashboard views with testing support.

feat: create reusable header partial with font, icon, and analytics i…

1720bbc

…ntegration

feat: implement file-based session persistence using session-file-store

c8f0a70

feat: implement user authentication controller with login, registrati…

dc51b92

…on, OAuth support, and database initialization on startup

MayankRai89 requested a review from arpit2006 as a code owner August 29, 2026 04:50

github-actions Bot added database dependencies Pull requests that update a dependency file environment security-review labels Aug 29, 2026

github-actions Bot added documentation Improvements or additions to documentation UI frontend backend needs-human-review suspected-ai-slop labels Aug 29, 2026

MayankRai89 added 2 commits August 29, 2026 10:31

fix(lint): resolve ESLint errors and warnings across controllers, ser…

6404dd0

…vices, scripts, and tests

perf(dashboard): migrate synchronous disk I/O in roadmap parser to as…

117fa85

…ynchronous read (arpit2006#63)

Navigation Menu

Debugging Google OAuthentication and Adding connection to supabase - #69

Debugging Google OAuthentication and Adding connection to supabase#69
MayankRai89 wants to merge 21 commits into
arpit2006:mainfrom
MayankRai89:main

MayankRai89 commented Aug 29, 2026 •

edited

Loading

Uh oh!

vercel Bot commented Aug 29, 2026

Uh oh!

github-actions Bot commented Aug 29, 2026

Uh oh!

github-actions Bot commented Aug 29, 2026 •

edited

Loading

Uh oh!

github-actions Bot commented Aug 29, 2026

Uh oh!

github-actions Bot commented Aug 29, 2026 •

edited

Loading

Uh oh!

github-actions Bot commented Aug 29, 2026

Uh oh!

github-actions Bot commented Aug 29, 2026 •

edited

Loading

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

1 participant

Conversation

MayankRai89 commented Aug 29, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Description

Key Changes:

Related Issue

Type of Change

How Has This Been Tested?

Checklist:

Uh oh!

vercel Bot commented Aug 29, 2026

Uh oh!

github-actions Bot commented Aug 29, 2026

Before Review

Uh oh!

github-actions Bot commented Aug 29, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

📏 Pull Request Size Report

Size Meter

Recommendation

Uh oh!

github-actions Bot commented Aug 29, 2026

👋 Welcome Back!

🗄 Database Changes

🌟 General Tips

Uh oh!

github-actions Bot commented Aug 29, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

📋 Pull Request Preview

📊 Summary

📂 File Breakdown

⏱️ Estimated Review Time

✅ Reviewer Checklist

⚠️ Large Pull Request

Uh oh!

github-actions Bot commented Aug 29, 2026

🛡️ Critical File Change Detector

Changed Files

Labels Added

Reviewer Checklist

Uh oh!

github-actions Bot commented Aug 29, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

⚠️ Suspicious Generated Code Detected

🔍 Detected Signals

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

1 participant

MayankRai89 commented Aug 29, 2026 •

edited

Loading

github-actions Bot commented Aug 29, 2026 •

edited

Loading

github-actions Bot commented Aug 29, 2026 •

edited

Loading

github-actions Bot commented Aug 29, 2026 •

edited

Loading