Skip to content

Fix OAuth client_id mismatch causing authentication failures - #111

Merged
github-actions[bot] merged 1 commit into
mainfrom
fix/oauth-credentials-mismatch
Oct 8, 2025
Merged

github-actions[bot] merged 1 commit into
mainfrom
fix/oauth-credentials-mismatch

Conversation

@ishaileshpant

Copy link
Copy Markdown
Collaborator

🐛 Fix OAuth Authentication Failure

Problem

Users were getting authentication failures with the error:

The client_id and/or client_secret passed are incorrect.

Root Cause

The frontend was using a hardcoded that didn't match the GitHub App settings:

  • Frontend (index.html): Ov23liX9YTDE0e7OaBLs
  • GitHub App: Iv23liU5MlIMYzcpLpop
  • Vercel Environment: Iv23liU5MlIMYzcpLpop

Solution

Updated the hardcoded in the frontend to match the GitHub App:

Fixed Frontend Configuration

  • Updated window.GITHUB_CONFIG.clientId in index.html
  • Changed from Ov23liX9YTDE0e7OaBLs to Iv23liU5MlIMYzcpLpop
  • Now matches the GitHub App and Vercel environment variables

Code Changes

// Before (incorrect)
window.GITHUB_CONFIG = {
    clientId: 'Ov23liX9YTDE0e7OaBLs',
    isConfigured: true
};

// After (correct)
window.GITHUB_CONFIG = {
    clientId: 'Iv23liU5MlIMYzcpLpop',
    isConfigured: true
};

Technical Details

  • File Modified: index.html
  • Change: Single line update to match GitHub App Client ID
  • Impact: Fixes OAuth authentication flow

Benefits

  • 🚀 Fixes OAuth authentication - Users can now log in successfully
  • 🔧 Consistent configuration - Frontend and backend use same client_id
  • 🛡️ Resolves authentication errors - No more client_id mismatch
  • 📈 Enables content submission - Login is required for content submission

Testing

  • ✅ OAuth authentication should now work
  • ✅ Users can log in with GitHub
  • ✅ Content submission will be accessible after login
  • ✅ No more 'client_id incorrect' errors

Related Issues

This fix resolves the OAuth authentication failure that was preventing users from logging in, which is required for content submission functionality.

Next Steps

After merging this PR:

  1. OAuth authentication will work correctly
  2. Users can log in with GitHub
  3. Content submission will be accessible
  4. The complete content submission flow will be functional

- Update hardcoded clientId in index.html to match GitHub App
- Change from 'Ov23liX9YTDE0e7OaBLs' to 'Iv23liU5MlIMYzcpLpop'
- Resolves 'Authentication Failed' error in OAuth callback
- Ensures frontend and backend use same client_id
@vercel

vercel Bot commented Oct 8, 2025

Copy link
Copy Markdown

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

Project Deployment Preview Comments Updated (UTC)
prepguides-dev Ready Ready Preview Comment Oct 8, 2025 3:18pm

@ishaileshpant

Copy link
Copy Markdown
Collaborator Author

/approve

@github-actions
github-actions Bot merged commit 2242350 into main Oct 8, 2025
11 checks passed
@github-actions

github-actions Bot commented Oct 8, 2025

Copy link
Copy Markdown
Contributor

🚀 Admin-Approved and Auto-Merged!

This PR has been automatically merged to main after admin approval by @ishaileshpant.

✅ **Status**: Merged and deployed
🔗 **Commit**: `2242350f148cc64a7e78ff72c164663f32ed083e`
📅 **Merged at**: 2025-10-08T18:26:24.344Z
👤 **Approved by**: Admin/Maintainer/Owner

Thank you for your contribution! 🎉

@ishaileshpant
ishaileshpant deleted the fix/oauth-credentials-mismatch branch October 13, 2025 18:19
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