More routes with no authentication check
Nadie ha tomado este issue todavía.
Evaluación
- Dificultad
- 4/5
- Tiempo estimado
- 3-5 días
- Aptitud para principiantes
- 45/100
- Tipo de issue
- Error
- Claridad
- Bastante claro
- Estado de actividad
- Activo
- Stack tecnológico
- docker, python
- Área
- authentication, backend
Línea de trabajo
Start by reading #366, the listed handlers in backend/routes/course.py, user.py, assignment.py, regrade_request.py and submission.py, and the helpers in util/auth.py. Check each route's callers and what access rule fits; then add authentication tests for 401, 403 and success cases. Done means every listed route has an appropriate check and its tests pass.
Escrito por el modelo de indexación a partir del texto del issue.
Descripción
While fixing #366 I scanned backend/routes/ for routes that never call require_authenticated, require_course_role, require_admin or one of the _verify_* helpers. These are left over. Each one needs a quick look to confirm it really has no check and to decide the right rule.
course.py
create_course: anyone can create a course and name any user as its instructor. The admin panel (pages/admin/courses/add.js) calls it.enroll_course: takesuser_idfrom the request body, so anyone with an entry code can enroll any user.leave_course: takesuser_idfrom the request body, so anyone can remove any student from any course.
user.py
get_user_by_email: returns a user's id, name, email and role to anyone. The instructor enrollment page calls it, so it probably needs a course staff rule rather than admin only.
assignment.py
get_assignment: returns full assignment details, including AI settings, to anyone with the id.get_extension,get_assignment_extensions: anyone can read extensions./courses
regrade_request.py
send_regrade_request,get_student_regrade_requests,check_regrade_request,delete_regrade_request
submission.py
test_autograder_submission: runs code in Docker, so this one matters most.
Suggested approach: same as #366. Use the helpers in util/auth.py, run auth before input checks, and add unit tests for 401, 403 and the success case.
- Lenguaje dominante
- Python
- Estrellas
- 13
- Forks
- 0
- Merge medio
- 2 h 4 min
- PR fusionados (30 d)
- 1
Preparar el entorno
- Incluye un Dockerfile o un archivo de Docker Compose
- Sin plantilla de pull request
- Sin guía de contribución
Primeros pasos
- Lee el issue completo y luego la guía de contribución del proyecto.
- Comenta en el issue que vas a ocuparte — evita que dos personas hagan lo mismo.
- Haz un fork del repositorio y trabaja en una rama.
- Abre un pull request que haga referencia al número del issue.
Más de kiat/codeAssist
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 68/100
kiat/codeAssist#391 ·
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 68/100
kiat/codeAssist#364 · 1 comentario ·
-
Dificultad 3/5 1-2 días Aptitud para principiantes 42/100
kiat/codeAssist#393 ·
-
Dificultad 3/5 1-2 días Aptitud para principiantes 63/100
kiat/codeAssist#392 ·
-
Dificultad 4/5 3-5 días Aptitud para principiantes 52/100
kiat/codeAssist#390 ·
Todos los issues de kiat/codeAssist
Issues similares
-
customer-reported question
Dificultad 2/5 1-3 horas Aptitud para principiantes 62/100
Azure/azure-sdk-for-python#49548 · 1 comentario · 1 reacción ·
Los mantenedores suelen responder en 1 día
-
Dificultad 1/5 Menos de una hora Aptitud para principiantes 88/100
btclib-org/btclib-wallet#409 ·
Los mantenedores suelen responder en 1 día
-
Phase 2 PSA Capability & Career Profile
Dificultad 2/5 1-3 horas Aptitud para principiantes 62/100
bcgov/digital-journeys#2310 ·
-
bug good first issue
Dificultad 2/5 1-3 horas Aptitud para principiantes 78/100
fastmachinelearning/hls4ml#1582 ·
Los mantenedores suelen responder en 3 días
-
Sort modelsAbiertoenhancement good first issue
Dificultad 2/5 1-3 horas Aptitud para principiantes 62/100
Los mantenedores suelen responder en 1 día