🔧 Update Discord OAuth to prioritize DISCORD_REDIRECT_URI over VITE_DISCORD_REDIRECT_URI and adjust error messages accordingly

This commit is contained in:
2026-01-30 05:26:34 +00:00
parent 6c952bbf48
commit 1f73b9744a
2 changed files with 3 additions and 5 deletions

View File

@@ -74,7 +74,7 @@ app.post('/oauth/token', async (req, res) => {
if (provider === 'discord') { if (provider === 'discord') {
const clientId = process.env.VITE_DISCORD_CLIENT_ID; const clientId = process.env.VITE_DISCORD_CLIENT_ID;
const clientSecret = process.env.DISCORD_CLIENT_SECRET; const clientSecret = process.env.DISCORD_CLIENT_SECRET;
const redirectUri = process.env.VITE_DISCORD_REDIRECT_URI; const redirectUri = process.env.DISCORD_REDIRECT_URI || process.env.VITE_DISCORD_REDIRECT_URI;
if (!clientId || !clientSecret || !redirectUri) { if (!clientId || !clientSecret || !redirectUri) {
logger.warn('Discord OAuth not configured', { logger.warn('Discord OAuth not configured', {
@@ -85,7 +85,7 @@ app.post('/oauth/token', async (req, res) => {
return res.status(503).json({ return res.status(503).json({
error: 'Discord OAuth not configured', error: 'Discord OAuth not configured',
message: message:
'Set VITE_DISCORD_CLIENT_ID, DISCORD_CLIENT_SECRET, and VITE_DISCORD_REDIRECT_URI environment variables' 'Set VITE_DISCORD_CLIENT_ID, DISCORD_CLIENT_SECRET, and DISCORD_REDIRECT_URI environment variables'
}); });
} }

View File

@@ -203,9 +203,7 @@ export function createAuthRouter({ secret, adminPassword } = {}) {
if (provider === 'discord') { if (provider === 'discord') {
const clientId = process.env.VITE_DISCORD_CLIENT_ID; const clientId = process.env.VITE_DISCORD_CLIENT_ID;
const clientSecret = process.env.DISCORD_CLIENT_SECRET; const clientSecret = process.env.DISCORD_CLIENT_SECRET;
const redirectUri = process.env.VITE_DISCORD_REDIRECT_URI; const redirectUri = process.env.DISCORD_REDIRECT_URI || process.env.VITE_DISCORD_REDIRECT_URI;
if (!clientId || !clientSecret) {
console.error('Discord OAuth not configured:', { console.error('Discord OAuth not configured:', {
hasClientId: !!clientId, hasClientId: !!clientId,
hasClientSecret: !!clientSecret hasClientSecret: !!clientSecret