acm-udayton / acm-udayton/ACM-Meeting-Records

[FEATURE] Add JWT authorization requirement to all relevant backend endpoints

Abierto
#159 0 comentarios 0 reacciones 0 asignados Ver en GitHub
enhancement
Lenguaje dominante
Python
Estrellas
7
Forks
2
Merge medio
4 d 23 h
PR fusionados (30 d)
2

Descripción

## Feature Description

For each new /api/v1 endpoint that should require authentication, add the decorator(s) to enforce those policies.

---

## Acceptance Criteria & Solution Requirements
*To close this issue, the following must be met (can be finalized in the comments):*
- [ ] 1. Application runs as expected with the new feature integrated seamlessly.
- [ ] 2. Code quality metrics are maintained, and linting has been considered in development.
- [ ] 3. Testing metrics are maintained, and tests have been updated/added wherever relevant during development.
- [ ] 4. Documentation has been updated to reflect the changes made during development.
- [ ] 5. All endpoints have been considered for login requirement.
- [ ] 6. A new decorator has been added that requires that the logged-in user is also an admin. Use an `is_admin` boolean JWT key.
- [ ] 7. All endpoints have been considered for admin requirement.

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.