ProjectTech4DevAI / ProjectTech4DevAI/kaapi-backend
Support API key configuration for Zerox in multi-tenant setup
Chưa có ai nhận issue này.
- Ngôn ngữ chính
- Python
- Star
- 18
- Fork
- 10
- Merge trung bình
- 2 ngày 20 giờ
- Pull request đã merge (30 ngày)
- 14
Mô tả
We are currently using Zerox for document transformation in our system. However, Zerox only supports reading the API key from environment variables.
Our platform is multi-tenant, and the API key is scoped per project. This makes it problematic to rely solely on environment variables for API key management.
We already opened a PR in the Zerox repo to add support for passing the API key explicitly instead of always fetching it from environment variables. However, in case we don’t receive a response from the maintainers, we will need to either:
- Move to another tool that supports this flexibility, or
- Implement a workaround where each Celery worker handles project-specific API keys.
Context:
Each Celery worker runs in a separate process, which means separate memory and environment, so we can set api_key in env.
Test:
Once fixed this add unit tests for transfomer
Hướng dẫn đóng góp
Bắt đầu từ đâu
- Đọc hết issue, rồi đọc hướng dẫn đóng góp của dự án.
- Bình luận trên issue rằng bạn sẽ nhận — tránh hai người làm cùng một việc.
- Fork repository và làm thay đổi trên một nhánh.
- Mở pull request có tham chiếu số hiệu của issue.
Hướng nghiên cứu
Bắt đầu bằng cách xác định integration của Zerox và khu vực unit-test của transformer trong Python backend. Kiểm tra xem upstream API-key PR có sẵn hay không, sau đó xác định cách các project-scoped keys phải được truyền đến transformer; được coi là hoàn tất khi cách tiếp cận đã chọn hoạt động cho việc sử dụng multi-tenant và có unit tests cho transformer.
Do mô hình lập chỉ mục viết ra từ nội dung của issue.
Đánh giá
- Công nghệ
- python
- Lĩnh vực
- backend
- Loại issue
- Tính năng
- Độ khó
- 4/5
- Thời gian dự kiến
- 3-5 ngày
- Mức độ hoạt động
- Đình trệ
- Độ rõ ràng
- Cần làm rõ
- Mức phù hợp với người mới
- 35/100