Merge pull request 'fix: rate limiter was blocking all /api/cats reads after 10 requests' (#3) from claude/sleepy-shirley-adfe57 into main
Reviewed-on: #3
This commit was merged in pull request #3.
This commit is contained in:
@@ -33,10 +33,11 @@ function AnimatedRoutes() {
|
||||
if (el) {
|
||||
el.classList.remove('page-enter-active');
|
||||
el.classList.add('page-enter');
|
||||
requestAnimationFrame(() => {
|
||||
const rafId = requestAnimationFrame(() => {
|
||||
el.classList.remove('page-enter');
|
||||
el.classList.add('page-enter-active');
|
||||
});
|
||||
return () => cancelAnimationFrame(rafId);
|
||||
}
|
||||
}, [location.pathname]);
|
||||
|
||||
|
||||
@@ -268,7 +268,6 @@ export default function ProfilePage() {
|
||||
<div className="divider mb-4" />
|
||||
|
||||
<div>
|
||||
<input type="file" ref={fileInputRef} onChange={onFileChange} accept="image/*" className="hidden" />
|
||||
<Button
|
||||
variant="outline"
|
||||
onClick={() => fileInputRef.current?.click()}
|
||||
|
||||
@@ -54,7 +54,11 @@ async function main() {
|
||||
standardHeaders: true,
|
||||
legacyHeaders: false,
|
||||
});
|
||||
app.use('/api/cats', uploadLimiter);
|
||||
// Rate-limit only cat uploads (POST), not reads
|
||||
app.use('/api/cats', (req, _res, next) => {
|
||||
if (req.method !== 'POST') return next();
|
||||
return uploadLimiter(req, _res, next);
|
||||
});
|
||||
|
||||
app.use('/uploads', express.static(path.join(__dirname, '..', 'uploads')));
|
||||
|
||||
|
||||
Reference in New Issue
Block a user