Conversation
- Refactored `Settings` class to delegate Redis URL and API key retrieval to dedicated services. - Removed deprecated authentication middleware and security headers middleware for cleaner codebase. - Introduced utility functions for container operations to reduce code duplication in container management. - Updated container execution logic to enhance readability and maintainability.
- Added validation to restrict file uploads based on allowed file types, raising a 415 error for disallowed types. - Expanded the list of allowed file extensions in the `Settings` class to include various document, image, and script formats. - Introduced unit tests to verify the functionality of file type validation and ensure blocked extensions are correctly handled.
- Removed deprecated `max_cpu_quota` and session container reuse settings from the `Settings` class for cleaner configuration. - Consolidated API key extraction and client IP retrieval into shared utility functions for improved code reuse and maintainability. - Eliminated unused event classes and methods to reduce code clutter and enhance readability. - Updated output processing to utilize the new filename sanitization method, ensuring consistent handling of filenames across the application.
…entation - Removed the deprecated `MAX_CPU_QUOTA` setting from the `.env.example` file. - Updated the `CONFIGURATION.md` documentation to reflect the change, replacing `MAX_CPU_QUOTA` with `MAX_CPUS` and adjusting its default value.
- Added `pyproject.toml` for pytest configuration with asyncio settings. - Updated dependencies in `requirements.txt` to newer versions for `uvicorn`, `pydantic-settings`, `httpx`, `redis`, `minio`, `pytest-asyncio`, `structlog`, `Unidecode`, and `locust`. - Updated GitHub Actions workflow to use the latest version of `setup-python`. - Updated Dockerfiles for C/C++, Go, and R to use newer base images.
- Changed the Dockerfile ENTRYPOINT to use a temporary directory for GOCACHE instead of a mounted data directory. - Modified the command in runner.py to limit the search depth when finding files, improving performance and accuracy in file listing.
File extension validation was moved to Settings class but the security property was still passing these fields to SecurityConfig. 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
The redis library's ping() method has an ambiguous return type (Awaitable[bool] | bool) that mypy can't resolve. Added type ignore comments to suppress these false positives. 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
Cleanup: Simplify configuration, add file type restrictions, update dependencies
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Changes (7 commits)
Test plan