astronomer / ask-astro

An end-to-end LLM reference implementation providing a Q&A interface for Airflow and Astronomer
https://ask.astronomer.io/
Apache License 2.0
192 stars 47 forks source link

Add sveltekit-rate-limiter & Fix Rate Limiter Issue #280

Closed davidgxue closed 8 months ago

davidgxue commented 8 months ago

Description

Technical Changes

Tests

Setup:

  1. Test cookie rate limiter. This should trigger after 5 requests using the same browser

https://github.com/astronomer/ask-astro/assets/26350341/0431cae3-b38d-479c-9b87-b1e7667286bd

  1. Using incognito, we can continue to send requests without being blocked. But after 5 more requests we hit our IP based rate limit

    image
  2. Now, just to be safe, I used a different computer with VPN and reconnected to the same dev website. And confirmed that I can continue to send additional requests

closes #268

cloudflare-workers-and-pages[bot] commented 8 months ago

Deploying with  Cloudflare Pages  Cloudflare Pages

Latest commit: cb04c91
Status: ✅  Deploy successful!
Preview URL: https://af57f4ba.ask-astro.pages.dev
Branch Preview URL: https://fix-rate-limiter-issue.ask-astro.pages.dev

View logs