search

cogentapps / chat-with-gpt

An open-source ChatGPT app with a voice
https://www.chatwithgpt.ai
MIT License
2.3k stars 493 forks source link

introduce env variable DISABLE_RATE_LIMIT to disable rate limit #98

Closed schmidp closed 1 year ago

schmidp commented 1 year ago

We locally run into rate limiting. since we don't want to specify a rate limiting within the company at all, especially because some NAT clients use the same IP, we introduced this variable with minimal changes.

Frajder commented 1 year ago

I ran into a similar problem, so I just increased the limit. I don't think it's a good idea to completely disable the limit because there is a good reason for having it. My suggestion would be to make this ENV variable represent the number of requests allowed in 15 minutes, with 0 representing unlimited requests as if we had set this DISABLE_RATE_LIMIT variable to true. If this variable is not specified, the default would be 100 as it is now. This would allow for more flexibility, in my opinion.

schmidp commented 1 year ago

@Frajder you are of course right, I was just lazy :)

Better now?

Frajder commented 1 year ago

Yes it's better, thank you!