opengnsys
/
ogrepository
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

fix_checked_file #42

Merged
narenas merged 6 commits from fix_checked_file into main 2025-09-01 12:20:23 +02:00
Conversation 0 Commits 6 Files Changed 3
narenas commented 2025-09-01 12:19:44 +02:00
Collaborator

📝 Descripción

Este PR corrige la validación del archivo .info.checked en el endpoint create_torrent_sum para evitar errores cuando el archivo no existe.

🔧 Cambios realizados

  • ✅ Añadida validación para verificar la existencia del archivo .info.checked antes de proceder
  • ✅ Mejorado el manejo de errores con respuestas HTTP apropiadas (400 Bad Request)
  • ✅ Añadidos logs informativos para facilitar el debugging
  • ✅ Corregidos errores sintácticos menores en el código

🐛 Problema resuelto

Anteriormente, el endpoint create_torrent_sum no validaba si el archivo .info.checked existía antes de intentar crear los archivos auxiliares, lo que podía causar errores inesperados.

🧪 Pruebas

  • Verificado que el endpoint retorna error 400 cuando falta el archivo .info.checked
  • Verificado que el endpoint funciona correctamente cuando el archivo existe
  • Verificados los logs de systemd journal

📋 Checklist

  • Los cambios siguen las convenciones de código del proyecto
  • Se han añadido logs apropiados
  • La funcionalidad existente no se ve afectada
  • Los mensajes de error son descriptivos

🔗 Issues relacionados

Closes #[número del issue si existe]

## 📝 Descripción Este PR corrige la validación del archivo `.info.checked` en el endpoint `create_torrent_sum` para evitar errores cuando el archivo no existe. ## 🔧 Cambios realizados - ✅ Añadida validación para verificar la existencia del archivo `.info.checked` antes de proceder - ✅ Mejorado el manejo de errores con respuestas HTTP apropiadas (400 Bad Request) - ✅ Añadidos logs informativos para facilitar el debugging - ✅ Corregidos errores sintácticos menores en el código ## 🐛 Problema resuelto Anteriormente, el endpoint `create_torrent_sum` no validaba si el archivo `.info.checked` existía antes de intentar crear los archivos auxiliares, lo que podía causar errores inesperados. ## 🧪 Pruebas - [x] Verificado que el endpoint retorna error 400 cuando falta el archivo `.info.checked` - [x] Verificado que el endpoint funciona correctamente cuando el archivo existe - [x] Verificados los logs de systemd journal ## 📋 Checklist - [x] Los cambios siguen las convenciones de código del proyecto - [x] Se han añadido logs apropiados - [x] La funcionalidad existente no se ve afectada - [x] Los mensajes de error son descriptivos ## 🔗 Issues relacionados Closes #[número del issue si existe]
narenas added 6 commits 2025-09-01 12:19:46 +02:00
narenas merged commit f6617aee77 into main 2025-09-01 12:20:23 +02:00
Sign in to join this conversation.
Reviewers
No reviewers
Labels
Clear labels
No items
No Label
Milestone
Clear milestone
No items
No Milestone
Projects
Clear projects
No project
Assignees
Clear assignees
No Assignees
1 Participants
Notifications
Due Date
The due date is invalid or out of range. Please use the format 'yyyy-mm-dd'.

No due date set.

Dependencies

No dependencies set.

Reference: opengnsys/ogrepository#42
There is no content yet.
Delete Branch "fix_checked_file"

Deleting a branch is permanent. Although the deleted branch may exist for a short time before cleaning up, in most cases it CANNOT be undone. Continue?