Skip to content

Commit

Permalink
Feat: Provide more details for 429 error msg
Browse files Browse the repository at this point in the history
- Added better error code for when rate limit exceeded including
consumed/remaining points, reset date and retry-after seconds
  • Loading branch information
simonha9 committed May 25, 2024
1 parent e5c8719 commit 115204e
Showing 1 changed file with 11 additions and 9 deletions.
20 changes: 11 additions & 9 deletions apps/api/src/controllers/auth.ts
Original file line number Diff line number Diff line change
@@ -1,12 +1,12 @@
import { parseApi } from "../../src/lib/parseApi";
import { getRateLimiter, } from "../../src/services/rate-limiter";
import { getRateLimiter, } from "../../src/services/rate-limiter";
import { AuthResponse, RateLimiterMode } from "../../src/types";
import { supabase_service } from "../../src/services/supabase";
import { withAuth } from "../../src/lib/withAuth";
import { RateLimiterRedis } from "rate-limiter-flexible";
import { setTraceAttributes } from '@hyperdx/node-opentelemetry';

export async function authenticateUser(req, res, mode?: RateLimiterMode) : Promise<AuthResponse> {
export async function authenticateUser(req, res, mode?: RateLimiterMode): Promise<AuthResponse> {
return withAuth(supaAuthenticateUser)(req, res, mode);
}
function setTrace(team_id: string, api_key: string) {
Expand All @@ -18,7 +18,7 @@ function setTrace(team_id: string, api_key: string) {
} catch (error) {
console.error('Error setting trace attributes:', error);
}

}
export async function supaAuthenticateUser(
req,
Expand Down Expand Up @@ -97,7 +97,7 @@ export async function supaAuthenticateUser(
team_id: team_id,
plan: plan
}
switch (mode) {
switch (mode) {
case RateLimiterMode.Crawl:
rateLimiter = getRateLimiter(RateLimiterMode.Crawl, token, subscriptionData.plan);
break;
Expand Down Expand Up @@ -126,9 +126,11 @@ export async function supaAuthenticateUser(
await rateLimiter.consume(iptoken);
} catch (rateLimiterRes) {
console.error(rateLimiterRes);
const secs = Math.round(rateLimiterRes.msBeforeNext / 1000) || 1;
const retryDate = new Date(Date.now() + rateLimiterRes.msBeforeNext);
return {
success: false,
error: "Rate limit exceeded. Too many requests, try again in 1 minute.",
error: `Rate limit exceeded for mode ${rateLimiter.keyPrefix}. Consumed points: ${rateLimiterRes.consumedPoints}, Remaining points: ${rateLimiterRes.remainingPoints}. Please retry after ${secs}s, resets at ${retryDate}`,
status: 429,
};
}
Expand All @@ -155,9 +157,9 @@ export async function supaAuthenticateUser(
normalizedApi = parseApi(token);

const { data, error } = await supabase_service
.from("api_keys")
.select("*")
.eq("key", normalizedApi);
.from("api_keys")
.select("*")
.eq("key", normalizedApi);

if (error || !data || data.length === 0) {
return {
Expand All @@ -170,7 +172,7 @@ export async function supaAuthenticateUser(
subscriptionData = data[0];
}

return { success: true, team_id: subscriptionData.team_id };
return { success: true, team_id: subscriptionData.team_id };
}

function getPlanByPriceId(price_id: string) {
Expand Down

0 comments on commit 115204e

Please sign in to comment.