Ревью 🔍

Несколько ключевых моментов, которые помогут в процессе проверки:

⏱️ Уровень сложности ревью: 4 🔵🔵🔵🔵⚪

🧪 Нет релевантных тестов

🔒 Проблемы с безопасностью

Sensitive information exposure:
Эндпоинт get_confluence_status (строки 598-616) возвращает api_token в ответе. Это критическая уязвимость, так как токен является секретом и не должен передаваться на клиент. Даже если фронтенд его маскирует, сам факт передачи по сети недопустим.

⚡ Рекомендовано к рассмотрению

Утечка секретов

Эндпоинт /api/sources/confluence/status возвращает api_token в открытом виде, что представляет серьезную угрозу безопасности.

"api_token": api_token,    # ← отдаём (фронтенд сам заменит на ••••)

Потенциальная уязвимость

Используется BeautifulSoup для парсинга ненадежного HTML из Confluence без явной санитизации, что может привести к XSS при отображении.

)
client.delete(
    collection_name=collection,

Небезопасное хранение

Ключи доступа Bitrix24 и Confluence сохраняются в настройках напрямую, без шифрования или использования безопасного хранилища.

kb_config["last_sync"] = now
settings.knowledge_sources["bitrix24_kb"] = kb_config
save_settings(settings)

artkod

Популярные репозитории