acmpesuecc / acmpesuecc/SCB-Vulnerability-Analysis

Enhance file size validation and limits

Abierto
#1 29 comentarios 0 reacciones 0 asignados Ver en GitHub
Bounty: 50 documentation enhancement good first issue
Lenguaje dominante
Python
Estrellas
0
Forks
3
Métricas de merge de PR
Sin PR fusionados en 30 d

Descripción

Currently, the GUI accepts APK and IPA files without any size restrictions, which can lead to several problems:

- Large files can cause memory issues during upload
- Excessive storage usage in MinIO
- Potential timeout issues during processing
- Poor user experience with no upload progress indication

Technical Requirements:

- Backend Validation (Python/FastAPI):
- Add configurable file size limit (default: 100MB, configurable via environment variable)
- Implement file size check in the /upload endpoint before MinIO upload
- Add proper error handling with descriptive messages
- Log file size information for monitoring

Frontend Enhancement (HTML/JavaScript):

- Display file size in the upload interface
- Add client-side size validation before upload
- Show upload progress bar for large files
- Display remaining upload time estimate

Configuration:

- Add MAX_FILE_SIZE_MB environment variable
- Update Helm charts to include the new environment variable
- Document the configuration option

Acceptance Criteria:

[ ] File size validation implemented in backend (configurable via env var)
[ ] Clear error messages for oversized files ("File too large. Maximum size: 100MB")
[ ] Frontend displays file size and upload progress
[ ] Client-side validation prevents unnecessary uploads
[ ] Documentation updated with size limits and configuration
[ ] Helm charts updated with new environment variable
[ ] Tested with files of various sizes (small, medium, large, oversized)

Files to Modify:

- gui/webapp/main.py - Backend validation
- gui/webapp/templates/index.html - Frontend enhancements
- gui/values.yaml - Helm configuration
- gui/templates/webapp-deployment.yaml - Environment variables
- documentation/markdown/gui-setup.md - Documentation

GLHF :)

Guía de contribución

Abrir la guía de contribución

Evaluación

Este issue todavía no se ha evaluado.

Recibe los nuevos issues en tu correo

Un resumen breve de issues de GitHub para principiantes.