Tall-Programacion-FIME / backend

Repositorio para la API del proyecto.
https://api.uanl.store
Apache License 2.0
0 stars 0 forks source link

Arreglar CORS #49

Closed sebastianmarines closed 3 years ago

sebastianmarines commented 3 years ago

Hay que eliminar todas las llamadas a la API de Elasticsearch

sebastianmarines commented 3 years ago

Hay que borrar esta linea: https://github.com/Tall-Programacion-FIME/backend/blob/9c97d13485f329f77ba51bf1f7746df9f390f08b/docker-compose.yml#L7

Y estos dos bloques: https://github.com/Tall-Programacion-FIME/backend/blob/9c97d13485f329f77ba51bf1f7746df9f390f08b/docker-compose.yml#L42-L47

https://github.com/Tall-Programacion-FIME/backend/blob/9c97d13485f329f77ba51bf1f7746df9f390f08b/docker-compose.yml#L48-L51

sebastianmarines commented 3 years ago

En todas las funciones CRUD de los libros hay que eliminar los parametros de elasticsearch y sus usos https://github.com/Tall-Programacion-FIME/backend/blob/9c97d13485f329f77ba51bf1f7746df9f390f08b/app/app/crud/crud_book.py#L7 https://github.com/Tall-Programacion-FIME/backend/blob/9c97d13485f329f77ba51bf1f7746df9f390f08b/app/app/crud/crud_book.py#L13 https://github.com/Tall-Programacion-FIME/backend/blob/9c97d13485f329f77ba51bf1f7746df9f390f08b/app/app/crud/crud_book.py#L24

sebastianmarines commented 3 years ago

En las rutas de los libros hay que eliminar la inyección de dependencias de Elasticsearch https://github.com/Tall-Programacion-FIME/backend/blob/9c97d13485f329f77ba51bf1f7746df9f390f08b/app/app/routers/endpoints/book.py#L46

y actualizar las llamadas a las funciones CRUD para no enviar la sesión de Elasticsearch como parámetro https://github.com/Tall-Programacion-FIME/backend/blob/9c97d13485f329f77ba51bf1f7746df9f390f08b/app/app/routers/endpoints/book.py#L73

sebastianmarines commented 3 years ago

@HectorMtz22

sebastianmarines commented 3 years ago

También hay que borrar la ruta de búsqueda https://github.com/Tall-Programacion-FIME/backend/blob/9c97d13485f329f77ba51bf1f7746df9f390f08b/app/app/routers/endpoints/book.py#L81-L96