feat(concurrency): added concurrency for increased performance

[StanGirard]

Added concurrency 4 for celery and 6 for backend

[vercel]

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name	Status	Preview	Comments	Updated (UTC)
docs	✅ Ready (Inspect)	Visit Preview	💬 Add feedback	Sep 17, 2023 8:35pm
quivrapp	✅ Ready (Inspect)	Visit Preview	💬 Add feedback	Sep 17, 2023 8:35pm

[github-actions]

Risk Level 2 - /home/runner/work/quivr/quivr/backend/models/databases/supabase/notifications.py

The get_notifications_by_chat_id function has been modified to only return notifications from the last 5 minutes. This could potentially cause issues if older notifications are expected elsewhere in the code. If this is the intended behavior, consider renaming the function to reflect this, for example get_recent_notifications_by_chat_id. Also, the datetime format string is hard-coded which could lead to issues if the format changes. Consider defining it as a constant at the top of your file.

DATETIME_FORMAT = \"%Y-%m-%d %H:%M:%S.%f\"

# later in the code
five_minutes_ago = (datetime.now() - timedelta(minutes=5)).strftime(DATETIME_FORMAT)

---

Risk Level 4 - /home/runner/work/quivr/quivr/backend/celery_worker.py

The openai_api_key is being passed around in plain text which is a security risk. Consider using environment variables or a secure way to handle sensitive data like API keys. Also, the CELERY_BROKER_URL is being used without any validation. It's better to validate this URL before using it. Here is a suggestion:

import os
from urllib.parse import urlparse

CELERY_BROKER_URL = os.getenv(\"CELERY_BROKER_URL\", \"\")

parsed_url = urlparse(CELERY_BROKER_URL)
if not all([parsed_url.scheme, parsed_url.netloc]):
    raise ValueError(f\"Invalid broker URL: {CELERY_BROKER_URL}\")

---

⏰🔒🔧

---

Powered by Code Review GPT