fix: update crawl and upload endpoints

[mamadoudicko]

• updae crawl and upload endpoints

[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 8, 2023 8:56am
quivrapp	✅ Ready (Inspect)	Visit Preview	💬 Add feedback	Sep 8, 2023 8:56am

[github-actions]

Risk Level 2 - /home/runner/work/quivr/quivr/backend/routes/crawl_routes.py

The code changes are generally safe. However, there are a few areas that could be improved for better code quality and maintainability:

1. The crawl_endpoint function is quite long and does a lot of things. It would be better to break it down into smaller functions for better readability and maintainability.

2. The if request.headers.get(\"Openai-Api-Key\"): check is done twice in the function. It would be better to do this check once and store the result in a variable.

3. The if not crawl_website.checkGithub(): and else: blocks have a lot of duplicated code. It would be better to refactor this to avoid code duplication.

4. The if remaining_free_space - file_size < 0: check could potentially lead to a negative number. It would be safer to use if file_size > remaining_free_space: instead.

---

Risk Level 2 - /home/runner/work/quivr/quivr/backend/routes/upload_routes.py

The code changes are generally safe. However, there are a few areas that could be improved for better code quality and maintainability:

1. The upload_file function is quite long and does a lot of things. It would be better to break it down into smaller functions for better readability and maintainability.

2. The if request.headers.get(\"Openai-Api-Key\"): check is done twice in the function. It would be better to do this check once and store the result in a variable.

3. The if openai_api_key is None: check is done twice in the function. It would be better to do this check once and store the result in a variable.

---

📚🔍🔄

---

Powered by Code Review GPT